Bug 127347

Summary: Delay initial layer flush during loading on all platforms
Product: WebKit Reporter: Antti Koivisto <koivisto>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, esprehn+autocc, glenn, kondapallykalyan, simon.fraser, thorton
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch kling: review+

Antti Koivisto
Reported 2014-01-21 07:57:21 PST
To reduce unnecessary repaints enable the same behaviour as iOS already has.
Attachments
patch (3.86 KB, patch)
2014-01-21 08:02 PST, Antti Koivisto 		kling: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Antti Koivisto
Comment 1 2014-01-21 08:02:04 PST
Created attachment 221743 [details] patch
Andreas Kling
Comment 2 2014-01-21 08:18:09 PST
Comment on attachment 221743 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=221743&action=review Well that looked easy. r=me > Source/WebCore/rendering/RenderLayerCompositor.cpp:253 > - , m_layerFlushThrottlingEnabled(false) > + , m_layerFlushThrottlingEnabled(page() && page()->progress().isMainLoadProgressing()) I wonder why page() returns a pointer, and not a reference.
Antti Koivisto
Comment 3 2014-01-21 08:40:08 PST
https://trac.webkit.org/r162443
Simon Fraser (smfr)
Comment 4 2014-01-21 10:53:04 PST
Does this mean we see a longer delay on back/forward before seeing the new page? Or is page cache navigation not affected?
Antti Koivisto
Comment 5 2014-01-21 14:04:24 PST
(In reply to comment #4) > Does this mean we see a longer delay on back/forward before seeing the new page? Or is page cache navigation not affected? Page cache navigation shouldn't be affected. Completed load always turn off throttling.
Note You need to log in before you can comment on or make changes to this bug.