WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
127347
Delay initial layer flush during loading on all platforms
https://bugs.webkit.org/show_bug.cgi?id=127347
Summary
Delay initial layer flush during loading on all platforms
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+
Details
Formatted Diff
Diff
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.
Top of Page
Format For Printing
XML
Clone This Bug