Mus: Install SwapBuffersCompleted/VSync callbacks to DirectOutputSurface(Ozone)

Mus: Fix Capabilities and Install SwapBuffersCompleted/VSync callbacks

DirectOutputSurface(Ozone) should be informed when a swap buffers is
completed or when vsync parameters are updated so that it can update the
compositor accordingly. We need to make sure that the callback is called on the Mus
compositor thread. InProcessCommandBuffer calls
these callbacks on the "origin" thread in this patch.

Furthermore, InProcessContextProvider was faking Capabilities instead of grabbing
them from InProcessContextProvider. This doesn't work on
ozone platforms.

BUG=668136
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/58b5a2252f651bc852b7afe57377e4f0f5a8bb91
Cr-Commit-Position: refs/heads/master@{#434285}

[Fady Samuel, 2016-11-23 18:36:18]

Description was changed from

==========
Mus: Install SwapBuffersCompleted/VSync callbacks to DirectOutputSurface(Ozone)

DirectOutputSurface(Ozone) should be informed when a swap buffers is completed
or when vsync parameters are updated so that it can update the compositor
accordingly.

BUG=668136
==========

to

==========
Mus: Install SwapBuffersCompleted/VSync callbacks to DirectOutputSurface(Ozone)

DirectOutputSurface(Ozone) should be informed when a swap buffers is completed
or when vsync parameters are updated so that it can update the compositor
accordingly.

BUG=668136
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Fady Samuel, 2016-11-23 18:36:26]

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

[Fady Samuel, 2016-11-23 18:36:33]

fsamuel@chromium.org changed reviewers:
+ kylechar@chromium.org

[commit-bot: I haz the power, 2016-11-23 18:36:54]

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

[Fady Samuel, 2016-11-23 19:25:24]

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

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

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

[Fady Samuel, 2016-11-23 19:53:34]

Description was changed from

==========
Mus: Install SwapBuffersCompleted/VSync callbacks to DirectOutputSurface(Ozone)

DirectOutputSurface(Ozone) should be informed when a swap buffers is completed
or when vsync parameters are updated so that it can update the compositor
accordingly.

BUG=668136
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Mus: Fix Capabilities and Install SwapBuffersCompleted/VSync callbacks

DirectOutputSurface(Ozone) should be informed when a swap buffers is completed
or when vsync parameters are updated so that it can update the compositor
accordingly.

BUG=668136
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Fady Samuel, 2016-11-23 19:53:34]

Description was changed from

==========
Mus: Install SwapBuffersCompleted/VSync callbacks to DirectOutputSurface(Ozone)

DirectOutputSurface(Ozone) should be informed when a swap buffers is completed
or when vsync parameters are updated so that it can update the compositor
accordingly.

BUG=668136
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Mus: Fix Capabilities and Install SwapBuffersCompleted/VSync callbacks

DirectOutputSurface(Ozone) should be informed when a swap buffers is completed
or when vsync parameters are updated so that it can update the compositor
accordingly.

BUG=668136
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Fady Samuel, 2016-11-23 19:55:04]

Description was changed from

==========
Mus: Fix Capabilities and Install SwapBuffersCompleted/VSync callbacks

DirectOutputSurface(Ozone) should be informed when a swap buffers is completed
or when vsync parameters are updated so that it can update the compositor
accordingly.

BUG=668136
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Mus: Fix Capabilities and Install SwapBuffersCompleted/VSync callbacks

DirectOutputSurface(Ozone) should be informed when a swap buffers is 
completed or when vsync parameters are updated so that it can update the
compositor accordingly. We need to make sure that the callback is called on the
Mus compositor thread. InProcessCommandBuffer calls
these callbacks on the "origin" thread in this patch.

Furthermore, InProcessContextProvider was faking Capabilities instead of
grabbing them from InProcessContextProvider. This doesn't work on
ozone platforms.

BUG=668136
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Fady Samuel, 2016-11-23 20:03:40]

Description was changed from

==========
Mus: Fix Capabilities and Install SwapBuffersCompleted/VSync callbacks

DirectOutputSurface(Ozone) should be informed when a swap buffers is 
completed or when vsync parameters are updated so that it can update the
compositor accordingly. We need to make sure that the callback is called on the
Mus compositor thread. InProcessCommandBuffer calls
these callbacks on the "origin" thread in this patch.

Furthermore, InProcessContextProvider was faking Capabilities instead of
grabbing them from InProcessContextProvider. This doesn't work on
ozone platforms.

BUG=668136
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Mus: Fix Capabilities and Install SwapBuffersCompleted/VSync callbacks

DirectOutputSurface(Ozone) should be informed when a swap buffers is 
completed or when vsync parameters are updated so that it can update the 
compositor accordingly. We need to make sure that the callback is called on the
Mus 
compositor thread. InProcessCommandBuffer calls
these callbacks on the "origin" thread in this patch.

Furthermore, InProcessContextProvider was faking Capabilities instead of
grabbing 
them from InProcessContextProvider. This doesn't work on
ozone platforms.

BUG=668136
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[kylechar, 2016-11-23 20:36:52]

lgtm

[Fady Samuel, 2016-11-23 20:40:36]

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

[Fady Samuel, 2016-11-23 20:40:37]

+piman@ for gpu/ipc and cc. Thanks!

[jbauman, 2016-11-23 22:13:23]

jbauman@chromium.org changed reviewers:
+ jbauman@chromium.org

[jbauman, 2016-11-23 22:13:23]

lgtm

[piman, 2016-11-23 22:18:24]

lgtm

[commit-bot: I haz the power, 2016-11-23 22:54:07]

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

[commit-bot: I haz the power, 2016-11-23 22:54:08]

Dry run: This issue passed the CQ dry run.

[Fady Samuel, 2016-11-23 22:57:48]

The CQ bit was checked by fsamuel@chromium.org

[commit-bot: I haz the power, 2016-11-23 22:58:57]

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-23 23:50:30]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1479941868340460,
"parent_rev": "f91ae7b3dc1dea70776fb11724e2ee114c8160ab", "commit_rev":
"95058070a2670aa09c71bc831c22e73ae2c4d0b2"}

[commit-bot: I haz the power, 2016-11-23 23:50:54]

Description was changed from

==========
Mus: Fix Capabilities and Install SwapBuffersCompleted/VSync callbacks

DirectOutputSurface(Ozone) should be informed when a swap buffers is 
completed or when vsync parameters are updated so that it can update the 
compositor accordingly. We need to make sure that the callback is called on the
Mus 
compositor thread. InProcessCommandBuffer calls
these callbacks on the "origin" thread in this patch.

Furthermore, InProcessContextProvider was faking Capabilities instead of
grabbing 
them from InProcessContextProvider. This doesn't work on
ozone platforms.

BUG=668136
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Mus: Fix Capabilities and Install SwapBuffersCompleted/VSync callbacks

DirectOutputSurface(Ozone) should be informed when a swap buffers is 
completed or when vsync parameters are updated so that it can update the 
compositor accordingly. We need to make sure that the callback is called on the
Mus 
compositor thread. InProcessCommandBuffer calls
these callbacks on the "origin" thread in this patch.

Furthermore, InProcessContextProvider was faking Capabilities instead of
grabbing 
them from InProcessContextProvider. This doesn't work on
ozone platforms.

BUG=668136
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[commit-bot: I haz the power, 2016-11-23 23:50:55]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-23 23:52:54]

Description was changed from

==========
Mus: Fix Capabilities and Install SwapBuffersCompleted/VSync callbacks

DirectOutputSurface(Ozone) should be informed when a swap buffers is 
completed or when vsync parameters are updated so that it can update the 
compositor accordingly. We need to make sure that the callback is called on the
Mus 
compositor thread. InProcessCommandBuffer calls
these callbacks on the "origin" thread in this patch.

Furthermore, InProcessContextProvider was faking Capabilities instead of
grabbing 
them from InProcessContextProvider. This doesn't work on
ozone platforms.

BUG=668136
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Mus: Fix Capabilities and Install SwapBuffersCompleted/VSync callbacks

DirectOutputSurface(Ozone) should be informed when a swap buffers is
completed or when vsync parameters are updated so that it can update the
compositor accordingly. We need to make sure that the callback is called on the
Mus
compositor thread. InProcessCommandBuffer calls
these callbacks on the "origin" thread in this patch.

Furthermore, InProcessContextProvider was faking Capabilities instead of
grabbing
them from InProcessContextProvider. This doesn't work on
ozone platforms.

BUG=668136
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/58b5a2252f651bc852b7afe57377e4f0f5a8bb91
Cr-Commit-Position: refs/heads/master@{#434285}
==========

[commit-bot: I haz the power, 2016-11-23 23:52:56]

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