Bug 76590

Summary: REGRESSION (r104060): Page contents not painted if inserting a new stylesheet and temporary body node
Product: WebKit Reporter: Mihai Parparita <mihaip>
Component: CSSAssignee: Antti Koivisto <koivisto>
Status: RESOLVED FIXED    
Severity: Normal CC: hyatt, kling, sroussey
Priority: P2 Keywords: HasReduction, Regression
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
URL: http://persistent.info/webkit/test-cases/crbug-109888.html
Attachments:
Description Flags
patch mjs: review+

Mihai Parparita
Reported 2012-01-18 17:10:35 PST
This was originally filed as http://crbug.com/109888, see http://persistent.info/webkit/test-cases/crbug-109888.html for a reduction (the text "The body contents should be visible" should appear on a green background, instead the entire page is white). The appending of the stylesheet dynamically is something that the page did, the inserting of the (temporary) body tag is from jQuery pre-1.6.2 which was included in the page (originally via a Chrome extension, though that's not relevant, since it happens when done directly too).
Attachments
patch (9.60 KB, patch)
2012-03-02 02:07 PST, Antti Koivisto
mjs: review+
Antti Koivisto
Comment 1 2012-01-19 04:41:38 PST
Delightfully obscure.
Mihai Parparita
Comment 2 2012-02-04 18:30:13 PST
Another regression from this change is http://stationstreetpgh.com/ (it should have pie charts that animate in, but they don't appear at all in WebKit nightlies post-r104060) So far, it seems to boil down to the "active" callback in Typekit's font loading code not firing (I've asked someone from their side if they can debug it). Here's a somewhat reduced test case: http://persistent.info/chromium/test-cases/crbug-112714/page.html It should display "loading" and then "active". In builds with the bug, it just displays "loading", and then eventually "inactive" (presumably a failure timeout firing).
Antti Koivisto
Comment 3 2012-03-02 02:07:57 PST
Maciej Stachowiak
Comment 4 2012-03-02 02:37:10 PST
Comment on attachment 129854 [details] patch r=,e
Antti Koivisto
Comment 5 2012-03-02 02:47:29 PST
trac.webkit.org/changeset/109543
Note You need to log in before you can comment on or make changes to this bug.