|Summary:||REGRESSION (r20410): In debug builds, loading a certain page is suspended until redisplay is forced|
|Component:||New Bugs||Assignee:||Nobody <webkit-unassigned>|
|Severity:||Normal||CC:||bdakin, ddkilzer, hyatt|
|Version:||523.x (Safari 3)|
|OS:||OS X 10.4|
Description mitz 2007-03-24 07:40:22 PDT
Steps to reproduce: 1) With a debug build of TOT WebKit and Safari 2.0.4, open the URL. Wait for it to finish loading. 2) Close the window. 3) Open a new window a load the same URL in it. While waiting for it to finish loading, do not move the window or Safari to the background. Expected result: The page should finish loading in step 3. Actual result: In step 3, the page does not finish loading and the window remains blank until you move the window or Safari to the background. This forces redisplay (to update active state), which makes the page resume loading and finally finish. Regression: Regressed in <http://trac.webkit.org/projects/webkit/changeset/20410>. Notes: Seems to affect only debug builds and only with Tiger Safari.
Comment 2 mitz 2007-03-24 08:44:08 PDT
Comment 3 mitz 2007-03-24 08:53:53 PDT
Created attachment 13797 [details] Move the root->needsLayout() check to FrameView::needsLayout() I tried the link from bug 13155 a few times to verify that this does not bring back the assertion failure.
Comment 4 David Kilzer (:ddkilzer) 2007-03-24 09:04:24 PDT
(In reply to comment #3) > I tried the link from bug 13155 a few times to verify that this does not bring > back the assertion failure. There is another way this assertion is hit that hasn't been fixed yet. (I filed a Radar for it, but didn't file a corresponding Bugzilla bug yet.) Here are the steps to reproduce. 1. Open Safari/WebKit. 2. Go to: http://ln-s.net/ 3. Click in the text field. 4. Hit Enter.
Comment 5 Dave Hyatt 2007-03-24 16:20:21 PDT
Comment on attachment 13797 [details] Move the root->needsLayout() check to FrameView::needsLayout() r=me