Title: [231325] trunk
Revision
231325
Author
[email protected]
Date
2018-05-03 13:52:46 -0700 (Thu, 03 May 2018)

Log Message

Remove [NoInterfaceObject] from DOMRectList
https://bugs.webkit.org/show_bug.cgi?id=185255

Patch by Chris Nardi <[email protected]> on 2018-05-03
Reviewed by Chris Dumez.

LayoutTests/imported/w3c:

Update Geometry IDL test from upstream and update expectations.

* web-platform-tests/css/geometry/interfaces-expected.txt:
* web-platform-tests/css/geometry/interfaces.worker-expected.txt:
* web-platform-tests/interfaces/geometry.idl:

Source/WebCore:

In https://github.com/w3c/fxtf-drafts/issues/233, [NoInterfaceObject] was removed
from DOMRectList. Remove it from our implementation to match the spec, as well as
Chrome and Firefox.

Updated web platform tests IDL test for the Geometry spec.

* dom/DOMRectList.idl:

Modified Paths

Diff

Modified: trunk/LayoutTests/imported/w3c/ChangeLog (231324 => 231325)


--- trunk/LayoutTests/imported/w3c/ChangeLog	2018-05-03 20:39:33 UTC (rev 231324)
+++ trunk/LayoutTests/imported/w3c/ChangeLog	2018-05-03 20:52:46 UTC (rev 231325)
@@ -1,3 +1,16 @@
+2018-05-03  Chris Nardi  <[email protected]>
+
+        Remove [NoInterfaceObject] from DOMRectList
+        https://bugs.webkit.org/show_bug.cgi?id=185255
+
+        Reviewed by Chris Dumez.
+
+        Update Geometry IDL test from upstream and update expectations.
+
+        * web-platform-tests/css/geometry/interfaces-expected.txt:
+        * web-platform-tests/css/geometry/interfaces.worker-expected.txt:
+        * web-platform-tests/interfaces/geometry.idl:
+
 2018-05-02  Youenn Fablet  <[email protected]>
 
         Rebase imported/w3c/web-platform-tests/service-workers/service-worker/fetch-request-redirect.https.html expectation

Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/css/geometry/interfaces-expected.txt (231324 => 231325)


--- trunk/LayoutTests/imported/w3c/web-platform-tests/css/geometry/interfaces-expected.txt	2018-05-03 20:39:33 UTC (rev 231324)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/css/geometry/interfaces-expected.txt	2018-05-03 20:52:46 UTC (rev 231325)
@@ -18,6 +18,8 @@
 PASS Unscopable handled correctly for w property on DOMPointReadOnly 
 PASS DOMPointReadOnly interface: operation matrixTransform(DOMMatrixInit) 
 PASS Unscopable handled correctly for matrixTransform(DOMMatrixInit) on DOMPointReadOnly 
+PASS DOMPointReadOnly interface: operation toJSON() 
+PASS Unscopable handled correctly for toJSON() on DOMPointReadOnly 
 PASS DOMPointReadOnly must be primary interface of new DOMPointReadOnly() 
 PASS Stringification of new DOMPointReadOnly() 
 PASS DOMPointReadOnly interface: new DOMPointReadOnly() must inherit property "fromPoint(DOMPointInit)" with the proper type 
@@ -28,6 +30,8 @@
 PASS DOMPointReadOnly interface: new DOMPointReadOnly() must inherit property "w" with the proper type 
 PASS DOMPointReadOnly interface: new DOMPointReadOnly() must inherit property "matrixTransform(DOMMatrixInit)" with the proper type 
 PASS DOMPointReadOnly interface: calling matrixTransform(DOMMatrixInit) on new DOMPointReadOnly() with too few arguments must throw TypeError 
+PASS DOMPointReadOnly interface: new DOMPointReadOnly() must inherit property "toJSON()" with the proper type 
+PASS Test default toJSON operation of DOMPointReadOnly 
 PASS DOMPoint interface: existence and properties of interface object 
 PASS DOMPoint interface object length 
 PASS DOMPoint interface object name 
@@ -65,6 +69,8 @@
 PASS DOMPointReadOnly interface: new DOMPoint() must inherit property "w" with the proper type 
 PASS DOMPointReadOnly interface: new DOMPoint() must inherit property "matrixTransform(DOMMatrixInit)" with the proper type 
 PASS DOMPointReadOnly interface: calling matrixTransform(DOMMatrixInit) on new DOMPoint() with too few arguments must throw TypeError 
+PASS DOMPointReadOnly interface: new DOMPoint() must inherit property "toJSON()" with the proper type 
+PASS Test default toJSON operation of DOMPoint 
 PASS DOMRectReadOnly interface: existence and properties of interface object 
 PASS DOMRectReadOnly interface object length 
 PASS DOMRectReadOnly interface object name 
@@ -89,6 +95,8 @@
 PASS Unscopable handled correctly for bottom property on DOMRectReadOnly 
 PASS DOMRectReadOnly interface: attribute left 
 PASS Unscopable handled correctly for left property on DOMRectReadOnly 
+PASS DOMRectReadOnly interface: operation toJSON() 
+PASS Unscopable handled correctly for toJSON() on DOMRectReadOnly 
 PASS DOMRectReadOnly must be primary interface of new DOMRectReadOnly() 
 PASS Stringification of new DOMRectReadOnly() 
 PASS DOMRectReadOnly interface: new DOMRectReadOnly() must inherit property "fromRect(DOMRectInit)" with the proper type 
@@ -101,6 +109,8 @@
 PASS DOMRectReadOnly interface: new DOMRectReadOnly() must inherit property "right" with the proper type 
 PASS DOMRectReadOnly interface: new DOMRectReadOnly() must inherit property "bottom" with the proper type 
 PASS DOMRectReadOnly interface: new DOMRectReadOnly() must inherit property "left" with the proper type 
+PASS DOMRectReadOnly interface: new DOMRectReadOnly() must inherit property "toJSON()" with the proper type 
+PASS Test default toJSON operation of DOMRectReadOnly 
 PASS DOMRect interface: existence and properties of interface object 
 PASS DOMRect interface object length 
 PASS DOMRect interface object name 
@@ -140,6 +150,19 @@
 PASS DOMRectReadOnly interface: new DOMRect() must inherit property "right" with the proper type 
 PASS DOMRectReadOnly interface: new DOMRect() must inherit property "bottom" with the proper type 
 PASS DOMRectReadOnly interface: new DOMRect() must inherit property "left" with the proper type 
+PASS DOMRectReadOnly interface: new DOMRect() must inherit property "toJSON()" with the proper type 
+PASS Test default toJSON operation of DOMRect 
+PASS DOMRectList interface: existence and properties of interface object 
+PASS DOMRectList interface object length 
+PASS DOMRectList interface object name 
+PASS DOMRectList interface: existence and properties of interface prototype object 
+PASS DOMRectList interface: existence and properties of interface prototype object's "constructor" property 
+PASS DOMRectList interface: existence and properties of interface prototype object's @@unscopables property 
+PASS DOMRectList interface: attribute length 
+PASS Unscopable handled correctly for length property on DOMRectList 
+PASS DOMRectList interface: operation item(unsigned long) 
+PASS Unscopable handled correctly for item(unsigned long) on DOMRectList 
+PASS DOMRectList must be primary interface of [object DOMRectList] 
 PASS Stringification of [object DOMRectList] 
 PASS DOMRectList interface: [object DOMRectList] must inherit property "length" with the proper type 
 PASS DOMRectList interface: [object DOMRectList] must inherit property "item(unsigned long)" with the proper type 
@@ -164,6 +187,8 @@
 PASS Unscopable handled correctly for p4 property on DOMQuad 
 PASS DOMQuad interface: operation getBounds() 
 PASS Unscopable handled correctly for getBounds() on DOMQuad 
+PASS DOMQuad interface: operation toJSON() 
+PASS Unscopable handled correctly for toJSON() on DOMQuad 
 PASS DOMQuad must be primary interface of new DOMQuad() 
 PASS Stringification of new DOMQuad() 
 PASS DOMQuad interface: new DOMQuad() must inherit property "fromRect(DOMRectInit)" with the proper type 
@@ -175,6 +200,8 @@
 PASS DOMQuad interface: new DOMQuad() must inherit property "p3" with the proper type 
 PASS DOMQuad interface: new DOMQuad() must inherit property "p4" with the proper type 
 PASS DOMQuad interface: new DOMQuad() must inherit property "getBounds()" with the proper type 
+PASS DOMQuad interface: new DOMQuad() must inherit property "toJSON()" with the proper type 
+FAIL Test default toJSON operation of DOMQuad assert_true: instanceof DOMPoint expected true got false
 PASS DOMMatrixReadOnly interface: existence and properties of interface object 
 PASS DOMMatrixReadOnly interface object length 
 PASS DOMMatrixReadOnly interface object name 
@@ -266,6 +293,8 @@
 PASS DOMMatrixReadOnly interface: operation toFloat64Array() 
 PASS Unscopable handled correctly for toFloat64Array() on DOMMatrixReadOnly 
 PASS DOMMatrixReadOnly interface: stringifier 
+PASS DOMMatrixReadOnly interface: operation toJSON() 
+PASS Unscopable handled correctly for toJSON() on DOMMatrixReadOnly 
 PASS DOMMatrixReadOnly must be primary interface of new DOMMatrixReadOnly() 
 PASS Stringification of new DOMMatrixReadOnly() 
 PASS DOMMatrixReadOnly interface: new DOMMatrixReadOnly() must inherit property "fromMatrix(DOMMatrixInit)" with the proper type 
@@ -323,6 +352,8 @@
 PASS DOMMatrixReadOnly interface: calling transformPoint(DOMPointInit) on new DOMMatrixReadOnly() with too few arguments must throw TypeError 
 PASS DOMMatrixReadOnly interface: new DOMMatrixReadOnly() must inherit property "toFloat32Array()" with the proper type 
 PASS DOMMatrixReadOnly interface: new DOMMatrixReadOnly() must inherit property "toFloat64Array()" with the proper type 
+PASS DOMMatrixReadOnly interface: new DOMMatrixReadOnly() must inherit property "toJSON()" with the proper type 
+PASS Test default toJSON operation of DOMMatrixReadOnly 
 PASS DOMMatrixReadOnly must be primary interface of DOMMatrixReadOnly.fromMatrix({is2D: false}) 
 PASS Stringification of DOMMatrixReadOnly.fromMatrix({is2D: false}) 
 PASS DOMMatrix interface: existence and properties of interface object 

Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/css/geometry/interfaces.worker-expected.txt (231324 => 231325)


--- trunk/LayoutTests/imported/w3c/web-platform-tests/css/geometry/interfaces.worker-expected.txt	2018-05-03 20:39:33 UTC (rev 231324)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/css/geometry/interfaces.worker-expected.txt	2018-05-03 20:52:46 UTC (rev 231325)
@@ -18,6 +18,8 @@
 PASS Unscopable handled correctly for w property on DOMPointReadOnly 
 PASS DOMPointReadOnly interface: operation matrixTransform(DOMMatrixInit) 
 PASS Unscopable handled correctly for matrixTransform(DOMMatrixInit) on DOMPointReadOnly 
+PASS DOMPointReadOnly interface: operation toJSON() 
+PASS Unscopable handled correctly for toJSON() on DOMPointReadOnly 
 PASS DOMPointReadOnly must be primary interface of new DOMPointReadOnly() 
 PASS Stringification of new DOMPointReadOnly() 
 PASS DOMPointReadOnly interface: new DOMPointReadOnly() must inherit property "fromPoint(DOMPointInit)" with the proper type 
@@ -28,6 +30,8 @@
 PASS DOMPointReadOnly interface: new DOMPointReadOnly() must inherit property "w" with the proper type 
 PASS DOMPointReadOnly interface: new DOMPointReadOnly() must inherit property "matrixTransform(DOMMatrixInit)" with the proper type 
 PASS DOMPointReadOnly interface: calling matrixTransform(DOMMatrixInit) on new DOMPointReadOnly() with too few arguments must throw TypeError 
+PASS DOMPointReadOnly interface: new DOMPointReadOnly() must inherit property "toJSON()" with the proper type 
+PASS Test default toJSON operation of DOMPointReadOnly 
 PASS DOMPoint interface: existence and properties of interface object 
 PASS DOMPoint interface object length 
 PASS DOMPoint interface object name 
@@ -61,6 +65,8 @@
 PASS DOMPointReadOnly interface: new DOMPoint() must inherit property "w" with the proper type 
 PASS DOMPointReadOnly interface: new DOMPoint() must inherit property "matrixTransform(DOMMatrixInit)" with the proper type 
 PASS DOMPointReadOnly interface: calling matrixTransform(DOMMatrixInit) on new DOMPoint() with too few arguments must throw TypeError 
+PASS DOMPointReadOnly interface: new DOMPoint() must inherit property "toJSON()" with the proper type 
+PASS Test default toJSON operation of DOMPoint 
 PASS DOMRectReadOnly interface: existence and properties of interface object 
 PASS DOMRectReadOnly interface object length 
 PASS DOMRectReadOnly interface object name 
@@ -85,6 +91,8 @@
 PASS Unscopable handled correctly for bottom property on DOMRectReadOnly 
 PASS DOMRectReadOnly interface: attribute left 
 PASS Unscopable handled correctly for left property on DOMRectReadOnly 
+PASS DOMRectReadOnly interface: operation toJSON() 
+PASS Unscopable handled correctly for toJSON() on DOMRectReadOnly 
 PASS DOMRectReadOnly must be primary interface of new DOMRectReadOnly() 
 PASS Stringification of new DOMRectReadOnly() 
 PASS DOMRectReadOnly interface: new DOMRectReadOnly() must inherit property "fromRect(DOMRectInit)" with the proper type 
@@ -97,6 +105,8 @@
 PASS DOMRectReadOnly interface: new DOMRectReadOnly() must inherit property "right" with the proper type 
 PASS DOMRectReadOnly interface: new DOMRectReadOnly() must inherit property "bottom" with the proper type 
 PASS DOMRectReadOnly interface: new DOMRectReadOnly() must inherit property "left" with the proper type 
+PASS DOMRectReadOnly interface: new DOMRectReadOnly() must inherit property "toJSON()" with the proper type 
+PASS Test default toJSON operation of DOMRectReadOnly 
 PASS DOMRect interface: existence and properties of interface object 
 PASS DOMRect interface object length 
 PASS DOMRect interface object name 
@@ -132,6 +142,9 @@
 PASS DOMRectReadOnly interface: new DOMRect() must inherit property "right" with the proper type 
 PASS DOMRectReadOnly interface: new DOMRect() must inherit property "bottom" with the proper type 
 PASS DOMRectReadOnly interface: new DOMRect() must inherit property "left" with the proper type 
+PASS DOMRectReadOnly interface: new DOMRect() must inherit property "toJSON()" with the proper type 
+PASS Test default toJSON operation of DOMRect 
+PASS DOMRectList interface: existence and properties of interface object 
 PASS DOMQuad interface: existence and properties of interface object 
 PASS DOMQuad interface object length 
 PASS DOMQuad interface object name 
@@ -152,6 +165,8 @@
 PASS Unscopable handled correctly for p4 property on DOMQuad 
 PASS DOMQuad interface: operation getBounds() 
 PASS Unscopable handled correctly for getBounds() on DOMQuad 
+PASS DOMQuad interface: operation toJSON() 
+PASS Unscopable handled correctly for toJSON() on DOMQuad 
 PASS DOMQuad must be primary interface of new DOMQuad() 
 PASS Stringification of new DOMQuad() 
 PASS DOMQuad interface: new DOMQuad() must inherit property "fromRect(DOMRectInit)" with the proper type 
@@ -163,6 +178,8 @@
 PASS DOMQuad interface: new DOMQuad() must inherit property "p3" with the proper type 
 PASS DOMQuad interface: new DOMQuad() must inherit property "p4" with the proper type 
 PASS DOMQuad interface: new DOMQuad() must inherit property "getBounds()" with the proper type 
+PASS DOMQuad interface: new DOMQuad() must inherit property "toJSON()" with the proper type 
+FAIL Test default toJSON operation of DOMQuad assert_true: instanceof DOMPoint expected true got false
 PASS DOMMatrixReadOnly interface: existence and properties of interface object 
 PASS DOMMatrixReadOnly interface object length 
 PASS DOMMatrixReadOnly interface object name 
@@ -254,6 +271,8 @@
 PASS DOMMatrixReadOnly interface: operation toFloat64Array() 
 PASS Unscopable handled correctly for toFloat64Array() on DOMMatrixReadOnly 
 PASS DOMMatrixReadOnly interface: member undefined 
+PASS DOMMatrixReadOnly interface: operation toJSON() 
+PASS Unscopable handled correctly for toJSON() on DOMMatrixReadOnly 
 PASS DOMMatrixReadOnly must be primary interface of new DOMMatrixReadOnly() 
 PASS Stringification of new DOMMatrixReadOnly() 
 PASS DOMMatrixReadOnly interface: new DOMMatrixReadOnly() must inherit property "fromMatrix(DOMMatrixInit)" with the proper type 
@@ -312,6 +331,8 @@
 PASS DOMMatrixReadOnly interface: new DOMMatrixReadOnly() must inherit property "toFloat32Array()" with the proper type 
 PASS DOMMatrixReadOnly interface: new DOMMatrixReadOnly() must inherit property "toFloat64Array()" with the proper type 
 PASS DOMMatrixReadOnly interface: new DOMMatrixReadOnly() must not have property "undefined" 
+PASS DOMMatrixReadOnly interface: new DOMMatrixReadOnly() must inherit property "toJSON()" with the proper type 
+PASS Test default toJSON operation of DOMMatrixReadOnly 
 PASS DOMMatrixReadOnly must be primary interface of DOMMatrixReadOnly.fromMatrix({is2D: false}) 
 PASS Stringification of DOMMatrixReadOnly.fromMatrix({is2D: false}) 
 PASS DOMMatrix interface: existence and properties of interface object 

Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/interfaces/geometry.idl (231324 => 231325)


--- trunk/LayoutTests/imported/w3c/web-platform-tests/interfaces/geometry.idl	2018-05-03 20:39:33 UTC (rev 231324)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/interfaces/geometry.idl	2018-05-03 20:52:46 UTC (rev 231325)
@@ -1,3 +1,7 @@
+// GENERATED CONTENT - DO NOT EDIT
+// Content of this file was automatically extracted from the Geometry Interfaces spec.
+// See https://drafts.fxtf.org/geometry/
+
 [Constructor(optional unrestricted double x = 0, optional unrestricted double y = 0,
              optional unrestricted double z = 0, optional unrestricted double w = 1),
  Exposed=(Window,Worker),
@@ -12,7 +16,7 @@
 
     DOMPoint matrixTransform(optional DOMMatrixInit matrix);
 
-    [Default] toJSON();
+    [Default] object toJSON();
 };
 
 [Constructor(optional unrestricted double x = 0, optional unrestricted double y = 0,
@@ -52,7 +56,7 @@
     readonly attribute unrestricted double bottom;
     readonly attribute unrestricted double left;
 
-    [Default] toJSON();
+    [Default] object toJSON();
 };
 
 [Constructor(optional unrestricted double x = 0, optional unrestricted double y = 0,
@@ -76,8 +80,6 @@
     unrestricted double height = 0;
 };
 
-[NoInterfaceObject,
- LegacyArrayClass]
 interface DOMRectList {
     readonly attribute unsigned long length;
     getter DOMRect? item(unsigned long index);
@@ -97,7 +99,7 @@
     [SameObject] readonly attribute DOMPoint p4;
     [NewObject] DOMRect getBounds();
 
-    [Default] toJSON();
+    [Default] object toJSON();
 };
 
 dictionary DOMQuadInit {
@@ -178,7 +180,7 @@
     [NewObject] Float64Array toFloat64Array();
 
     [Exposed=Window] stringifier;
-    [Default] toJSON();
+    [Default] object toJSON();
 };
 
 [Constructor(optional (DOMString or sequence<unrestricted double>) init),

Modified: trunk/Source/WebCore/ChangeLog (231324 => 231325)


--- trunk/Source/WebCore/ChangeLog	2018-05-03 20:39:33 UTC (rev 231324)
+++ trunk/Source/WebCore/ChangeLog	2018-05-03 20:52:46 UTC (rev 231325)
@@ -1,3 +1,18 @@
+2018-05-03  Chris Nardi  <[email protected]>
+
+        Remove [NoInterfaceObject] from DOMRectList
+        https://bugs.webkit.org/show_bug.cgi?id=185255
+
+        Reviewed by Chris Dumez.
+
+        In https://github.com/w3c/fxtf-drafts/issues/233, [NoInterfaceObject] was removed
+        from DOMRectList. Remove it from our implementation to match the spec, as well as
+        Chrome and Firefox.
+
+        Updated web platform tests IDL test for the Geometry spec.
+
+        * dom/DOMRectList.idl:
+
 2018-05-03  Chris Dumez  <[email protected]>
 
         REGRESSION(iOS 11.3): Crashes in TimerBase::~TimerBase() in Tencent x5gamehelper

Modified: trunk/Source/WebCore/dom/DOMRectList.idl (231324 => 231325)


--- trunk/Source/WebCore/dom/DOMRectList.idl	2018-05-03 20:39:33 UTC (rev 231324)
+++ trunk/Source/WebCore/dom/DOMRectList.idl	2018-05-03 20:52:46 UTC (rev 231325)
@@ -25,8 +25,7 @@
 
 [
      ExportMacro=WEBCORE_EXPORT,
-     ImplementationLacksVTable,
-     NoInterfaceObject
+     ImplementationLacksVTable
 ] interface DOMRectList {
     readonly attribute unsigned long length;
     getter DOMRect item(unsigned long index);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to