mus/gpu: Rename the client/server side memory buffer managers.

mus/gpu: Rename the client/server side memory buffer managers.

Rename MojoGpuMemoryBufferManager to ClientGpuMemoryBufferManager.
Rename MusGpuMemoryBufferManager to ServerGpuMemoryBufferManager.

BUG=none

Committed: https://crrev.com/301ad2f8d2533080ab401d6685d7d9ec5f2ba2a8
Cr-Commit-Position: refs/heads/master@{#437438}

[sadrul, 2016-12-08 17:49:55]

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

[commit-bot: I haz the power, 2016-12-08 17:50:14]

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

[sadrul, 2016-12-08 17:50:52]

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

[sadrul, 2016-12-08 17:50:52]

(As discussed in an earlier CL)

[Fady Samuel, 2016-12-08 18:05:55]

lgtm

[commit-bot: I haz the power, 2016-12-08 19:44:39]

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

[commit-bot: I haz the power, 2016-12-08 19:44:40]

Dry run: 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_...)

[sky, 2016-12-08 20:11:05]

https://codereview.chromium.org/2562623005/diff/1/services/ui/common/server_g...
File services/ui/common/server_gpu_memory_buffer_manager.h (right):

https://codereview.chromium.org/2562623005/diff/1/services/ui/common/server_g...
services/ui/common/server_gpu_memory_buffer_manager.h:21: class
ServerGpuMemoryBufferManager : public gpu::GpuMemoryBufferManager,
Does this need to be in services/ui/common? Is it only used by the
window-server?

[sadrul, 2016-12-08 20:28:34]

https://codereview.chromium.org/2562623005/diff/1/services/ui/common/server_g...
File services/ui/common/server_gpu_memory_buffer_manager.h (right):

https://codereview.chromium.org/2562623005/diff/1/services/ui/common/server_g...
services/ui/common/server_gpu_memory_buffer_manager.h:21: class
ServerGpuMemoryBufferManager : public gpu::GpuMemoryBufferManager,
On 2016/12/08 20:11:04, sky wrote:
> Does this need to be in services/ui/common? Is it only used by the
> window-server?

It is also used by the display-compositor, which lives in the gpu process.

[sky, 2016-12-08 20:33:48]

I tend to think shared code for the servers should live else where, but we can
deal later. LGTM

[sadrul, 2016-12-08 20:42:00]

The CQ bit was checked by sadrul@chromium.org

[commit-bot: I haz the power, 2016-12-08 20:43:00]

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-12-08 21:50:09]

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

[commit-bot: I haz the power, 2016-12-08 21:50:10]

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_...)

[sadrul, 2016-12-08 22:33:53]

The CQ bit was checked by sadrul@chromium.org

[commit-bot: I haz the power, 2016-12-08 22:34:29]

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-12-08 22:51:12]

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

[commit-bot: I haz the power, 2016-12-08 22:51:13]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[sadrul, 2016-12-09 00:44:10]

The CQ bit was checked by sadrul@chromium.org

[sadrul, 2016-12-09 00:44:10]

The patchset sent to the CQ was uploaded after l-g-t-m from
fsamuel@chromium.org, sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2562623005/#ps20001
(title: "tot merge")

[commit-bot: I haz the power, 2016-12-09 00:44:52]

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-12-09 02:25:32]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481244250113410,
"parent_rev": "18e1980681be21176c5e50a4b0b7c0c478a5293b", "commit_rev":
"831114fe397081e24b55b6995a9df5790ed27d90"}

[commit-bot: I haz the power, 2016-12-09 02:25:57]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-09 02:28:15]

Description was changed from

==========
mus/gpu: Rename the client/server side memory buffer managers.

Rename MojoGpuMemoryBufferManager to ClientGpuMemoryBufferManager.
Rename MusGpuMemoryBufferManager to ServerGpuMemoryBufferManager.

BUG=none
==========

to

==========
mus/gpu: Rename the client/server side memory buffer managers.

Rename MojoGpuMemoryBufferManager to ClientGpuMemoryBufferManager.
Rename MusGpuMemoryBufferManager to ServerGpuMemoryBufferManager.

BUG=none

Committed: https://crrev.com/301ad2f8d2533080ab401d6685d7d9ec5f2ba2a8
Cr-Commit-Position: refs/heads/master@{#437438}
==========

[commit-bot: I haz the power, 2016-12-09 02:28:16]

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