bgsound should use HTMLUnknownElement interface
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 21 Oct 2015 20:04:12 +0000 (20:04 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 21 Oct 2015 20:04:12 +0000 (20:04 +0000)
https://bugs.webkit.org/show_bug.cgi?id=148857
<rdar://problem/22589036>

Reviewed by Sam Weinig.

LayoutTests/imported/w3c:

Rebaseline a couple of W3C HTML tests now that more checks are passing.

* web-platform-tests/html/dom/interfaces-expected.txt:
* web-platform-tests/html/semantics/interfaces-expected.txt:

Source/WebCore:

The blink, bgsound, isindex, multicol, nextid, and spacer elements must
use the HTMLUnknownElement interface, as per the HTML specification:
https://html.spec.whatwg.org/multipage/obsolete.html#other-elements,-attributes-and-apis

WebKit was using HTMLUnknownElement for all of them except bgsound.
This patch fixes the issue and aligns our behavior with Chrome and
Firefox.

No new tests, already covered by existing tests.

* html/HTMLTagNames.in:

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

LayoutTests/imported/w3c/ChangeLog
LayoutTests/imported/w3c/web-platform-tests/html/dom/interfaces-expected.txt
LayoutTests/imported/w3c/web-platform-tests/html/semantics/interfaces-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/html/HTMLTagNames.in

index c3d539f..76bb9aa 100644 (file)
@@ -1,3 +1,16 @@
+2015-10-21  Chris Dumez  <cdumez@apple.com>
+
+        bgsound should use HTMLUnknownElement interface
+        https://bugs.webkit.org/show_bug.cgi?id=148857
+        <rdar://problem/22589036>
+
+        Reviewed by Sam Weinig.
+
+        Rebaseline a couple of W3C HTML tests now that more checks are passing.
+
+        * web-platform-tests/html/dom/interfaces-expected.txt:
+        * web-platform-tests/html/semantics/interfaces-expected.txt:
+
 2015-10-21  Youenn Fablet  <youenn.fablet@crf.canon.fr>
 
         W3C test importer should not import reference files as regular tests
index 2aca886..a6b7bd0 100644 (file)
@@ -148,10 +148,6 @@ CONSOLE MESSAGE: line 794: Deprecated attempt to access property 'nodeType' on a
 CONSOLE MESSAGE: line 2476: Deprecated attempt to access property 'nodeType' on a non-Node object.
 CONSOLE MESSAGE: line 794: Deprecated attempt to access property 'nodeType' on a non-Node object.
 CONSOLE MESSAGE: line 2476: Deprecated attempt to access property 'nodeType' on a non-Node object.
-CONSOLE MESSAGE: line 794: Deprecated attempt to access property 'nodeType' on a non-Node object.
-CONSOLE MESSAGE: line 2476: Deprecated attempt to access property 'nodeType' on a non-Node object.
-CONSOLE MESSAGE: line 794: Deprecated attempt to access property 'nodeType' on a non-Node object.
-CONSOLE MESSAGE: line 2476: Deprecated attempt to access property 'nodeType' on a non-Node object.
 CONSOLE MESSAGE: line 1152: Deprecated attempt to access property 'version' on a non-HTMLHtmlElement object.
 CONSOLE MESSAGE: line 1152: Deprecated attempt to access property 'text' on a non-HTMLTitleElement object.
 CONSOLE MESSAGE: line 1152: Deprecated attempt to access property 'href' on a non-HTMLBaseElement object.
@@ -2230,8 +2226,8 @@ PASS HTMLUnknownElement must be primary interface of document.createElement("bli
 PASS Stringification of document.createElement("blink") 
 PASS HTMLUnknownElement must be primary interface of document.createElement("quasit") 
 PASS Stringification of document.createElement("quasit") 
-FAIL HTMLUnknownElement must be primary interface of document.createElement("bgsound") assert_equals: document.createElement("bgsound")'s prototype is not HTMLUnknownElement.prototype expected Node object of unknown type but got Node object of unknown type
-FAIL Stringification of document.createElement("bgsound") assert_equals: class string of document.createElement("bgsound") expected "[object HTMLUnknownElement]" but got "[object HTMLElement]"
+PASS HTMLUnknownElement must be primary interface of document.createElement("bgsound") 
+PASS Stringification of document.createElement("bgsound") 
 PASS HTMLUnknownElement must be primary interface of document.createElement("isindex") 
 PASS Stringification of document.createElement("isindex") 
 PASS HTMLUnknownElement must be primary interface of document.createElement("multicol") 
index e6c396e..9a9b660 100644 (file)
@@ -17,8 +17,8 @@ FAIL Interfaces for basefont assert_equals: Element basefont should have HTMLUnk
 FAIL Interfaces for BASEFONT assert_equals: Element BASEFONT should have HTMLUnknownElement as its primary interface. expected "[object HTMLUnknownElement]" but got "[object HTMLBaseFontElement]"
 PASS Interfaces for bdo 
 PASS Interfaces for BDO 
-FAIL Interfaces for bgsound assert_equals: Element bgsound should have HTMLUnknownElement as its primary interface. expected "[object HTMLUnknownElement]" but got "[object HTMLElement]"
-FAIL Interfaces for BGSOUND assert_equals: Element BGSOUND should have HTMLUnknownElement as its primary interface. expected "[object HTMLUnknownElement]" but got "[object HTMLElement]"
+PASS Interfaces for bgsound 
+PASS Interfaces for BGSOUND 
 PASS Interfaces for big 
 PASS Interfaces for BIG 
 PASS Interfaces for blink 
index 7708335..2ae1b99 100644 (file)
@@ -1,3 +1,23 @@
+2015-10-21  Chris Dumez  <cdumez@apple.com>
+
+        bgsound should use HTMLUnknownElement interface
+        https://bugs.webkit.org/show_bug.cgi?id=148857
+        <rdar://problem/22589036>
+
+        Reviewed by Sam Weinig.
+
+        The blink, bgsound, isindex, multicol, nextid, and spacer elements must
+        use the HTMLUnknownElement interface, as per the HTML specification:
+        https://html.spec.whatwg.org/multipage/obsolete.html#other-elements,-attributes-and-apis
+
+        WebKit was using HTMLUnknownElement for all of them except bgsound.
+        This patch fixes the issue and aligns our behavior with Chrome and
+        Firefox.
+
+        No new tests, already covered by existing tests.
+
+        * html/HTMLTagNames.in:
+
 2015-10-21  Antoine Quint  <graouts@apple.com>
 
         Support for the SVG `onend` attribute
index 95c8343..2818c3a 100644 (file)
@@ -18,7 +18,7 @@ base
 basefont interfaceName=HTMLBaseFontElement
 bdi interfaceName=HTMLBDIElement, JSInterfaceName=HTMLElement
 bdo interfaceName=HTMLElement
-bgsound interfaceName=HTMLElement
+bgsound interfaceName=HTMLUnknownElement
 big interfaceName=HTMLElement
 blockquote interfaceName=HTMLQuoteElement
 body