mus: Move MusGpuMemoryBufferManager into common.

services/ui/common/BUILD.gn

 # Copyright 2015 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/config/ui.gni")
 import("//services/service_manager/public/service_manifest.gni")
 import("//testing/test.gni")
 
 source_set("mus_common") {
   sources = [
     "event_matcher_util.cc",
     "event_matcher_util.h",
-    "generic_shared_memory_id_generator.cc",
-    "generic_shared_memory_id_generator.h",
     "switches.cc",
     "switches.h",
     "transient_window_utils.h",
     "types.h",
     "util.h",
   ]
 
   deps = [
     "//gpu/command_buffer/client",
     "//gpu/config",
     "//gpu/ipc/client",
     "//gpu/ipc/common:command_buffer_traits",
     "//ipc:ipc",
     "//services/service_manager/public/cpp",
     "//ui/events:events",
     "//ui/gfx/ipc/geometry",
   ]
 
   public_deps = [
     "//mojo/public/cpp/bindings",
     "//mojo/public/cpp/system",
     "//services/ui/public/interfaces",
     "//ui/base",
   ]
 }
 
+source_set("service_common") {

[sky, 2016/11/26 16:42:19]

service_common is quite redundant given this is the common directory in the
service directory. How about 'gpu'? At least add a description as what this is
for.

[sadrul, 2016/11/27 01:13:44]

The 'mus_common' target above is used by both the window-server and mus-clients.
That's why I thought it'd be good to have a name that makes it obvious that this
is not for clients. Maybe server_common, or service_gpu?

[sadrul, 2016/11/27 04:16:25]

Renamed to 'server_gpu', and added comment here to mention where this is used.

+  sources = [
+    "generic_shared_memory_id_generator.cc",
+    "generic_shared_memory_id_generator.h",
+    "mus_gpu_memory_buffer_manager.cc",
+    "mus_gpu_memory_buffer_manager.h",
+  ]
+
+  deps = [
+    "//gpu/ipc/client",
+  ]
+
+  public_deps = [
+    "//gpu/command_buffer/client",
+    "//gpu/ipc/common",
+    "//services/ui/gpu/interfaces",
+    "//ui/gfx",
+  ]
+}
+
 source_set("run_all_service_tests") {
   testonly = true
 
   sources = [
     "run_all_service_tests.cc",
   ]
 
   deps = [
     "//base/test:test_support",
     "//mojo/edk/system",
@@ skipping 12 matching lines @@
     "task_runner_test_base.cc",
     "task_runner_test_base.h",
   ]
 
   deps = [
     "//base",
     "//base/test:test_support",
     "//testing/gtest",
   ]
 }