Remove [NoInterfaceObject] from XPathExpression and NodeIterator
authorch.dumez@sisa.samsung.com <ch.dumez@sisa.samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 4 Jun 2013 20:27:29 +0000 (20:27 +0000)
committerch.dumez@sisa.samsung.com <ch.dumez@sisa.samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 4 Jun 2013 20:27:29 +0000 (20:27 +0000)
https://bugs.webkit.org/show_bug.cgi?id=117204

Reviewed by Alexey Proskuryakov.

Source/WebCore:

Remove [NoInterfaceObject] from XPathExpression and NodeIterator to match
their respective specifications:
- http://www.w3.org/TR/DOM-Level-3-XPath/xpath.html#XPathExpression
- http://www.w3.org/TR/DOM-Level-2-Traversal-Range/traversal.html#Traversal-NodeIterator

No new tests, already covered by:
fast/js/global-constructors-attributes.html

* dom/NodeIterator.idl:
* xml/XPathExpression.idl:

LayoutTests:

Rebaseline fast/js/global-constructors-attributes.html and fast/dom/dom-constructors.html
now that XPathExpression and NodeIterator properties are exposed on the global Window
object.

* fast/dom/dom-constructors-expected.txt:
* fast/js/global-constructors-attributes-expected.txt:
* platform/efl/fast/js/global-constructors-attributes-expected.txt:
* platform/gtk/fast/js/global-constructors-attributes-expected.txt:
* platform/mac-lion/fast/js/global-constructors-attributes-expected.txt:
* platform/qt/fast/js/global-constructors-attributes-expected.txt:

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

LayoutTests/ChangeLog
LayoutTests/fast/dom/dom-constructors-expected.txt
LayoutTests/fast/js/global-constructors-attributes-expected.txt
LayoutTests/platform/efl/fast/js/global-constructors-attributes-expected.txt
LayoutTests/platform/gtk/fast/js/global-constructors-attributes-expected.txt
LayoutTests/platform/mac-lion/fast/js/global-constructors-attributes-expected.txt
LayoutTests/platform/qt/fast/js/global-constructors-attributes-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/dom/NodeIterator.idl
Source/WebCore/xml/XPathExpression.idl

index c891c55..a96d615 100644 (file)
@@ -1,3 +1,21 @@
+2013-06-04  Christophe Dumez  <ch.dumez@sisa.samsung.com>
+
+        Remove [NoInterfaceObject] from XPathExpression and NodeIterator
+        https://bugs.webkit.org/show_bug.cgi?id=117204
+
+        Reviewed by Alexey Proskuryakov.
+
+        Rebaseline fast/js/global-constructors-attributes.html and fast/dom/dom-constructors.html
+        now that XPathExpression and NodeIterator properties are exposed on the global Window
+        object.
+
+        * fast/dom/dom-constructors-expected.txt:
+        * fast/js/global-constructors-attributes-expected.txt:
+        * platform/efl/fast/js/global-constructors-attributes-expected.txt:
+        * platform/gtk/fast/js/global-constructors-attributes-expected.txt:
+        * platform/mac-lion/fast/js/global-constructors-attributes-expected.txt:
+        * platform/qt/fast/js/global-constructors-attributes-expected.txt:
+
 2013-06-04  Greg Hughes  <ghughes@apple.com>
 
         Add "VisibleOnly" key to search predicate
index 0641e2d..5da6635 100644 (file)
@@ -117,11 +117,11 @@ PASS TryAllocate('History') is 'exception'
 PASS TryAllocate('HTMLOptionsCollection') is 'exception'
 PASS TryAllocate('Location') is 'exception'
 PASS TryAllocate('Navigator') is 'exception'
-FAIL TryAllocate('NodeIterator') should be exception. Was no constructor.
+PASS TryAllocate('NodeIterator') is 'exception'
 PASS TryAllocate('RGBColor') is 'exception'
 PASS TryAllocate('Screen') is 'exception'
 FAIL TryAllocate('TreeWalker') should be exception. Was no constructor.
-FAIL TryAllocate('XPathExpression') should be exception. Was no constructor.
+PASS TryAllocate('XPathExpression') is 'exception'
 PASS TryAllocate('Worker') is 'exception'
 PASS TryAllocate('EventTargetNode') is 'no constructor'
 PASS TryAllocate('UndetectableHTMLCollection') is 'no constructor'
index bdb7b1e..ad41977 100644 (file)
@@ -858,6 +858,11 @@ PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').hasOwnProperty('get')
 PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').enumerable is false
 PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').configurable is true
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').value is NodeIterator
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').hasOwnProperty('get') is false
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').hasOwnProperty('set') is false
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').enumerable is false
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').configurable is true
 PASS Object.getOwnPropertyDescriptor(global, 'NodeList').value is NodeList
 PASS Object.getOwnPropertyDescriptor(global, 'NodeList').hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(global, 'NodeList').hasOwnProperty('set') is false
@@ -1978,6 +1983,11 @@ PASS Object.getOwnPropertyDescriptor(global, 'XPathException').hasOwnProperty('g
 PASS Object.getOwnPropertyDescriptor(global, 'XPathException').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(global, 'XPathException').enumerable is false
 PASS Object.getOwnPropertyDescriptor(global, 'XPathException').configurable is true
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').value is XPathExpression
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').hasOwnProperty('get') is false
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').hasOwnProperty('set') is false
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').enumerable is false
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').configurable is true
 PASS Object.getOwnPropertyDescriptor(global, 'XPathResult').value is XPathResult
 PASS Object.getOwnPropertyDescriptor(global, 'XPathResult').hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(global, 'XPathResult').hasOwnProperty('set') is false
index 6d9994b..f02e3c8 100644 (file)
@@ -928,6 +928,11 @@ PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').hasOwnProperty('get')
 PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').enumerable is false
 PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').configurable is true
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').value is NodeIterator
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').hasOwnProperty('get') is false
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').hasOwnProperty('set') is false
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').enumerable is false
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').configurable is true
 PASS Object.getOwnPropertyDescriptor(global, 'NodeList').value is NodeList
 PASS Object.getOwnPropertyDescriptor(global, 'NodeList').hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(global, 'NodeList').hasOwnProperty('set') is false
@@ -2058,6 +2063,11 @@ PASS Object.getOwnPropertyDescriptor(global, 'XPathException').hasOwnProperty('g
 PASS Object.getOwnPropertyDescriptor(global, 'XPathException').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(global, 'XPathException').enumerable is false
 PASS Object.getOwnPropertyDescriptor(global, 'XPathException').configurable is true
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').value is XPathExpression
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').hasOwnProperty('get') is false
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').hasOwnProperty('set') is false
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').enumerable is false
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').configurable is true
 PASS Object.getOwnPropertyDescriptor(global, 'XPathResult').value is XPathResult
 PASS Object.getOwnPropertyDescriptor(global, 'XPathResult').hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(global, 'XPathResult').hasOwnProperty('set') is false
index 16c59ac..b8e40b0 100644 (file)
@@ -923,6 +923,11 @@ PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').hasOwnProperty('get')
 PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').enumerable is false
 PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').configurable is true
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').value is NodeIterator
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').hasOwnProperty('get') is false
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').hasOwnProperty('set') is false
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').enumerable is false
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').configurable is true
 PASS Object.getOwnPropertyDescriptor(global, 'NodeList').value is NodeList
 PASS Object.getOwnPropertyDescriptor(global, 'NodeList').hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(global, 'NodeList').hasOwnProperty('set') is false
@@ -2063,6 +2068,11 @@ PASS Object.getOwnPropertyDescriptor(global, 'XPathException').hasOwnProperty('g
 PASS Object.getOwnPropertyDescriptor(global, 'XPathException').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(global, 'XPathException').enumerable is false
 PASS Object.getOwnPropertyDescriptor(global, 'XPathException').configurable is true
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').value is XPathExpression
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').hasOwnProperty('get') is false
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').hasOwnProperty('set') is false
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').enumerable is false
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').configurable is true
 PASS Object.getOwnPropertyDescriptor(global, 'XPathResult').value is XPathResult
 PASS Object.getOwnPropertyDescriptor(global, 'XPathResult').hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(global, 'XPathResult').hasOwnProperty('set') is false
index 925763b..2d0c204 100644 (file)
@@ -858,6 +858,11 @@ PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').hasOwnProperty('get')
 PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').enumerable is false
 PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').configurable is true
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').value is NodeIterator
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').hasOwnProperty('get') is false
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').hasOwnProperty('set') is false
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').enumerable is false
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').configurable is true
 PASS Object.getOwnPropertyDescriptor(global, 'NodeList').value is NodeList
 PASS Object.getOwnPropertyDescriptor(global, 'NodeList').hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(global, 'NodeList').hasOwnProperty('set') is false
@@ -1973,6 +1978,11 @@ PASS Object.getOwnPropertyDescriptor(global, 'XPathException').hasOwnProperty('g
 PASS Object.getOwnPropertyDescriptor(global, 'XPathException').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(global, 'XPathException').enumerable is false
 PASS Object.getOwnPropertyDescriptor(global, 'XPathException').configurable is true
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').value is XPathExpression
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').hasOwnProperty('get') is false
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').hasOwnProperty('set') is false
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').enumerable is false
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').configurable is true
 PASS Object.getOwnPropertyDescriptor(global, 'XPathResult').value is XPathResult
 PASS Object.getOwnPropertyDescriptor(global, 'XPathResult').hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(global, 'XPathResult').hasOwnProperty('set') is false
index fae6edd..1f53818 100644 (file)
@@ -788,6 +788,11 @@ PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').hasOwnProperty('get')
 PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').enumerable is false
 PASS Object.getOwnPropertyDescriptor(global, 'NodeFilter').configurable is true
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').value is NodeIterator
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').hasOwnProperty('get') is false
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').hasOwnProperty('set') is false
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').enumerable is false
+PASS Object.getOwnPropertyDescriptor(global, 'NodeIterator').configurable is true
 PASS Object.getOwnPropertyDescriptor(global, 'NodeList').value is NodeList
 PASS Object.getOwnPropertyDescriptor(global, 'NodeList').hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(global, 'NodeList').hasOwnProperty('set') is false
@@ -1878,6 +1883,11 @@ PASS Object.getOwnPropertyDescriptor(global, 'XPathException').hasOwnProperty('g
 PASS Object.getOwnPropertyDescriptor(global, 'XPathException').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(global, 'XPathException').enumerable is false
 PASS Object.getOwnPropertyDescriptor(global, 'XPathException').configurable is true
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').value is XPathExpression
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').hasOwnProperty('get') is false
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').hasOwnProperty('set') is false
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').enumerable is false
+PASS Object.getOwnPropertyDescriptor(global, 'XPathExpression').configurable is true
 PASS Object.getOwnPropertyDescriptor(global, 'XPathResult').value is XPathResult
 PASS Object.getOwnPropertyDescriptor(global, 'XPathResult').hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(global, 'XPathResult').hasOwnProperty('set') is false
index 02ca9aa..9d3dbf2 100644 (file)
@@ -1,3 +1,21 @@
+2013-06-04  Christophe Dumez  <ch.dumez@sisa.samsung.com>
+
+        Remove [NoInterfaceObject] from XPathExpression and NodeIterator
+        https://bugs.webkit.org/show_bug.cgi?id=117204
+
+        Reviewed by Alexey Proskuryakov.
+
+        Remove [NoInterfaceObject] from XPathExpression and NodeIterator to match
+        their respective specifications:
+        - http://www.w3.org/TR/DOM-Level-3-XPath/xpath.html#XPathExpression
+        - http://www.w3.org/TR/DOM-Level-2-Traversal-Range/traversal.html#Traversal-NodeIterator
+
+        No new tests, already covered by:
+        fast/js/global-constructors-attributes.html
+
+        * dom/NodeIterator.idl:
+        * xml/XPathExpression.idl:
+
 2013-06-04  Chris Fleizach  <cfleizach@apple.com>
 
         AX: MathML multiscripts not exposed in Accessibility
index 7f8cd31..77259d0 100644 (file)
@@ -20,7 +20,6 @@
 
 // Introduced in DOM Level 2:
 [
-    NoInterfaceObject,
     JSCustomMarkFunction,
     ImplementationLacksVTable
 ] interface NodeIterator {
index 019b4d0..e8aaa9c 100644 (file)
@@ -18,7 +18,6 @@
  * Boston, MA 02110-1301, USA.
  */
 [
-    NoInterfaceObject,
     ImplementationLacksVTable
 ] interface XPathExpression {
     [ObjCLegacyUnnamedParameters] XPathResult evaluate([Default=Undefined] optional Node contextNode,