Bug 16334 - REGRESSION (r28299): Homepage of any DotMac Web Gallery won't load completely
Summary: REGRESSION (r28299): Homepage of any DotMac Web Gallery won't load completely
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P1 Normal
Assignee: mitz
URL: http://gallery.mac.com/emily_parker#g...
Keywords: InRadar, NeedsReduction, Regression
Depends on:
Blocks:
 
Reported: 2007-12-06 16:13 PST by Vincent Gasseling
Modified: 2007-12-07 10:27 PST (History)
1 user (show)

See Also:


Attachments
Test case (1012 bytes, text/html)
2007-12-07 08:59 PST, mitz
no flags Details
Avoid re-entering scheduleRelayoutOfSubtree() (10.37 KB, patch)
2007-12-07 09:57 PST, mitz
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Vincent Gasseling 2007-12-06 16:13:40 PST
The homepage of any gallery.mac.com-site won't load completely, only a couple of placeholders do appear. If you click on a placeholder and then after the next page has loaded go back via 'Emily's Web Gallery' (left top) to the homepage it will load, but then after a reload again only a couple of placeholders will appear.
Comment 1 David Kilzer (:ddkilzer) 2007-12-06 20:52:18 PST
Confirmed with a local debug build of WebKit r28483 with Safari 3.0.4 (523.12) on Mac OS X 10.4.11 (8S165).

This is a regression from Safari 3.0.4 (523.12) with original WebKit on Tiger.

Comment 2 David Kilzer (:ddkilzer) 2007-12-06 21:38:27 PST
Searching internal builds, this regressed from r28298 to r28299.
http://trac.webkit.org/projects/webkit/changeset/28299

Comment 3 David Kilzer (:ddkilzer) 2007-12-06 21:39:03 PST
<rdar://problem/5634923>
Comment 4 mitz 2007-12-06 22:02:12 PST
Using Safari 3.0.4 (523.12) on Mac OS X 10.4.11 I get a page that tells me that my browser is incompatible with .Mac Web Gallery :-\
Comment 5 David Kilzer (:ddkilzer) 2007-12-07 04:32:27 PST
(In reply to comment #4)
> Using Safari 3.0.4 (523.12) on Mac OS X 10.4.11 I get a page that tells me that
> my browser is incompatible with .Mac Web Gallery :-\

I saw that intermittently when testing nightly revisions.  Hmmm...guess it happens all the time with Safari 3.0.4 on Tiger.  Works okay with Safari 3.0.4 on Leopard.

Comment 6 mitz 2007-12-07 07:11:24 PST
(In reply to comment #5)
> (In reply to comment #4)
> > Using Safari 3.0.4 (523.12) on Mac OS X 10.4.11 I get a page that tells me that
> > my browser is incompatible with .Mac Web Gallery :-\
> 
> I saw that intermittently when testing nightly revisions.  Hmmm...guess it
> happens all the time with Safari 3.0.4 on Tiger.  Works okay with Safari 3.0.4
> on Leopard.
> 

I got it intermittently on some .Mac galleries regardless of the browser I was using. I've found another gallery where I could reproduce the problem and I know what's happening - something I've convinced myself couldn't happen when I wrote the original patch :-)
Comment 7 mitz 2007-12-07 08:59:07 PST
Created attachment 17771 [details]
Test case

You should see a green square (without having to resize the window or otherwise force relayout after the page loads).
Comment 8 mitz 2007-12-07 09:57:40 PST
Created attachment 17772 [details]
Avoid re-entering scheduleRelayoutOfSubtree()

No layout test regressions.
Comment 9 Darin Adler 2007-12-07 10:17:40 PST
Comment on attachment 17772 [details]
Avoid re-entering scheduleRelayoutOfSubtree()

r=me

I am getting frustrated with the difficulty of understanding things like the second boolean to setChildNeedsLayout and the various constraints on what is and is not safe to call. We need to make some progress in clarity in the render tree code when viewed up close -- ideally we will get to the point where we can reason about the correctness of a fragment of code without having to constantly consider the big picture.
Comment 10 mitz 2007-12-07 10:27:28 PST
Landed in <http://trac.webkit.org/projects/webkit/changeset/28519>.