Reviewed by Kevin.
authormjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 23 Jul 2004 07:25:13 +0000 (07:25 +0000)
committermjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 23 Jul 2004 07:25:13 +0000 (07:25 +0000)
<rdar://problem/3725467> REGRESSION(140-142) nothing displayed at http://Ye-Olde-Movies.tripod.com
<rdar://problem/3599494> REGRESSION (100-125): Frame does not refresh until user clicks on browser.

        * khtml/rendering/render_frames.cpp:
        (RenderFrameSet::layout): When returning early because there are
no children, make sure to mark self as no longer needing
layout. Otherwise, when children are added, we'll fail to
propagate childrenNeedLayout to our own parent! This is almost
certainly a long-standing bug that was just masked by our old
layout tomfoolery.

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

WebCore/ChangeLog-2005-08-23
WebCore/khtml/rendering/render_frames.cpp

index 58220ec..6394973 100644 (file)
@@ -1,3 +1,18 @@
+2004-07-22  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Kevin.
+
+       <rdar://problem/3725467> REGRESSION(140-142) nothing displayed at http://Ye-Olde-Movies.tripod.com
+       <rdar://problem/3599494> REGRESSION (100-125): Frame does not refresh until user clicks on browser.
+       
+        * khtml/rendering/render_frames.cpp:
+        (RenderFrameSet::layout): When returning early because there are
+       no children, make sure to mark self as no longer needing
+       layout. Otherwise, when children are added, we'll fail to
+       propagate childrenNeedLayout to our own parent! This is almost
+       certainly a long-standing bug that was just masked by our old
+       layout tomfoolery.
+
 2004-07-22  Kevin Decker  <kdecker@apple.com>
 
        Reviewed by Darin
index 2a702b3..e523824 100644 (file)
@@ -218,7 +218,7 @@ void RenderFrameSet::layout( )
 
     RenderObject *child = firstChild();
     if ( !child )
-      return;
+       goto end2;
 
     if(!m_hSplitVar && !m_vSplitVar)
     {