viz: Move some code into //components/viz/common.

components/viz/common/BUILD.gn

Index: components/viz/common/BUILD.gn
diff --git a/components/viz/common/BUILD.gn b/components/viz/common/BUILD.gn
new file mode 100644
index 0000000000000000000000000000000000000000..e3a4c6ad39640928dad037e42104c6301c5c647b
--- /dev/null
+++ b/components/viz/common/BUILD.gn
@@ -0,0 +1,22 @@
+# Copyright 2017 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.
+
+source_set("common") {
+  sources = [
+    "server_gpu_memory_buffer_manager.cc",
+    "server_gpu_memory_buffer_manager.h",
+  ]
+
+  deps = [
+    "//gpu/ipc/client",
+  ]
+
+  public_deps = [
+    "//gpu/command_buffer/client",
+    "//gpu/ipc/common",
+    "//gpu/ipc/host",
+    "//services/ui/gpu/interfaces",

[danakj, 2017/06/14 22:57:57]

does this need to be a public dep for a forward declare?

i'm not sure what the other deps are for, can you make sure they make sense as
public deps too for the whole directory?

[sadrul, 2017/06/14 23:18:33]

I moved gpu/ipc/common into deps. But the rest are exposed in the header file,
so it would be better to leave them in public deps.

For mojom, the user of ServerGMBManager has to inject in the mojom impl. So it
feels reasonable to me to make it a public dep too.

I am not really sure if all of these deps will apply equally to all code in
here. We could have sub-components here too, but maybe we can wait until this
grows a bit to see how/whether we can organize it better?

[danakj, 2017/06/15 15:08:58]

Similar to the header, the user of this has to construct the mojom types and
pass them in, so it needs to depend on the mojom targets in order to call the
constructors. That dep does not need to come from here.

+    "//ui/gfx",
+  ]
+}