cc: Make OutputSurface entirely live on one thread in tests.

cc: Make OutputSurface entirely live on one thread in tests.

OutputSurface is single-threaded but it ends up being created on the
main thread and used on compositor thread in cc unit tests and layout
tests, where it is directly attached to the renderer compositor.

Rework these tests so OutputSurface is only on a single thread to
reduce complexity of the class (will allow removal of BindToClient
entirely).

R=enne@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/7bbd659851f812efc44d37e36315e9524cdae44b
Cr-Commit-Position: refs/heads/master@{#427495}

[danakj, 2016-10-22 00:58:23]

Description was changed from

==========
cc: Make OutputSurface entirely live on one thread in tests.

OutputSurface is single-threaded but it ends up being created on the
main thread and used on compositor thread in cc unit tests and layout
tests, where it is directly attached to the renderer compositor.

Rework these tests so OutputSurface is only on a single thread to
reduce complexity of the class (will allow removal of BindToClient
entirely).

R=enne@chromium.org
BUG=606056
==========

to

==========
cc: Make OutputSurface entirely live on one thread in tests.

OutputSurface is single-threaded but it ends up being created on the
main thread and used on compositor thread in cc unit tests and layout
tests, where it is directly attached to the renderer compositor.

Rework these tests so OutputSurface is only on a single thread to
reduce complexity of the class (will allow removal of BindToClient
entirely).

R=enne@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[enne (OOO), 2016-10-22 01:11:00]

lgtm

[danakj, 2016-10-24 18:30:43]

danakj@chromium.org changed reviewers:
+ piman@chromium.org

[danakj, 2016-10-24 18:30:43]

+piman for content/

[piman, 2016-10-24 22:49:27]

lgtm

[danakj, 2016-10-24 23:37:05]

The CQ bit was checked by danakj@chromium.org

[danakj, 2016-10-24 23:37:51]

The CQ bit was unchecked by danakj@chromium.org

[danakj, 2016-10-25 00:50:47]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2016-10-25 00:51:11]

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-10-25 01:25:22]

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

[commit-bot: I haz the power, 2016-10-25 01:25:23]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[danakj, 2016-10-25 20:45:36]

The CQ bit was checked by danakj@chromium.org

[danakj, 2016-10-25 20:45:37]

The patchset sent to the CQ was uploaded after l-g-t-m from enne@chromium.org,
piman@chromium.org
Link to the patchset: https://codereview.chromium.org/2445553002/#ps20001
(title: "osonthread: fixcontextdeleteintest")

[commit-bot: I haz the power, 2016-10-25 20:46:21]

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-10-25 21:54:02]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-25 21:57:24]

Description was changed from

==========
cc: Make OutputSurface entirely live on one thread in tests.

OutputSurface is single-threaded but it ends up being created on the
main thread and used on compositor thread in cc unit tests and layout
tests, where it is directly attached to the renderer compositor.

Rework these tests so OutputSurface is only on a single thread to
reduce complexity of the class (will allow removal of BindToClient
entirely).

R=enne@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Make OutputSurface entirely live on one thread in tests.

OutputSurface is single-threaded but it ends up being created on the
main thread and used on compositor thread in cc unit tests and layout
tests, where it is directly attached to the renderer compositor.

Rework these tests so OutputSurface is only on a single thread to
reduce complexity of the class (will allow removal of BindToClient
entirely).

R=enne@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/7bbd659851f812efc44d37e36315e9524cdae44b
Cr-Commit-Position: refs/heads/master@{#427495}
==========

[commit-bot: I haz the power, 2016-10-25 21:57:25]

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