Document.adoptNode() should be able to explicitly adopt a DocumentType node
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 14 Sep 2015 05:06:40 +0000 (05:06 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 14 Sep 2015 05:06:40 +0000 (05:06 +0000)
https://bugs.webkit.org/show_bug.cgi?id=149097
LayoutTests/imported/w3c:

<rdar://problem/22549345>

Reviewed by Ryosuke Niwa.

Rebaseline W3C test now that a new check is passing.

* web-platform-tests/dom/nodes/Document-adoptNode-expected.txt:

Source/WebCore:

<rdar://problem/22549345>

Reviewed by Ryosuke Niwa.

Document.adoptNode() should be able to explicitly adopt a DocumentType
node as per the latest DOM specification:
https://dom.spec.whatwg.org/#dom-document-adoptnode

Chrome and Firefox match the specidicaiton but WebKit was throwing a
NotSupportedError.

No new tests, already covered by existing test.

* dom/Document.cpp:
(WebCore::Document::adoptNode): Deleted.

LayoutTests:

Reviewed by Ryosuke Niwa.

Rebaseline DOM3 tests, those seem to be outdated now.

* dom/xhtml/level3/core/documentadoptnode10-expected.txt:
* dom/xhtml/level3/core/documentadoptnode11-expected.txt:
* dom/xhtml/level3/core/documentadoptnode12-expected.txt:

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

LayoutTests/ChangeLog
LayoutTests/dom/xhtml/level3/core/documentadoptnode10-expected.txt
LayoutTests/dom/xhtml/level3/core/documentadoptnode11-expected.txt
LayoutTests/dom/xhtml/level3/core/documentadoptnode12-expected.txt
LayoutTests/imported/w3c/ChangeLog
LayoutTests/imported/w3c/web-platform-tests/dom/nodes/Document-adoptNode-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/dom/Document.cpp

index 8ae5af5..da49204 100644 (file)
@@ -1,5 +1,18 @@
 2015-09-13  Chris Dumez  <cdumez@apple.com>
 
+        Document.adoptNode() should be able to explicitly adopt a DocumentType node
+        https://bugs.webkit.org/show_bug.cgi?id=149097
+
+        Reviewed by Ryosuke Niwa.
+
+        Rebaseline DOM3 tests, those seem to be outdated now.
+
+        * dom/xhtml/level3/core/documentadoptnode10-expected.txt:
+        * dom/xhtml/level3/core/documentadoptnode11-expected.txt:
+        * dom/xhtml/level3/core/documentadoptnode12-expected.txt:
+
+2015-09-13  Chris Dumez  <cdumez@apple.com>
+
         document.lastModified should use the user's local time zone
         https://bugs.webkit.org/show_bug.cgi?id=149092
 
index 13e406d..d84d38c 100644 (file)
@@ -1,2 +1,3 @@
 Test   http://www.w3.org/2001/DOM-Test-Suite/level3/core/documentadoptnode10
-Status Success
+Status failure
+Message        throw_NOT_SUPPORTED_ERR: assertTrue failed
index 309ad05..be2991d 100644 (file)
@@ -1,2 +1,3 @@
 Test   http://www.w3.org/2001/DOM-Test-Suite/level3/core/documentadoptnode11
-Status Success
+Status failure
+Message        throw_NOT_SUPPORTED_ERR: assertTrue failed
index 722892d..4453580 100644 (file)
@@ -1,2 +1,3 @@
 Test   http://www.w3.org/2001/DOM-Test-Suite/level3/core/documentadoptnode12
-Status Success
+Status failure
+Message        throw_NOT_SUPPORTED_ERR: assertTrue failed
index 78e9bf9..0c4110f 100644 (file)
@@ -1,5 +1,17 @@
 2015-09-13  Chris Dumez  <cdumez@apple.com>
 
+        Document.adoptNode() should be able to explicitly adopt a DocumentType node
+        https://bugs.webkit.org/show_bug.cgi?id=149097
+        <rdar://problem/22549345>
+
+        Reviewed by Ryosuke Niwa.
+
+        Rebaseline W3C test now that a new check is passing. 
+
+        * web-platform-tests/dom/nodes/Document-adoptNode-expected.txt:
+
+2015-09-13  Chris Dumez  <cdumez@apple.com>
+
         Document.title does not behave according to specification
         https://bugs.webkit.org/show_bug.cgi?id=149098
 
index aca1bf8..a7db3e4 100644 (file)
@@ -1,6 +1,6 @@
 
 PASS Adopting an Element called 'x<' should work. 
 PASS Adopting an Element called ':good:times:' should work. 
-FAIL Explicitly adopting a DocumentType should work. NotSupportedError: DOM Exception 9
+PASS Explicitly adopting a DocumentType should work. 
 PASS Adopting a Document should throw. 
 
index 12aae10..65b62ae 100644 (file)
@@ -1,5 +1,25 @@
 2015-09-13  Chris Dumez  <cdumez@apple.com>
 
+        Document.adoptNode() should be able to explicitly adopt a DocumentType node
+        https://bugs.webkit.org/show_bug.cgi?id=149097
+        <rdar://problem/22549345>
+
+        Reviewed by Ryosuke Niwa.
+
+        Document.adoptNode() should be able to explicitly adopt a DocumentType
+        node as per the latest DOM specification:
+        https://dom.spec.whatwg.org/#dom-document-adoptnode
+
+        Chrome and Firefox match the specidicaiton but WebKit was throwing a
+        NotSupportedError.
+
+        No new tests, already covered by existing test.
+
+        * dom/Document.cpp:
+        (WebCore::Document::adoptNode): Deleted.
+
+2015-09-13  Chris Dumez  <cdumez@apple.com>
+
         Document.title does not behave according to specification
         https://bugs.webkit.org/show_bug.cgi?id=149098
 
index ddc981f..cb43837 100644 (file)
@@ -986,7 +986,6 @@ RefPtr<Node> Document::adoptNode(PassRefPtr<Node> source, ExceptionCode& ec)
     switch (source->nodeType()) {
     case ENTITY_NODE:
     case DOCUMENT_NODE:
-    case DOCUMENT_TYPE_NODE:
     case XPATH_NAMESPACE_NODE:
         ec = NOT_SUPPORTED_ERR;
         return nullptr;