2011-04-28 Dominic Cooney <dominicc@chromium.org>
authordglazkov@chromium.org <dglazkov@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 28 Apr 2011 18:43:54 +0000 (18:43 +0000)
committerdglazkov@chromium.org <dglazkov@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 28 Apr 2011 18:43:54 +0000 (18:43 +0000)
        Reviewed by Geoffrey Garen.

        Walk shadow hosts as well as parent nodes when finding opaque roots.
        https://bugs.webkit.org/show_bug.cgi?id=59571

        Remove expectation of failure.

        * platform/mac/fast/dom/shadow/gc-shadow-expected.txt: Removed.
2011-04-28  Dominic Cooney  <dominicc@chromium.org>

        Reviewed by Geoffrey Garen.

        Walk shadow hosts as well as parent nodes when finding opaque roots.
        https://bugs.webkit.org/show_bug.cgi?id=59571

        Test: fast/dom/shadow/gc-shadow.html

        * bindings/js/JSDOMBinding.h:
        (WebCore::root):

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

LayoutTests/ChangeLog
LayoutTests/platform/mac/fast/dom/shadow/gc-shadow-expected.txt [deleted file]
Source/WebCore/ChangeLog
Source/WebCore/bindings/js/JSDOMBinding.h

index 8eafcd6..1c34703 100644 (file)
@@ -1,3 +1,14 @@
+2011-04-28  Dominic Cooney  <dominicc@chromium.org>
+
+        Reviewed by Geoffrey Garen.
+
+        Walk shadow hosts as well as parent nodes when finding opaque roots.
+        https://bugs.webkit.org/show_bug.cgi?id=59571
+
+        Remove expectation of failure.
+
+        * platform/mac/fast/dom/shadow/gc-shadow-expected.txt: Removed.
+
 2011-04-28  Dimitri Glazkov  <dglazkov@chromium.org>
 
         Added the test results, missed in r85209.
diff --git a/LayoutTests/platform/mac/fast/dom/shadow/gc-shadow-expected.txt b/LayoutTests/platform/mac/fast/dom/shadow/gc-shadow-expected.txt
deleted file mode 100644 (file)
index 87041c6..0000000
+++ /dev/null
@@ -1,17 +0,0 @@
-This tests that the JavaScript wrapper objects of shadow DOM objects
-are not prematurely garbage collected.
-
-FAIL layoutTestController.shadowRoot(ps[0]).tattoo should be I <3 WebKit (of type string). Was undefined (of type undefined).
-FAIL layoutTestController.shadowRoot(ps[1]).tattoo should be I <3 WebKit (of type string). Was undefined (of type undefined).
-FAIL layoutTestController.shadowRoot(ps[2]).tattoo should be I <3 WebKit (of type string). Was undefined (of type undefined).
-FAIL layoutTestController.shadowRoot(ps[3]).tattoo should be I <3 WebKit (of type string). Was undefined (of type undefined).
-FAIL layoutTestController.shadowRoot(ps[4]).tattoo should be I <3 WebKit (of type string). Was undefined (of type undefined).
-FAIL layoutTestController.shadowRoot(ps[5]).tattoo should be I <3 WebKit (of type string). Was undefined (of type undefined).
-FAIL layoutTestController.shadowRoot(ps[6]).tattoo should be I <3 WebKit (of type string). Was undefined (of type undefined).
-FAIL layoutTestController.shadowRoot(ps[7]).tattoo should be I <3 WebKit (of type string). Was undefined (of type undefined).
-FAIL layoutTestController.shadowRoot(ps[8]).tattoo should be I <3 WebKit (of type string). Was undefined (of type undefined).
-FAIL layoutTestController.shadowRoot(ps[9]).tattoo should be I <3 WebKit (of type string). Was undefined (of type undefined).
-PASS successfullyParsed is true
-
-TEST COMPLETE
-
index 26d334c..607b3de 100644 (file)
@@ -1,3 +1,15 @@
+2011-04-28  Dominic Cooney  <dominicc@chromium.org>
+
+        Reviewed by Geoffrey Garen.
+
+        Walk shadow hosts as well as parent nodes when finding opaque roots.
+        https://bugs.webkit.org/show_bug.cgi?id=59571
+
+        Test: fast/dom/shadow/gc-shadow.html
+
+        * bindings/js/JSDOMBinding.h:
+        (WebCore::root):
+
 2011-04-28  Martin Robinson  <mrobinson@igalia.com>
 
         Reviewed by Xan Lopez.
index 4ac4252..2837875 100644 (file)
@@ -171,8 +171,8 @@ namespace WebCore {
         if (node->inDocument())
             return node->document();
 
-        while (node->parentNode())
-            node = node->parentNode();
+        while (node->parentOrHostNode())
+            node = node->parentOrHostNode();
         return node;
     }