Fix LayerTreeHostCacheBehaviorOnOutputSurfaceRecreated test

Fix LayerTreeHostCacheBehaviorOnOutputSurfaceRecreated test

LayerTreeHostCacheBehaviorOnOutputSurfaceRecreated test was incorrectly
relying on WillBeginImplFrameOnThread only being called once after the
OutputSurface was recreated. In reality, this function could be called
multiple times.

This patch causes us to handle cleanup in DidInitializeOutputSurface,
which will only be called once after recreating an OutputSurface.

BUG=642881, 642836
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/1e16f5f83b5a634369a7e9d82fddfe2e1ec10a9d
Cr-Commit-Position: refs/heads/master@{#415791}

[ericrk, 2016-08-31 21:48:30]

Description was changed from

==========
Fix LayerTreeHostCacheBehaviorOnOutputSurfaceRecreated test

LayerTreeHostCacheBehaviorOnOutputSurfaceRecreated test was incorrectly
relying on WillBeginImplFrameOnThread only being called once after the
OutputSurface was recreated. In reality, this function could be called
multiple times.

This patch causes us to handle cleanup in DidInitializeOutputSurface,
which will only be called once after recreating an OutputSurface.

BUG=642881,642836
==========

to

==========
Fix LayerTreeHostCacheBehaviorOnOutputSurfaceRecreated test

LayerTreeHostCacheBehaviorOnOutputSurfaceRecreated test was incorrectly
relying on WillBeginImplFrameOnThread only being called once after the
OutputSurface was recreated. In reality, this function could be called
multiple times.

This patch causes us to handle cleanup in DidInitializeOutputSurface,
which will only be called once after recreating an OutputSurface.

BUG=642881,642836
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[ericrk, 2016-08-31 21:51:00]

Patchset #1 (id:1) has been deleted

[ericrk, 2016-08-31 21:53:31]

Patchset #1 (id:20001) has been deleted

[ericrk, 2016-08-31 21:54:12]

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

[ericrk, 2016-08-31 21:54:13]

Can you take a look at this build bot fix?

[ericrk, 2016-08-31 21:57:04]

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

[commit-bot: I haz the power, 2016-08-31 21:57:54]

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-31 22:42:30]

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

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

Dry run: This issue passed the CQ dry run.

[vmpstr, 2016-08-31 22:51:08]

lgtm

[ericrk, 2016-08-31 22:51:31]

The CQ bit was checked by ericrk@chromium.org

[commit-bot: I haz the power, 2016-08-31 22:52:22]

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-31 22:56:25]

Committed patchset #1 (id:40001)

[commit-bot: I haz the power, 2016-08-31 22:57:39]

Description was changed from

==========
Fix LayerTreeHostCacheBehaviorOnOutputSurfaceRecreated test

LayerTreeHostCacheBehaviorOnOutputSurfaceRecreated test was incorrectly
relying on WillBeginImplFrameOnThread only being called once after the
OutputSurface was recreated. In reality, this function could be called
multiple times.

This patch causes us to handle cleanup in DidInitializeOutputSurface,
which will only be called once after recreating an OutputSurface.

BUG=642881,642836
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
Fix LayerTreeHostCacheBehaviorOnOutputSurfaceRecreated test

LayerTreeHostCacheBehaviorOnOutputSurfaceRecreated test was incorrectly
relying on WillBeginImplFrameOnThread only being called once after the
OutputSurface was recreated. In reality, this function could be called
multiple times.

This patch causes us to handle cleanup in DidInitializeOutputSurface,
which will only be called once after recreating an OutputSurface.

BUG=642881,642836
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/1e16f5f83b5a634369a7e9d82fddfe2e1ec10a9d
Cr-Commit-Position: refs/heads/master@{#415791}
==========

[commit-bot: I haz the power, 2016-08-31 22:57:40]

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