Handle compositor/worker context the same wrt. visibility

Handle compositor/worker context the same wrt. visibility

When transitioning the worker context to not-visible mode and cleaning
up resources, we wait for pending tile tasks to complete, as they may
be using resources. Given that the resources used by tile tasks are
created/destroyed on the compositor context (not the worker context), we
should also delay compositor context's cleanup until all tile tasks are
completed.

This change modifies LayerTreeHostImpl so that both contexts have their
visibility handled in the same way.

Additionally, it fixes an issue where we were checking resource pool
for pending cleanup in the wrong way (looking only at in-use resources,
not at all resources).

R=enne
BUG=663179
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Committed: https://crrev.com/174ef9083488b3d8528e9fa60820d0cec3badd50
Cr-Commit-Position: refs/heads/master@{#432282}

[ericrk, 2016-11-15 01:21:41]

Description was changed from

==========
Handle compositor/worker context the same wrt. visibility

When transitioning the worker context to not-visible mode and cleaning
up resources, we wait for pending tile tasks to complete, as they may
be using resources. Given that the resources used by tile tasks are
created/destroyed on the compositor context (not the worker context), we
should also delay compositor context's cleanup until all tile tasks are
completed.

This change modifies LayerTreeHostImpl so that both contexts have their
visibility handled in the same way.

Additionally, it fixes an issue where we were checking resource pool
for pending cleanup in the wrong way (looking only at in-use resources,
not at all resources).

R=enne
BUG=663179
==========

to

==========
Handle compositor/worker context the same wrt. visibility

When transitioning the worker context to not-visible mode and cleaning
up resources, we wait for pending tile tasks to complete, as they may
be using resources. Given that the resources used by tile tasks are
created/destroyed on the compositor context (not the worker context), we
should also delay compositor context's cleanup until all tile tasks are
completed.

This change modifies LayerTreeHostImpl so that both contexts have their
visibility handled in the same way.

Additionally, it fixes an issue where we were checking resource pool
for pending cleanup in the wrong way (looking only at in-use resources,
not at all resources).

R=enne
BUG=663179
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[ericrk, 2016-11-15 17:35:36]

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

[commit-bot: I haz the power, 2016-11-15 17:36:18]

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

[enne (OOO), 2016-11-15 19:10:34]

lgtm

[commit-bot: I haz the power, 2016-11-15 19:38:00]

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

[commit-bot: I haz the power, 2016-11-15 19:38:01]

Dry run: Try jobs failed on following builders:
  linux_trusty_blink_rel on master.tryserver.blink (JOB_TIMED_OUT, no build URL)

[ericrk, 2016-11-15 22:53:31]

The CQ bit was checked by ericrk@chromium.org

[commit-bot: I haz the power, 2016-11-15 22:54:03]

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-11-15 23:11:20]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-15 23:14:09]

Description was changed from

==========
Handle compositor/worker context the same wrt. visibility

When transitioning the worker context to not-visible mode and cleaning
up resources, we wait for pending tile tasks to complete, as they may
be using resources. Given that the resources used by tile tasks are
created/destroyed on the compositor context (not the worker context), we
should also delay compositor context's cleanup until all tile tasks are
completed.

This change modifies LayerTreeHostImpl so that both contexts have their
visibility handled in the same way.

Additionally, it fixes an issue where we were checking resource pool
for pending cleanup in the wrong way (looking only at in-use resources,
not at all resources).

R=enne
BUG=663179
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Handle compositor/worker context the same wrt. visibility

When transitioning the worker context to not-visible mode and cleaning
up resources, we wait for pending tile tasks to complete, as they may
be using resources. Given that the resources used by tile tasks are
created/destroyed on the compositor context (not the worker context), we
should also delay compositor context's cleanup until all tile tasks are
completed.

This change modifies LayerTreeHostImpl so that both contexts have their
visibility handled in the same way.

Additionally, it fixes an issue where we were checking resource pool
for pending cleanup in the wrong way (looking only at in-use resources,
not at all resources).

R=enne
BUG=663179
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Committed: https://crrev.com/174ef9083488b3d8528e9fa60820d0cec3badd50
Cr-Commit-Position: refs/heads/master@{#432282}
==========

[commit-bot: I haz the power, 2016-11-15 23:14:09]

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