Node.nodeName should not be nullable
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 30 Apr 2016 00:12:35 +0000 (00:12 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 30 Apr 2016 00:12:35 +0000 (00:12 +0000)
https://bugs.webkit.org/show_bug.cgi?id=157211

Reviewed by Ryosuke Niwa.

Node.nodeName should not be nullable as per the specification:
https://dom.spec.whatwg.org/#interface-node

Our implementation never returns null anyway. However, having
it as nullable in the IDL means we use jsStringOrNull() instead
of jsStringWithCache(), thus doing an unnecessary null check.

This should not be observable by JS.

* dom/Node.idl:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@200271 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/dom/Node.idl

index 38a821c..27228c0 100644 (file)
@@ -1,3 +1,21 @@
+2016-04-29  Chris Dumez  <cdumez@apple.com>
+
+        Node.nodeName should not be nullable
+        https://bugs.webkit.org/show_bug.cgi?id=157211
+
+        Reviewed by Ryosuke Niwa.
+
+        Node.nodeName should not be nullable as per the specification:
+        https://dom.spec.whatwg.org/#interface-node
+
+        Our implementation never returns null anyway. However, having
+        it as nullable in the IDL means we use jsStringOrNull() instead
+        of jsStringWithCache(), thus doing an unnecessary null check.
+
+        This should not be observable by JS.
+
+        * dom/Node.idl:
+
 2016-04-29  Commit Queue  <commit-queue@webkit.org>
 
         Unreviewed, rolling out r200150 and r200256.
index 0089c5a..a3764c6 100644 (file)
@@ -47,7 +47,7 @@
     const unsigned short      DOCUMENT_FRAGMENT_NODE         = 11;
     const unsigned short      NOTATION_NODE                  = 12;
 
-    readonly attribute DOMString?        nodeName;
+    readonly attribute DOMString nodeName;
 
     // FIXME: the spec says this can also raise on retrieval.
     [SetterRaisesException] attribute DOMString? nodeValue;