Bug 11590

Summary:

REGRESSION (r17726-r17742): Wikipedia page intermittently loads but doesn't render

Product:

WebKit

Reporter:

mitz

Component:

CSS

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Major

CC:

dev+webkit, kevin, rwlbuis

Priority:

Keywords:

InRadar, Regression

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

URL:

http://en.wikipedia.org/wiki/Cliff_Stoll

Attachments:

Description	Flags
First attempt	none
Test case	none
Improved patch	none
Improved patch	none
Testcase now has commetns	beidson: review+

Description mitz 2006-11-13 10:23:27 PST

From time to time, when loading a Wikipedia article, the page finishes loading, but nothing is rendered, or only the page background is rendered. The DOM Inspector works (and even sometimes even highlights "invisible" nodes no the page). Reloading often fixes the problem.

The regression happened sometime between r17726 and r17742, probably in r17730, the fix for bug 10893 (InsertRule can not handle @import statements).

Comment 1 Matt Lilek 2006-11-13 10:27:47 PST

After the first couple page loads, this happens to me almost 100% of the time on Wikipedia and mysql.com

Comment 2 Rob Buis 2006-11-13 12:27:42 PST

Created attachment 11507 [details]
First attempt

Hi Matt,

Does this patch fix the problem?
Cheers,

Rob.

Comment 3 Rob Buis 2006-11-13 12:52:20 PST

Hi,

Note to any reviewers, I posted the same patch for 11591.
Thnx to Mitz for pointing that out.
Cheers,

Rob.

(In reply to comment #2)
> Created an attachment (id=11507) [edit]
> First attempt
> 
> Hi Matt,
> 
> Does this patch fix the problem?
> Cheers,
> 
> Rob.
>

Comment 4 Kevin M. Dean 2006-11-14 04:28:36 PST

I'm seeing the same/similar thing on http://www.maccentral.com/

Article pages don't fully display, but go to the URL and press return and then it displays fine.

Comment 5 mitz 2006-11-14 09:12:16 PST

Created attachment 11518 [details]
Test case

Comment 6 Rob Buis 2006-11-15 10:08:14 PST

Created attachment 11527 [details]
Improved patch

This patch should fix the regression, prevent the crash in 11591 and keep the functionality of bug 10893.
Cheers,

Rob.

Comment 7 Rob Buis 2006-11-15 12:01:33 PST

Created attachment 11530 [details]
Improved patch

D'oh I attached the wrong patch, this is the one I intended to attach.
Cheers,

Rob.

Comment 8 Rob Buis 2006-11-16 01:39:02 PST

Comment on attachment 11530 [details]
Improved patch

Should add comments first

Comment 9 Rob Buis 2006-11-16 04:45:08 PST

Created attachment 11536 [details]
Testcase now has commetns

As suggested by Mitz, some comments to go  with the testcase.
Cheers,

Rob.

Comment 10 Chris Petersen 2006-11-16 11:57:41 PST

This is covered in <rdar://problem/4841115>.

Comment 11 Brady Eidson 2006-11-16 18:14:30 PST

Comment on attachment 11536 [details]
Testcase now has commetns

Looks good, methinks

Comment 12 Brady Eidson 2006-11-16 18:22:04 PST

Committed in r17825

Next time, make sure you don't have any tabs  ;)