Un-expose DOMSettableTokenList
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 7 Aug 2016 17:14:54 +0000 (17:14 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 7 Aug 2016 17:14:54 +0000 (17:14 +0000)
https://bugs.webkit.org/show_bug.cgi?id=160643

Reviewed by Darin Adler.

LayoutTests/imported/w3c:

Rebaseline W3C test now that one more check is passing.

* web-platform-tests/dom/historical-expected.txt:

Source/WebCore:

Un-expose DOMSettableTokenList. We merged DOMSettableTokenList into
DOMTokenList a while back, as per the specification. At the time, we
kept DOMSettableTokenList on Window, as an alias to DOMTokenList.
However, Firefox and Chrome dropped DOMSettableTokenList completely.
We now do the same as other browsers.

No new tests, rebaselined existing test.

* page/DOMWindow.idl:

LayoutTests:

Update / rebaseline existing tests.

* fast/dom/HTMLLinkElement/sizes-setter-expected.txt:
* fast/dom/HTMLLinkElement/sizes-setter.html:
* fast/dom/HTMLOutputElement/dom-settable-token-list-expected.txt:
* fast/dom/HTMLOutputElement/script-tests/dom-settable-token-list.js:
* fast/frames/sandbox-attribute-expected.txt:
* fast/frames/sandbox-attribute.html:
* js/dom/global-constructors-attributes-expected.txt:
* platform/efl/js/dom/global-constructors-attributes-expected.txt:
* platform/gtk/js/dom/global-constructors-attributes-expected.txt:
* platform/mac-yosemite/js/dom/global-constructors-attributes-expected.txt:
* platform/mac/js/dom/global-constructors-attributes-expected.txt:
* platform/win/js/dom/global-constructors-attributes-expected.txt:

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

17 files changed:
LayoutTests/ChangeLog
LayoutTests/fast/dom/HTMLLinkElement/sizes-setter-expected.txt
LayoutTests/fast/dom/HTMLLinkElement/sizes-setter.html
LayoutTests/fast/dom/HTMLOutputElement/dom-settable-token-list-expected.txt
LayoutTests/fast/dom/HTMLOutputElement/script-tests/dom-settable-token-list.js
LayoutTests/fast/frames/sandbox-attribute-expected.txt
LayoutTests/fast/frames/sandbox-attribute.html
LayoutTests/imported/w3c/ChangeLog
LayoutTests/imported/w3c/web-platform-tests/dom/historical-expected.txt
LayoutTests/js/dom/global-constructors-attributes-expected.txt
LayoutTests/platform/efl/js/dom/global-constructors-attributes-expected.txt
LayoutTests/platform/gtk/js/dom/global-constructors-attributes-expected.txt
LayoutTests/platform/mac-yosemite/js/dom/global-constructors-attributes-expected.txt
LayoutTests/platform/mac/js/dom/global-constructors-attributes-expected.txt
LayoutTests/platform/win/js/dom/global-constructors-attributes-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/page/DOMWindow.idl

index d3b002c..f370f39 100644 (file)
@@ -1,3 +1,25 @@
+2016-08-07  Chris Dumez  <cdumez@apple.com>
+
+        Un-expose DOMSettableTokenList
+        https://bugs.webkit.org/show_bug.cgi?id=160643
+
+        Reviewed by Darin Adler.
+
+        Update / rebaseline existing tests.
+
+        * fast/dom/HTMLLinkElement/sizes-setter-expected.txt:
+        * fast/dom/HTMLLinkElement/sizes-setter.html:
+        * fast/dom/HTMLOutputElement/dom-settable-token-list-expected.txt:
+        * fast/dom/HTMLOutputElement/script-tests/dom-settable-token-list.js:
+        * fast/frames/sandbox-attribute-expected.txt:
+        * fast/frames/sandbox-attribute.html:
+        * js/dom/global-constructors-attributes-expected.txt:
+        * platform/efl/js/dom/global-constructors-attributes-expected.txt:
+        * platform/gtk/js/dom/global-constructors-attributes-expected.txt:
+        * platform/mac-yosemite/js/dom/global-constructors-attributes-expected.txt:
+        * platform/mac/js/dom/global-constructors-attributes-expected.txt:
+        * platform/win/js/dom/global-constructors-attributes-expected.txt:
+
 2016-08-07  Youenn Fablet  <youenn@apple.com>
 
         LayoutTest http/tests/fetch/fetch-in-worker-crash.html is flaky
index 8e9c38d..f3d8f45 100644 (file)
@@ -3,7 +3,7 @@ Tests that the HTMLLinkElement.sizes setter updates the associated sizes attribu
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
 
-PASS link.sizes.__proto__ is DOMSettableTokenList.prototype
+PASS link.sizes.__proto__ is DOMTokenList.prototype
 link.sizes = '10x10  20x20'
 PASS String(link.sizes) is "10x10 20x20"
 PASS link.getAttribute('sizes') is "10x10 20x20"
index 8d4b35f..5d12a2c 100644 (file)
@@ -6,7 +6,7 @@
 description("Tests that the HTMLLinkElement.sizes setter updates the associated sizes attribute");
 
 var link = document.createElement("link");
-shouldBe("link.sizes.__proto__", "DOMSettableTokenList.prototype");
+shouldBe("link.sizes.__proto__", "DOMTokenList.prototype");
 evalAndLog("link.sizes = '10x10  20x20'");
 shouldBeEqualToString("String(link.sizes)", "10x10 20x20");
 shouldBeEqualToString("link.getAttribute('sizes')", "10x10 20x20");
index b059eb8..a1c9cfd 100644 (file)
@@ -4,7 +4,7 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
 
 
 - Tests from http://simon.html5.org/test/html/dom/reflecting/DOMTokenList/
-PASS element.htmlFor.__proto__ is DOMSettableTokenList.prototype
+PASS element.htmlFor.__proto__ is DOMTokenList.prototype
 PASS String(element.htmlFor) is "y z"
 PASS element.getAttribute("for") is "y z"
 PASS String(element.htmlFor) is "r s t"
@@ -66,11 +66,11 @@ PASS element.htmlFor.length is 2
 PASS element.htmlFor.length is 2
 PASS element.htmlFor.length is 2
 PASS element.htmlFor.length is 2
-- DOMSettableTokenList presence and type
-PASS 'undefined' != typeof DOMSettableTokenList is true
-PASS typeof DOMSettableTokenList.prototype is "object"
+- DOMTokenList presence and type
+PASS 'undefined' != typeof DOMTokenList is true
+PASS typeof DOMTokenList.prototype is "object"
 PASS typeof element.htmlFor is "object"
-PASS element.htmlFor.constructor is DOMSettableTokenList
+PASS element.htmlFor.constructor is DOMTokenList
 PASS element.htmlFor === element.htmlFor is true
 PASS successfullyParsed is true
 
index de185bf..8eb77aa 100644 (file)
@@ -16,7 +16,7 @@ debug('- Tests from http://simon.html5.org/test/html/dom/reflecting/DOMTokenList
 
 // HTMLOutputElement::htmlFor should be settable.
 createElement('x');
-shouldBe('element.htmlFor.__proto__', 'DOMSettableTokenList.prototype');
+shouldBe('element.htmlFor.__proto__', 'DOMTokenList.prototype');
 element.htmlFor = 'y  z';
 shouldBeEqualToString('String(element.htmlFor)', 'y z');
 shouldBeEqualToString('element.getAttribute("for")', 'y z');
@@ -232,21 +232,21 @@ createElement('x\u000Dy');
 shouldEvaluateTo('element.htmlFor.length', 2);
 
 
-debug('- DOMSettableTokenList presence and type');
+debug('- DOMTokenList presence and type');
 
 
 // Safari returns object
 // Firefox returns object
 // IE8 returns object
 // Chrome returns function
-// assertEquals('object', typeof DOMSettableTokenList);
-shouldBeTrue('\'undefined\' != typeof DOMSettableTokenList');
+// assertEquals('object', typeof DOMTokenList);
+shouldBeTrue('\'undefined\' != typeof DOMTokenList');
 
-shouldBeEqualToString('typeof DOMSettableTokenList.prototype', 'object');
+shouldBeEqualToString('typeof DOMTokenList.prototype', 'object');
 
 createElement('x');
 shouldBeEqualToString('typeof element.htmlFor', 'object');
 
-shouldEvaluateTo('element.htmlFor.constructor', 'DOMSettableTokenList');
+shouldEvaluateTo('element.htmlFor.constructor', 'DOMTokenList');
 
 shouldBeTrue('element.htmlFor === element.htmlFor');
index 8aceede..0abab1a 100644 (file)
@@ -1,9 +1,9 @@
-Checks that HTMLIframeElement.sandbox is a DOMSettableTokenList.
+Checks that HTMLIframeElement.sandbox is a DOMTokenList.
 
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
 
-PASS iframe.sandbox.__proto__ is DOMSettableTokenList.prototype
+PASS iframe.sandbox.__proto__ is DOMTokenList.prototype
 
 iframe.sandbox = 'allow-popups   allow-scripts  allow-scripts  allow-same-origin'
 PASS iframe.sandbox.length is 3
index 7a6dc30..2a34c08 100644 (file)
@@ -3,10 +3,10 @@
 <body>
 <script src="../../resources/js-test-pre.js"></script>
 <script>
-description("Checks that HTMLIframeElement.sandbox is a DOMSettableTokenList.");
+description("Checks that HTMLIframeElement.sandbox is a DOMTokenList.");
 
 var iframe = document.createElement("iframe");
-shouldBe("iframe.sandbox.__proto__", "DOMSettableTokenList.prototype");
+shouldBe("iframe.sandbox.__proto__", "DOMTokenList.prototype");
 
 debug("");
 evalAndLog("iframe.sandbox = 'allow-popups   allow-scripts  allow-scripts  allow-same-origin'");
index 443e397..3ccfbac 100644 (file)
@@ -1,3 +1,14 @@
+2016-08-07  Chris Dumez  <cdumez@apple.com>
+
+        Un-expose DOMSettableTokenList
+        https://bugs.webkit.org/show_bug.cgi?id=160643
+
+        Reviewed by Darin Adler.
+
+        Rebaseline W3C test now that one more check is passing.
+
+        * web-platform-tests/dom/historical-expected.txt:
+
 2016-08-06  Chris Dumez  <cdumez@apple.com>
 
         a.replaceChild(a, a) should throw a HierarchyRequestError
index fa2debe..c3f73c2 100644 (file)
@@ -9,9 +9,7 @@ PASS Historical DOM features must be removed: DOMImplementationList
 PASS Historical DOM features must be removed: DOMImplementationSource 
 PASS Historical DOM features must be removed: DOMLocator 
 PASS Historical DOM features must be removed: DOMObject 
-FAIL Historical DOM features must be removed: DOMSettableTokenList assert_equals: expected (undefined) undefined but got (function) function "function DOMTokenList() {
-    [native code]
-}"
+PASS Historical DOM features must be removed: DOMSettableTokenList 
 PASS Historical DOM features must be removed: DOMUserData 
 PASS Historical DOM features must be removed: Entity 
 PASS Historical DOM features must be removed: EntityReference 
index 5e160e1..37aca91 100644 (file)
@@ -283,11 +283,6 @@ PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').hasOwnProperty('get')
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').enumerable is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').configurable is true
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').value is DOMSettableTokenList
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').hasOwnProperty('get') is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').hasOwnProperty('set') is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').enumerable is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').configurable is true
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').value is DOMStringList
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').hasOwnProperty('set') is false
index 9a947ba..d1f4c56 100644 (file)
@@ -283,11 +283,6 @@ PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').hasOwnProperty('get')
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').enumerable is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').configurable is true
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').value is DOMSettableTokenList
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').hasOwnProperty('get') is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').hasOwnProperty('set') is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').enumerable is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').configurable is true
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').value is DOMStringList
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').hasOwnProperty('set') is false
index aa5293d..a4585b3 100644 (file)
@@ -283,11 +283,6 @@ PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').hasOwnProperty('get')
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').enumerable is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').configurable is true
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').value is DOMSettableTokenList
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').hasOwnProperty('get') is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').hasOwnProperty('set') is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').enumerable is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').configurable is true
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').value is DOMStringList
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').hasOwnProperty('set') is false
index b615415..cb5f64b 100644 (file)
@@ -303,11 +303,6 @@ PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').hasOwnProperty('get')
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').enumerable is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').configurable is true
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').value is DOMSettableTokenList
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').hasOwnProperty('get') is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').hasOwnProperty('set') is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').enumerable is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').configurable is true
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').value is DOMStringList
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').hasOwnProperty('set') is false
index 5fd98e6..7fff16c 100644 (file)
@@ -303,11 +303,6 @@ PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').hasOwnProperty('get')
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').enumerable is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').configurable is true
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').value is DOMSettableTokenList
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').hasOwnProperty('get') is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').hasOwnProperty('set') is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').enumerable is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').configurable is true
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').value is DOMStringList
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').hasOwnProperty('set') is false
index be87b7c..51875d9 100644 (file)
@@ -218,11 +218,6 @@ PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').hasOwnProperty('get')
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').hasOwnProperty('set') is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').enumerable is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMParser').configurable is true
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').value is DOMSettableTokenList
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').hasOwnProperty('get') is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').hasOwnProperty('set') is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').enumerable is false
-PASS Object.getOwnPropertyDescriptor(global, 'DOMSettableTokenList').configurable is true
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').value is DOMStringList
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').hasOwnProperty('get') is false
 PASS Object.getOwnPropertyDescriptor(global, 'DOMStringList').hasOwnProperty('set') is false
index e5dc2de..29f259e 100644 (file)
@@ -1,3 +1,20 @@
+2016-08-07  Chris Dumez  <cdumez@apple.com>
+
+        Un-expose DOMSettableTokenList
+        https://bugs.webkit.org/show_bug.cgi?id=160643
+
+        Reviewed by Darin Adler.
+
+        Un-expose DOMSettableTokenList. We merged DOMSettableTokenList into
+        DOMTokenList a while back, as per the specification. At the time, we
+        kept DOMSettableTokenList on Window, as an alias to DOMTokenList.
+        However, Firefox and Chrome dropped DOMSettableTokenList completely.
+        We now do the same as other browsers.
+
+        No new tests, rebaselined existing test.
+
+        * page/DOMWindow.idl:
+
 2016-08-07  Andreas Kling  <akling@apple.com>
 
         DocumentRuleSets::collectFeatures() should shrink-to-fit.
index 06e5f78..8af422e 100644 (file)
 #if defined(LANGUAGE_JAVASCRIPT) && LANGUAGE_JAVASCRIPT
     // Additional constructors.
     [CustomGetter, CustomConstructor] attribute HTMLImageElementNamedConstructor Image; // Usable with new operator
-    attribute DOMTokenListConstructor DOMSettableTokenList; // Map DOMSettableTokenList to DOMTokenList for backward compatibility.
 
     attribute DOMURLConstructor webkitURL; // FIXME: deprecate this.
     attribute MutationObserverConstructor WebKitMutationObserver; // FIXME: Add metrics to determine when we can remove this.