REGRESSION (r95926) Assert firing in svg/clip-path/clip-path-on-svg.svg
authormitz@apple.com <mitz@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 26 Sep 2011 20:08:11 +0000 (20:08 +0000)
committermitz@apple.com <mitz@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 26 Sep 2011 20:08:11 +0000 (20:08 +0000)
https://bugs.webkit.org/show_bug.cgi?id=68819

Reviewed by Dave Hyatt.

* rendering/RenderObject.cpp:
(WebCore::RenderObject::scheduleRelayout): Revert to using view() instead of the RenderView**
parameter of isRooted, since only the former returns 0 when the document is detached.

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

Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderObject.cpp

index 1597bb0eadd5a5e07c321d3eb1f5c7f118d64376..126d017e67acd5ac832a1da8d0dd14c90b698fd7 100644 (file)
@@ -1,3 +1,14 @@
+2011-09-26  Dan Bernstein  <mitz@apple.com>
+
+        REGRESSION (r95926) Assert firing in svg/clip-path/clip-path-on-svg.svg
+        https://bugs.webkit.org/show_bug.cgi?id=68819
+
+        Reviewed by Dave Hyatt.
+
+        * rendering/RenderObject.cpp:
+        (WebCore::RenderObject::scheduleRelayout): Revert to using view() instead of the RenderView**
+        parameter of isRooted, since only the former returns 0 when the document is detached.
+
 2011-09-26  Vsevolod Vlasov  <vsevik@chromium.org>
 
         Web Inspector: XMLHttpRequest console logging messages should link to network panel when possible.
index d88f3b874fa0b17b1919f94804880e1b9db43315..237da514316bcd49330799351c7aa5f17a6eee42 100644 (file)
@@ -2303,10 +2303,11 @@ void RenderObject::scheduleRelayout()
         if (view)
             view->scheduleRelayout();
     } else {
-        RenderView* renderView;
-        if (isRooted(&renderView)) {
-            if (FrameView* frameView = renderView->frameView())
-                frameView->scheduleRelayoutOfSubtree(this);
+        if (isRooted()) {
+            if (RenderView* renderView = view()) {
+                if (FrameView* frameView = renderView->frameView())
+                    frameView->scheduleRelayoutOfSubtree(this);
+            }
         }
     }
 }