Drop support for non-standard document.all.tags()
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 9 Mar 2017 02:37:42 +0000 (02:37 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 9 Mar 2017 02:37:42 +0000 (02:37 +0000)
https://bugs.webkit.org/show_bug.cgi?id=169337

Reviewed by Sam Weinig.

LayoutTests/imported/w3c:

Rebaseline web platform test now that one more check is passing.

* web-platform-tests/html/infrastructure/common-dom-interfaces/collections/htmlallcollection-expected.txt:

Source/WebCore:

Drop support for non-standard document.all.tags(). It is not part of the specification:
- https://html.spec.whatwg.org/multipage/infrastructure.html#the-htmlallcollection-interface

It is not supported by Firefox and its support was dropped from Chrome back in early 2014:
- https://src.chromium.org/viewvc/blink?view=revision&revision=166870

No new tests, updated existing test.

* html/HTMLAllCollection.cpp:
* html/HTMLAllCollection.h:
* html/HTMLAllCollection.idl:

LayoutTests:

Update existing test to check that document.all.tags does not exist.

* fast/dom/document-all-tags-expected.txt:
* fast/dom/document-all-tags.html:

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

LayoutTests/ChangeLog
LayoutTests/fast/dom/HTMLDocument/document-all-expected.txt [deleted file]
LayoutTests/fast/dom/HTMLDocument/document-all.html [deleted file]
LayoutTests/fast/dom/document-all-tags-expected.txt
LayoutTests/fast/dom/document-all-tags.html
LayoutTests/imported/w3c/ChangeLog
LayoutTests/imported/w3c/web-platform-tests/html/infrastructure/common-dom-interfaces/collections/htmlallcollection-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/html/HTMLAllCollection.cpp
Source/WebCore/html/HTMLAllCollection.h
Source/WebCore/html/HTMLAllCollection.idl

index 782956e..907b794 100644 (file)
@@ -1,5 +1,17 @@
 2017-03-08  Chris Dumez  <cdumez@apple.com>
 
+        Drop support for non-standard document.all.tags()
+        https://bugs.webkit.org/show_bug.cgi?id=169337
+
+        Reviewed by Sam Weinig.
+
+        Update existing test to check that document.all.tags does not exist.
+
+        * fast/dom/document-all-tags-expected.txt:
+        * fast/dom/document-all-tags.html:
+
+2017-03-08  Chris Dumez  <cdumez@apple.com>
+
         Drop select.remove() / options.remove() overloads taking an option element in parameter
         https://bugs.webkit.org/show_bug.cgi?id=169339
 
diff --git a/LayoutTests/fast/dom/HTMLDocument/document-all-expected.txt b/LayoutTests/fast/dom/HTMLDocument/document-all-expected.txt
deleted file mode 100644 (file)
index 5fb7b10..0000000
+++ /dev/null
@@ -1,7 +0,0 @@
-PASS htmlallcollection.toString() is '[object HTMLAllCollection]'
-PASS typeof htmlallcollection.tags == "function" is true
-PASS htmlallcollection.tags("body").length is 1
-PASS successfullyParsed is true
-
-TEST COMPLETE
-
diff --git a/LayoutTests/fast/dom/HTMLDocument/document-all.html b/LayoutTests/fast/dom/HTMLDocument/document-all.html
deleted file mode 100644 (file)
index 6a42997..0000000
+++ /dev/null
@@ -1,10 +0,0 @@
-<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML//EN">
-<html>
-<head>
-<script src="../../../resources/js-test-pre.js"></script>
-</head>
-<body>
-<script src="script-tests/document-all.js"></script>
-<script src="../../../resources/js-test-post.js"></script>
-</body>
-</html>
index 60e178a..85952fb 100644 (file)
@@ -1,9 +1,9 @@
-Test that HTMLAllCollection.tags() matches IE behavior with too few arguments.
+Test that HTMLAllCollection.tags() does not exist.
 
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
 
-PASS document.all.tags() threw exception TypeError: Not enough arguments.
+PASS document.all.tags is undefined.
 PASS successfullyParsed is true
 
 TEST COMPLETE
index 75ee124..d671f31 100644 (file)
@@ -5,8 +5,8 @@
 </head>
 <body>
 <script>
-description('Test that HTMLAllCollection.tags() matches IE behavior with too few arguments.');
-shouldThrow("document.all.tags()");
+description('Test that HTMLAllCollection.tags() does not exist.');
+shouldBeUndefined("document.all.tags");
 </script>
 <script src="../../resources/js-test-post.js"></script>
 </body>
index 892e52b..5c1ec91 100644 (file)
@@ -1,3 +1,14 @@
+2017-03-08  Chris Dumez  <cdumez@apple.com>
+
+        Drop support for non-standard document.all.tags()
+        https://bugs.webkit.org/show_bug.cgi?id=169337
+
+        Reviewed by Sam Weinig.
+
+        Rebaseline web platform test now that one more check is passing.
+
+        * web-platform-tests/html/infrastructure/common-dom-interfaces/collections/htmlallcollection-expected.txt:
+
 2017-03-08  Simon Fraser  <simon.fraser@apple.com>
 
         Add support for history.scrollRestoration
index 14cbba8..d80969f 100644 (file)
@@ -8,7 +8,7 @@ PASS Test lookup IMG by namedItem
 PASS Test lookup IMG in collection using () 
 PASS Test lookup IMG in collection using [] 
 PASS Test lookup IMG in collection using . 
-FAIL Test lookup tags in collection using . assert_equals: expected (string) "tags" but got (undefined) undefined
+PASS Test lookup tags in collection using . 
 PASS Should find root element too 
 FAIL Should find both anchors and produce a list undefined is not an object (evaluating 'document.all["foo"].length')
 
index ea55fb2..5a96b39 100644 (file)
@@ -1,3 +1,22 @@
+2017-03-08  Chris Dumez  <cdumez@apple.com>
+
+        Drop support for non-standard document.all.tags()
+        https://bugs.webkit.org/show_bug.cgi?id=169337
+
+        Reviewed by Sam Weinig.
+
+        Drop support for non-standard document.all.tags(). It is not part of the specification:
+        - https://html.spec.whatwg.org/multipage/infrastructure.html#the-htmlallcollection-interface
+
+        It is not supported by Firefox and its support was dropped from Chrome back in early 2014:
+        - https://src.chromium.org/viewvc/blink?view=revision&revision=166870
+
+        No new tests, updated existing test.
+
+        * html/HTMLAllCollection.cpp:
+        * html/HTMLAllCollection.h:
+        * html/HTMLAllCollection.idl:
+
 2017-03-08  Said Abou-Hallawa  <sabouhallawa@apple.com>
 
         Enable async image decoding for large images
index 13d8de7..eaef792 100644 (file)
@@ -59,12 +59,4 @@ Element* HTMLAllCollection::namedItemWithIndex(const AtomicString& name, unsigne
     return nullptr;
 }
 
-RefPtr<NodeList> HTMLAllCollection::tags(const String& name)
-{
-    if (name.isNull())
-        return nullptr;
-
-    return ownerNode().getElementsByTagName(name);
-}
-
 } // namespace WebCore
index 0924994..2539e91 100644 (file)
@@ -34,7 +34,6 @@ public:
     static Ref<HTMLAllCollection> create(Document&, CollectionType);
 
     Element* namedItemWithIndex(const AtomicString& name, unsigned index) const;
-    RefPtr<NodeList> tags(const String&);
 
 private:
     HTMLAllCollection(Document&, CollectionType);
index eb35047..67f4c93 100644 (file)
@@ -35,8 +35,5 @@
     [Custom] getter Element? item(unsigned long index);
     [Custom] getter (HTMLCollection or Element)? namedItem(DOMString name);
     // [Custom] legacycaller (HTMLCollection or Element)? item(optional DOMString nameOrIndex);
-
-    // FIXME: This is not in the standard. Can we remove it?
-    NodeList tags(DOMString name);
 };