Fix for 3726471, need to ensure that it's safe to do updateRendering from within...
authorhyatt <hyatt@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 16 Jul 2004 22:20:35 +0000 (22:20 +0000)
committerhyatt <hyatt@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 16 Jul 2004 22:20:35 +0000 (22:20 +0000)
relies on it.  The fix is to just recalcStyle up front before beginning the layout, so that all of our renderobjects
will be up-to-date.

        Reviewed by john

        * khtml/khtmlview.cpp:
        (KHTMLView::layout):

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

WebCore/ChangeLog-2005-08-23
WebCore/khtml/khtmlview.cpp

index b6eac38..3b9cda3 100644 (file)
@@ -1,3 +1,14 @@
+2004-07-16  David Hyatt  <hyatt@apple.com>
+
+       Fix for 3726471, need to ensure that it's safe to do updateRendering from within layout, since isContentEditable
+       relies on it.  The fix is to just recalcStyle up front before beginning the layout, so that all of our renderobjects
+       will be up-to-date.
+       
+        Reviewed by john
+
+        * khtml/khtmlview.cpp:
+        (KHTMLView::layout):
+
 2004-07-16  Ken Kocienda  <kocienda@apple.com>
 
         Reviewed by Hyatt
index 6c3fccb..8120d40 100644 (file)
@@ -579,6 +579,11 @@ void KHTMLView::layout()
         return;
     }
 
+    // Always ensure our style info is up-to-date.  This can happen in situations where
+    // the layout beats any sort of style recalc update that needs to occur.
+    if (document->hasChangedChild())
+        document->recalcStyle();
+
     khtml::RenderCanvas* root = static_cast<khtml::RenderCanvas*>(document->renderer());
     if (!root) {
         // FIXME: Do we need to set _width or _height here?