Encourage IOSurface reuse for CSS filter effects.

Encourage IOSurface reuse for CSS filter effects.

This comes at the expense of a temporary increase in memory usage. The resource
expiration delay was changed from 1s to 3s, so there's no long term increase in
memory usage, unless there is a corresponding performance gain.

BUG=636884
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/b8818116176eebbb37637a3b30cec284aa01d937
Cr-Commit-Position: refs/heads/master@{#413489}

[erikchen, 2016-08-17 20:01:31]

Description was changed from

==========
Encourage IOSurface reuse for CSS filter effects.

This comes at the expense of a temporary increase in memory usage. The resource
expiration delay was changed from 1s to 3s, so there's no long term increase in
memory usage, unless there is a corresponding performance gain.

BUG=636884
==========

to

==========
Encourage IOSurface reuse for CSS filter effects.

This comes at the expense of a temporary increase in memory usage. The resource
expiration delay was changed from 1s to 3s, so there's no long term increase in
memory usage, unless there is a corresponding performance gain.

BUG=636884
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[erikchen, 2016-08-17 20:02:09]

The CQ bit was checked by erikchen@chromium.org to run a CQ dry run

[erikchen, 2016-08-17 20:02:17]

erikchen@chromium.org changed reviewers:
+ ccameron@chromium.org

[erikchen, 2016-08-17 20:02:18]

ccameron: Please review.

[commit-bot: I haz the power, 2016-08-17 20:02:26]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-17 20:49:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-17 20:49:19]

Dry run: This issue passed the CQ dry run.

[erikchen, 2016-08-19 17:41:53]

erikchen@chromium.org changed reviewers:
+ ericrk@chromium.org

[erikchen, 2016-08-19 17:41:53]

ericrk: Please review.

ccameron is kind-of mostly OOO.

[ericrk, 2016-08-19 18:36:56]

lgtm

I wonder if we should just pass the delay in the constructor and keep runtime
changes to test-only? Up to you.

[erikchen, 2016-08-19 19:22:31]

The CQ bit was checked by erikchen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-19 19:23:01]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-19 20:22:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-19 20:22:07]

Dry run: This issue passed the CQ dry run.

[erikchen, 2016-08-22 17:34:29]

On 2016/08/19 18:36:56, ericrk wrote:
> lgtm
> 
> I wonder if we should just pass the delay in the constructor and keep runtime
> changes to test-only? Up to you.

Done.

[erikchen, 2016-08-22 17:34:37]

The CQ bit was checked by erikchen@chromium.org

[erikchen, 2016-08-22 17:34:37]

The patchset sent to the CQ was uploaded after l-g-t-m from ericrk@chromium.org
Link to the patchset: https://codereview.chromium.org/2254033002/#ps20001
(title: "Comments from ericrk.")

[commit-bot: I haz the power, 2016-08-22 17:35:26]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-22 18:51:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-22 18:53:24]

Description was changed from

==========
Encourage IOSurface reuse for CSS filter effects.

This comes at the expense of a temporary increase in memory usage. The resource
expiration delay was changed from 1s to 3s, so there's no long term increase in
memory usage, unless there is a corresponding performance gain.

BUG=636884
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
Encourage IOSurface reuse for CSS filter effects.

This comes at the expense of a temporary increase in memory usage. The resource
expiration delay was changed from 1s to 3s, so there's no long term increase in
memory usage, unless there is a corresponding performance gain.

BUG=636884
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/b8818116176eebbb37637a3b30cec284aa01d937
Cr-Commit-Position: refs/heads/master@{#413489}
==========

[commit-bot: I haz the power, 2016-08-22 18:53:25]

Patchset 2 (id:??) landed as
https://crrev.com/b8818116176eebbb37637a3b30cec284aa01d937
Cr-Commit-Position: refs/heads/master@{#413489}