Bug #: 5316
authoreseidel <eseidel@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 29 Nov 2005 09:55:00 +0000 (09:55 +0000)
committereseidel <eseidel@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 29 Nov 2005 09:55:00 +0000 (09:55 +0000)
Submitted by: darin
Reviewed by: eseidel
        Re-enabling hc_noderemovechild.xhtml and fixing bug (hang).
        http://bugzilla.opendarwin.org/show_bug.cgi?id=5316

        * dom/xhtml/level1/core/hc_noderemovechild-expected.txt: Added.
        * dom/xhtml/level1/core/hc_noderemovechild.js: hang workaround.
        * dom/xhtml/level1/core/hc_noderemovechild.xhtml: Added.
        * dom/xhtml/level1/core/hc_noderemovechild.xhtml-disabled: Removed.

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

LayoutTests/ChangeLog
LayoutTests/dom/xhtml/level1/core/hc_noderemovechild-expected.txt [new file with mode: 0644]
LayoutTests/dom/xhtml/level1/core/hc_noderemovechild.js
LayoutTests/dom/xhtml/level1/core/hc_noderemovechild.xhtml [moved from LayoutTests/dom/xhtml/level1/core/hc_noderemovechild.xhtml-disabled with 100% similarity]

index 6f1203a82607c30de607aa93629def076b40b9bb..4d10ae2e79a5176a34c380887f977461156cfdd8 100644 (file)
@@ -1,3 +1,15 @@
+2005-11-29  Darin Adler  <darin@apple.com>
+
+        Reviewed by eseidel.  Landed by eseidel.
+
+        Re-enabling hc_noderemovechild.xhtml and fixing bug (hang).
+        http://bugzilla.opendarwin.org/show_bug.cgi?id=5316
+
+        * dom/xhtml/level1/core/hc_noderemovechild-expected.txt: Added.
+        * dom/xhtml/level1/core/hc_noderemovechild.js: hang workaround.
+        * dom/xhtml/level1/core/hc_noderemovechild.xhtml: Added.
+        * dom/xhtml/level1/core/hc_noderemovechild.xhtml-disabled: Removed.
+
 2005-11-28  Alexey Proskuryakov  <ap@nypop.com>
 
         Reviewed by Darin. Commited by Maciej.
diff --git a/LayoutTests/dom/xhtml/level1/core/hc_noderemovechild-expected.txt b/LayoutTests/dom/xhtml/level1/core/hc_noderemovechild-expected.txt
new file mode 100644 (file)
index 0000000..398bffe
--- /dev/null
@@ -0,0 +1,2 @@
+Test   http://www.w3.org/2001/DOM-Test-Suite/level1/core/hc_noderemovechild
+Status Success
index a02001b75f531b5f3252364889c67913e100d3b7..adff19a6c94f6cbfa9836f0c728ef9f431db52cc 100644 (file)
@@ -107,6 +107,16 @@ function hc_noderemovechild() {
 
       childList = rootNode.childNodes;
 
+//
+//  WebKit modification: 9-October-2005
+//
+//  Added the next two lines of code to remove first employee as
+//  documented rather than removing the body element.
+//  Workaround while waiting for <http://www.w3.org/Bugs/Public/show_bug.cgi?id=2334>.
+//
+      rootNode = childList.item(1);
+      childList = rootNode.childNodes;
+
       childToRemove = childList.item(1);
       removedChild = rootNode.removeChild(childToRemove);
       parentNode = removedChild.parentNode;