Bug 6142

Summary:

intermittent failures in some paste tests

Product:

WebKit

Reporter:

Darin Adler <darin>

Component:

HTML Editing

Assignee:

Darin Adler <darin>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

justin.garcia

Priority:

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+

Description Darin Adler 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.

Comment 1 Darin Adler 2005-12-18 14:54:57 PST

Created attachment 5143 [details]
use the more powerful flavor of updateLayout in cases that warrant it

Comment 2 Justin Garcia 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?

Comment 3 Darin Adler 2005-12-21 10:07:06 PST

Yes. See the code at the start of CSSStyleSelector::styleForElement.

Comment 4 Justin Garcia 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