Bug 84583

Summary: Incremental rendering should be unsuppressed after the load event or a configurable timeout, whichever occurs first.
Product: WebKit Reporter: Andy Estes <aestes>
Component: Layout and RenderingAssignee: Andy Estes <aestes>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, koivisto
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch andersca: review+

Andy Estes
Reported 2012-04-23 03:16:06 PDT
Incremental rendering should be unsuppressed after the load event or a configurable timeout, whichever occurs first.
Attachments
Patch (13.34 KB, patch)
2012-04-23 03:33 PDT, Andy Estes 		no flags
DetailsFormatted DiffDiff
Patch (13.33 KB, patch)
2012-04-23 12:31 PDT, Andy Estes 		andersca: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Andy Estes
Comment 1 2012-04-23 03:17:28 PDT
<rdar://problem/11292192>
Andy Estes
Comment 2 2012-04-23 03:33:02 PDT
Created attachment 138320 [details] Patch
Andy Estes
Comment 3 2012-04-23 12:16:26 PDT
Comment on attachment 138320 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=138320&action=review > Source/WebCore/dom/Document.cpp:1203 > +#if USE(ACCELENRATED_COMPOSITING) Oops. > Source/WebKit/mac/WebView/WebView.mm:1519 > + settings->setIncrementalRenderingSuppressionTimeoutInSeconds([preferences incrementalRenderingSuppressionTimeoutInSeconds]); This has the effect of disabling incremental rendering suppression unless the preference is actually set. What I actually want in this case is the default timeout.
Andy Estes
Comment 4 2012-04-23 12:31:57 PDT
Created attachment 138398 [details] Patch
Andy Estes
Comment 5 2012-04-24 00:43:51 PDT
Committed r115005: <http://trac.webkit.org/changeset/115005>
Note You need to log in before you can comment on or make changes to this bug.