Pull gpu CmdBuf service/client shared GpuMemoryBuffer code to common

gpu/command_buffer/common/BUILD.gn

 # Copyright (c) 2013 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.
 
 # The files here go into the "gpu" component in a component build (with
 # "command_buffer_common" just forwarding) and goes into a static library in
 # non-component build. This needs to match the GYP build which was likely an
 # attempt to make larger components to help with loading.
 import("//mojo/public/tools/bindings/mojom.gni")
 
@@ skipping 44 matching lines @@
     "value_state.cc",
     "value_state.h",
   ]
 
   configs += [ "//gpu:gpu_implementation" ]
 
   deps = [
     ":gles2_utils",
     "//base",
   ]
+
+  # gpu_memory_buffer_support.cc includes //ui/gl/gl_bindings.h which doesn't
+  # compile on nacl.

[piman, 2016/04/04 20:49:10]

Can you replace the include by <GLES2/gl2.h> and use
//third_party/khronos:khronos_headers as a dependency?

That should compile correctly in NaCl, it's needed by other bits.

[Peng, 2016/04/04 20:54:35]

gpu_memory_buffer_support.cc needs some chromium specified GL defines which are
defined in ui/gl/gl_bindings.h, so have to depends on it.

For example: GL_RGB_YCBCR_422_CHROMIUM.

[Peng, 2016/04/06 14:28:54]

Done.

+  if (!is_nacl) {
+    sources += [
+      "gpu_memory_buffer_support.cc",
+      "gpu_memory_buffer_support.h",
+    ]
+    public_deps = [
+      "//ui/gfx",
+      "//ui/gfx/geometry",
+    ]
+    deps += [ "//ui/gl" ]
+  }
 }
 
 component("gles2_utils") {
   sources = [
     "gles2_cmd_utils.cc",
     "gles2_cmd_utils.h",
     "gles2_utils_export.h",
   ]
 
   defines = [ "GLES2_UTILS_IMPLEMENTATION" ]
@@ skipping 11 matching lines @@
     "command_buffer.mojom",
     "mailbox.mojom",
     "mailbox_holder.mojom",
     "sync_token.mojom",
     "value_state.mojom",
   ]
 
   typemaps = [ "mojo.typemap" ]
   typemap_deps = [ "//gpu/ipc/common:command_buffer_traits" ]
 }