Flush contexts after LayerTreeHostImpl cleanup

Flush contexts after LayerTreeHostImpl cleanup

Currently, we do not flush our GL contexts after cleaning up
LayerTreeHostImpl (LTHI) resources. This delays the time it takes for
these deletes to reach the GPU process, and can allow driver caches to
grow unnecessarily.

This CL does a ShallowFlushCHROMIUM after freeing LTHI resources, which
reduces cache growth on some platforms (observed on N9).

R=vmpstr
BUG=643251
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

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

[ericrk, 2016-09-20 15:37:33]

Description was changed from

==========
Flush contexts after LayerTreeHostImpl cleanup

Currently, we do not flush our GL contexts after cleaning up
LayerTreeHostImpl (LTHI) resources. This delays the time it takes for
these deletes to reach the GPU process, and can allow driver caches to
grow unnecessarily.

This CL does a ShallowFlushCHROMIUM after freeing LTHI resources, which
reduces cache growth on some platforms (observed on N9).

R=vmpstr
BUG=643251
==========

to

==========
Flush contexts after LayerTreeHostImpl cleanup

Currently, we do not flush our GL contexts after cleaning up
LayerTreeHostImpl (LTHI) resources. This delays the time it takes for
these deletes to reach the GPU process, and can allow driver caches to
grow unnecessarily.

This CL does a ShallowFlushCHROMIUM after freeing LTHI resources, which
reduces cache growth on some platforms (observed on N9).

R=vmpstr
BUG=643251
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[ericrk, 2016-09-20 15:38:00]

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

[commit-bot: I haz the power, 2016-09-20 15:38:14]

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

[ericrk, 2016-09-20 15:39:14]

Working around bad driver caching :/ - we used to (inadvertently) have the
behavior before I cleaned up visibility handling.

[commit-bot: I haz the power, 2016-09-20 16:48:16]

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

[commit-bot: I haz the power, 2016-09-20 16:48:17]

Dry run: This issue passed the CQ dry run.

[vmpstr, 2016-09-20 21:00:32]

lgtm

[ericrk, 2016-09-20 22:23:23]

The CQ bit was checked by ericrk@chromium.org

[commit-bot: I haz the power, 2016-09-20 22:24:16]

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-09-20 22:29:18]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-20 22:31:41]

Description was changed from

==========
Flush contexts after LayerTreeHostImpl cleanup

Currently, we do not flush our GL contexts after cleaning up
LayerTreeHostImpl (LTHI) resources. This delays the time it takes for
these deletes to reach the GPU process, and can allow driver caches to
grow unnecessarily.

This CL does a ShallowFlushCHROMIUM after freeing LTHI resources, which
reduces cache growth on some platforms (observed on N9).

R=vmpstr
BUG=643251
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
Flush contexts after LayerTreeHostImpl cleanup

Currently, we do not flush our GL contexts after cleaning up
LayerTreeHostImpl (LTHI) resources. This delays the time it takes for
these deletes to reach the GPU process, and can allow driver caches to
grow unnecessarily.

This CL does a ShallowFlushCHROMIUM after freeing LTHI resources, which
reduces cache growth on some platforms (observed on N9).

R=vmpstr
BUG=643251
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

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

[commit-bot: I haz the power, 2016-09-20 22:31:42]

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