Summary: | REGRESSION (r20410): In debug builds, loading a certain page is suspended until redisplay is forced | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | mitz | ||||
Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | bdakin, ddkilzer, hyatt | ||||
Priority: | P1 | Keywords: | InRadar, Regression | ||||
Version: | 523.x (Safari 3) | ||||||
Hardware: | Mac | ||||||
OS: | OS X 10.4 | ||||||
URL: | http://digg.com/design | ||||||
Attachments: |
|
Description
mitz
2007-03-24 07:40:22 PDT
I didn't debug this through, but it seems like FrameView::needsLayout() would have been a more appropriate place to add the root->needsLayout() check in r20410. That would have matched more closely the behavior prior to r20324. FrameView::layoutPending() should only be about layout scheduling. 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. (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 on attachment 13797 [details]
Move the root->needsLayout() check to FrameView::needsLayout()
r=me
|