Bug 6142

Summary: intermittent failures in some paste tests
Product: WebKit Reporter: Darin Adler <darin>
Component: HTML EditingAssignee: Darin Adler <darin>
Status: RESOLVED FIXED    
Severity: Normal CC: justin.garcia
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
use the more powerful flavor of updateLayout in cases that warrant it justin.garcia: review+

Darin Adler
Reported 2005-12-18 14:41:37 PST
I tracked down some intermittent failures in some paste-related tests to the fact that the tests are running early, when all style is "display: none" as part of our "FOUC-prevention". There's a simple fix.
Attachments
use the more powerful flavor of updateLayout in cases that warrant it (20.68 KB, patch)
2005-12-18 14:54 PST, Darin Adler 		justin.garcia: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Darin Adler
Comment 1 2005-12-18 14:54:57 PST
Created attachment 5143 [details] use the more powerful flavor of updateLayout in cases that warrant it
Justin Garcia
Comment 2 2005-12-21 01:04:53 PST
So the vanilla updateLayout() will use display:none for an element whose style depends on a pending stylesheet?
Darin Adler
Comment 3 2005-12-21 10:07:06 PST
Yes. See the code at the start of CSSStyleSelector::styleForElement.
Justin Garcia
Comment 4 2005-12-21 14:22:00 PST
Comment on attachment 5143 [details] use the more powerful flavor of updateLayout in cases that warrant it r=me
Note You need to log in before you can comment on or make changes to this bug.