Bug 136104

Summary: [TexMap] Only notify of a required flush on the first layer property change
Product: WebKit Reporter: Zan Dobersek <zan>
Component: New BugsAssignee: Zan Dobersek <zan>
Status: RESOLVED FIXED    
Severity: Normal CC: cmarcelo, commit-queue, kondapallykalyan, luiz, mrobinson, noam
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Zan Dobersek
Reported 2014-08-20 08:49:57 PDT
[TexMap] Only notify of a required flush on the first layer property change
Attachments
Patch (1.58 KB, patch)
2014-08-20 08:55 PDT, Zan Dobersek 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Zan Dobersek
Comment 1 2014-08-20 08:55:21 PDT
Created attachment 236871 [details] Patch
Martin Robinson
Comment 2 2014-08-20 09:00:58 PDT
What is the symptom necessitating this change?
Zan Dobersek
Comment 3 2014-08-20 09:51:00 PDT
(In reply to comment #2) > What is the symptom necessitating this change? It avoids unnecessary calls through RenderLayerBacking::notifyFlushRequired() down to LayerTreeHost::scheduleLayerFlush() where in the end the flush timer is already scheduled, so nothing is done.
Zan Dobersek
Comment 4 2014-08-23 00:11:26 PDT
Comment on attachment 236871 [details] Patch Clearing flags on attachment: 236871 Committed r172885: <http://trac.webkit.org/changeset/172885>
Zan Dobersek
Comment 5 2014-08-23 00:11:35 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.