mus: Get the gpu memory buffer from the window server.

mus: Get the gpu memory buffer from the window server.

Instead of allocating shared memory on the client side, allocate it
from the gpu host (i.e. the window server). The window server still
isn't complete: it currently always allocates shared memory. In
subsequent CLs, the window server will be updated to allocate native
gpu memory throug mus-gpu.

BUG=643746
Committed: https://crrev.com/4f2b76d28896df4fc12d768eb1d9a5c7a9602211
Cr-Commit-Position: refs/heads/master@{#430833}

[sadrul, 2016-11-06 20:04:37]

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

[commit-bot: I haz the power, 2016-11-06 20:04:42]

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

[sadrul, 2016-11-06 20:10:38]

Patchset #2 (id:20001) has been deleted

[sadrul, 2016-11-08 02:48:36]

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

[commit-bot: I haz the power, 2016-11-08 02:49:02]

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-11-08 04:01:59]

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

[commit-bot: I haz the power, 2016-11-08 04:02:00]

Dry run: This issue passed the CQ dry run.

[sadrul, 2016-11-08 04:42:24]

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

[commit-bot: I haz the power, 2016-11-08 04:42:43]

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

[sadrul, 2016-11-08 04:56:06]

sadrul@chromium.org changed reviewers:
+ reveman@chromium.org, tsepez@chromium.org

[sadrul, 2016-11-08 04:56:07]

tsepez@ for the mojom and struct-traits change.

reveman@ for the rest of the changes.

[commit-bot: I haz the power, 2016-11-08 05:49:51]

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

[commit-bot: I haz the power, 2016-11-08 05:49:52]

Dry run: This issue passed the CQ dry run.

[Tom Sepez, 2016-11-08 18:28:03]

lgtm

[reveman, 2016-11-08 19:46:46]

lgtm

https://codereview.chromium.org/2479043002/diff/60001/services/ui/ws/gpu_serv...
File services/ui/ws/gpu_service_proxy.cc (right):

https://codereview.chromium.org/2479043002/diff/60001/services/ui/ws/gpu_serv...
services/ui/ws/gpu_service_proxy.cc:151: const gpu::SyncToken& sync_token) {
fyi: this sync_token is only needed for macos and I'm hoping we can remove it
soon

[sadrul, 2016-11-08 20:18:47]

sadrul@chromium.org changed reviewers:
+ sky@chromium.org

[sadrul, 2016-11-08 20:18:48]

+sky@ for owner review.

/cc +piman@ FYI

[sky, 2016-11-08 23:51:10]

LGTM

[sadrul, 2016-11-09 02:21:42]

The CQ bit was checked by sadrul@chromium.org

[commit-bot: I haz the power, 2016-11-09 02:22:19]

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-09 02:30:14]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-11-09 02:33:43]

Description was changed from

==========
mus: Get the gpu memory buffer from the window server.

Instead of allocating shared memory on the client side, allocate it
from the gpu host (i.e. the window server). The window server still
isn't complete: it currently always allocates shared memory. In
subsequent CLs, the window server will be updated to allocate native
gpu memory throug mus-gpu.

BUG=643746
==========

to

==========
mus: Get the gpu memory buffer from the window server.

Instead of allocating shared memory on the client side, allocate it
from the gpu host (i.e. the window server). The window server still
isn't complete: it currently always allocates shared memory. In
subsequent CLs, the window server will be updated to allocate native
gpu memory throug mus-gpu.

BUG=643746

Committed: https://crrev.com/4f2b76d28896df4fc12d768eb1d9a5c7a9602211
Cr-Commit-Position: refs/heads/master@{#430833}
==========

[commit-bot: I haz the power, 2016-11-09 02:33:44]

Patchset 3 (id:??) landed as
https://crrev.com/4f2b76d28896df4fc12d768eb1d9a5c7a9602211
Cr-Commit-Position: refs/heads/master@{#430833}