viz: Move ServerGpuMemoryBufferManager into host/gpu.

viz: Move ServerGpuMemoryBufferManager into host/gpu.

ServerGpuMemoryBufferManager used to live in viz/common, because it was used
by both the gpu host, and the display compositor service. Now that the latter
uses InProcessGpuMemoryBufferManager, ServerGpuMemoryBufferManager can move
into its rightful place.

BUG=733482
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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

Review-Url: https://codereview.chromium.org/2941423002
Cr-Commit-Position: refs/heads/master@{#482669}
Committed: https://chromium.googlesource.com/chromium/src/+/20c537c4f7982d52ba84cbbef7de7f48380dbd17

[sadrul, 2017-06-17 03:33:53]

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

[commit-bot: I haz the power, 2017-06-17 03:34: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, 2017-06-17 05:57:02]

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

[commit-bot: I haz the power, 2017-06-17 05:57:03]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[sadrul, 2017-06-18 05:19:01]

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

[commit-bot: I haz the power, 2017-06-18 05:19:11]

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, 2017-06-18 07:32:44]

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

[commit-bot: I haz the power, 2017-06-18 07:32:44]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[sadrul, 2017-06-19 20:10:37]

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

[commit-bot: I haz the power, 2017-06-19 20:10:59]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sadrul, 2017-06-19 20:23:31]

Description was changed from

==========
viz: Move ServerGpuMemoryBufferManager back into host/gpu.

ServerGpuMemoryBufferManager used to live in viz/common, because it was used
by both the gpu host, and the display compositor service. Now that the latter
uses InProcessGpuMemoryBufferManager, ServerGpuMemoryBufferManager can move
into its rightful place.

BUG=733482
==========

to

==========
viz: Move ServerGpuMemoryBufferManager into host/gpu.

ServerGpuMemoryBufferManager used to live in viz/common, because it was used
by both the gpu host, and the display compositor service. Now that the latter
uses InProcessGpuMemoryBufferManager, ServerGpuMemoryBufferManager can move
into its rightful place.

BUG=733482
==========

[commit-bot: I haz the power, 2017-06-19 22:19:41]

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

[commit-bot: I haz the power, 2017-06-19 22:19:41]

Dry run: This issue passed the CQ dry run.

[sadrul, 2017-06-21 18:29:07]

Description was changed from

==========
viz: Move ServerGpuMemoryBufferManager into host/gpu.

ServerGpuMemoryBufferManager used to live in viz/common, because it was used
by both the gpu host, and the display compositor service. Now that the latter
uses InProcessGpuMemoryBufferManager, ServerGpuMemoryBufferManager can move
into its rightful place.

BUG=733482
==========

to

==========
viz: Move ServerGpuMemoryBufferManager into host/gpu.

ServerGpuMemoryBufferManager used to live in viz/common, because it was used
by both the gpu host, and the display compositor service. Now that the latter
uses InProcessGpuMemoryBufferManager, ServerGpuMemoryBufferManager can move
into its rightful place.

BUG=733482
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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
==========

[sadrul, 2017-06-21 18:29:11]

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

[commit-bot: I haz the power, 2017-06-21 18:29:33]

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, 2017-06-21 18:32:13]

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

[commit-bot: I haz the power, 2017-06-21 18:32:13]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[sadrul, 2017-06-21 18:44:59]

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

[commit-bot: I haz the power, 2017-06-21 18:45:20]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sadrul, 2017-06-21 18:52:02]

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

[danakj, 2017-06-21 18:53:42]

What goes in host/gpu/ vs in host/ ?

[sadrul, 2017-06-21 18:56:38]

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

[sadrul, 2017-06-21 18:56:39]

+fsamuel@

On 2017/06/21 18:53:42, danakj wrote:
> What goes in host/gpu/ vs in host/ ?

I think we are going to move some of the current stuff in host/ into
host/frame_sinks? i.e. each of viz/client, viz/host, and viz/service would have
sub-directories: gpu, frame_sinks, display_compositor, hit_test (as
appropriate).

[commit-bot: I haz the power, 2017-06-21 20:46:22]

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

[commit-bot: I haz the power, 2017-06-21 20:46:25]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[danakj, 2017-06-21 21:51:32]

On 2017/06/21 18:56:39, sadrul wrote:
> +fsamuel@
> 
> On 2017/06/21 18:53:42, danakj wrote:
> > What goes in host/gpu/ vs in host/ ?
> 
> I think we are going to move some of the current stuff in host/ into
> host/frame_sinks? i.e. each of viz/client, viz/host, and viz/service would
have
> sub-directories: gpu, frame_sinks, display_compositor, hit_test (as
> appropriate).

Hm, maybe we could do that once there's enough files in viz/host/ to justify
making a split?

In cc/ at least we generally split stuff up once we reached like hundreds of
files, and then
resplit/rebalanced when some directories got too large again.

[danakj, 2017-06-21 21:52:24]

So.. LGTM to put in c/viz/host. I would suggest not the gpu subdir for now.

[sadrul, 2017-06-26 22:34:17]

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

[commit-bot: I haz the power, 2017-06-26 22:34:51]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sadrul, 2017-06-26 22:36:36]

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

[sadrul, 2017-06-26 22:36:39]

+sky@ for components/BUILD.gn, and services/ui/ws/

On 2017/06/21 21:52:24, danakj wrote:
> So.. LGTM to put in c/viz/host. I would suggest not the gpu subdir for now.

Moved out of the gpu subdir.

[commit-bot: I haz the power, 2017-06-26 23:03:50]

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

[commit-bot: I haz the power, 2017-06-26 23:03:53]

Dry run: Try jobs failed on following builders:
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sadrul, 2017-06-27 01:57:16]

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

[commit-bot: I haz the power, 2017-06-27 01:57:26]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sky, 2017-06-27 02:08:16]

LGTM

[commit-bot: I haz the power, 2017-06-27 03:43:37]

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

[commit-bot: I haz the power, 2017-06-27 03:43:38]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sadrul, 2017-06-27 04:31:50]

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

[commit-bot: I haz the power, 2017-06-27 04:32:03]

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, 2017-06-27 05:42:37]

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

[commit-bot: I haz the power, 2017-06-27 05:42:38]

Dry run: This issue passed the CQ dry run.

[sadrul, 2017-06-27 17:36:00]

The CQ bit was checked by sadrul@chromium.org

[sadrul, 2017-06-27 17:36:00]

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

[commit-bot: I haz the power, 2017-06-27 17:36: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, 2017-06-27 17:40:05]

CQ is committing da patch.

Bot data: {"patchset_id": 160001, "attempt_start_ts": 1498584960338780,
"parent_rev": "9a740797201d02dea291dfc83c8abb01f26fbc53", "commit_rev":
"20c537c4f7982d52ba84cbbef7de7f48380dbd17"}

[commit-bot: I haz the power, 2017-06-27 17:40:20]

Description was changed from

==========
viz: Move ServerGpuMemoryBufferManager into host/gpu.

ServerGpuMemoryBufferManager used to live in viz/common, because it was used
by both the gpu host, and the display compositor service. Now that the latter
uses InProcessGpuMemoryBufferManager, ServerGpuMemoryBufferManager can move
into its rightful place.

BUG=733482
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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

==========
viz: Move ServerGpuMemoryBufferManager into host/gpu.

ServerGpuMemoryBufferManager used to live in viz/common, because it was used
by both the gpu host, and the display compositor service. Now that the latter
uses InProcessGpuMemoryBufferManager, ServerGpuMemoryBufferManager can move
into its rightful place.

BUG=733482
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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

Review-Url: https://codereview.chromium.org/2941423002
Cr-Commit-Position: refs/heads/master@{#482669}
Committed:
https://chromium.googlesource.com/chromium/src/+/20c537c4f7982d52ba84cbbef7de...
==========

[commit-bot: I haz the power, 2017-06-27 17:40:21]

Committed patchset #9 (id:160001) as
https://chromium.googlesource.com/chromium/src/+/20c537c4f7982d52ba84cbbef7de...