Bug 136104 - [TexMap] Only notify of a required flush on the first layer property change
Summary: [TexMap] Only notify of a required flush on the first layer property change
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Zan Dobersek
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-08-20 08:49 PDT by Zan Dobersek
Modified: 2014-08-23 00:11 PDT (History)
6 users (show)

See Also:


Attachments
Patch (1.58 KB, patch)
2014-08-20 08:55 PDT, Zan Dobersek
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Zan Dobersek 2014-08-20 08:49:57 PDT
[TexMap] Only notify of a required flush on the first layer property change
Comment 1 Zan Dobersek 2014-08-20 08:55:21 PDT
Created attachment 236871 [details]
Patch
Comment 2 Martin Robinson 2014-08-20 09:00:58 PDT
What is the symptom necessitating this change?
Comment 3 Zan Dobersek 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.
Comment 4 Zan Dobersek 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>
Comment 5 Zan Dobersek 2014-08-23 00:11:35 PDT
All reviewed patches have been landed.  Closing bug.