Bug 136104

Summary:

[TexMap] Only notify of a required flush on the first layer property change

Product:

WebKit

Reporter:

Zan Dobersek <zan>

Component:

New Bugs

Assignee:

Zan Dobersek <zan>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

cmarcelo, commit-queue, kondapallykalyan, luiz, mrobinson, noam

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none

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.