mus: Move MusGpuMemoryBufferManager into common.

services/ui/common/BUILD.gn

Index: services/ui/common/BUILD.gn
diff --git a/services/ui/common/BUILD.gn b/services/ui/common/BUILD.gn
index a52f23d7ce8094f98df8d847c2e9cf179f12bae2..a00b64f6d2867f043aca48a77443669f7b9f9e0a 100644
--- a/services/ui/common/BUILD.gn
+++ b/services/ui/common/BUILD.gn
@@ -10,8 +10,6 @@ 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",
@@ -38,6 +36,26 @@ source_set("mus_common") {
   ]
 }
 
+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