Prevent static mojo builds from pulling //ui/gl into mojo_shell

gpu/command_buffer/client/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.
 
 source_set("client") {
   sources = [
     "cmd_buffer_helper.cc",
     "cmd_buffer_helper.h",
     "fenced_allocator.cc",
     "fenced_allocator.h",
@@ skipping 105 matching lines @@
     # TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
     cflags = [ "/wd4267" ]  # size_t to int truncation.
   }
 
   deps = [
     ":gles2_cmd_helper",
     ":gles2_interface",
     "//base",
     "//gpu/command_buffer/common",
     "//ui/gfx/geometry",
-    "//ui/gl",

[jamesr, 2014/09/25 20:09:44]

this isn't quite correct - gles2_implementation.cc #includes
ui/gl/gl_switches.h, so it should declare a dependency on something to match the
code.  Could you split the switches out into a separate target perhaps?

[DaveMoore, 2014/09/26 18:29:22]

I created a new kEnableGPUServiceLoggingGPU flag in the new
client/gpu_switches.{cc,h} in a way that matches the use of
kEnableGPUServiceLoggingGPU in service/gpu_switches.{cc,h}

On 2014/09/25 20:09:44, jamesr wrote:

   ]
 }
 
 # Library emulates GLES2 using command_buffers.
 component("gles2_implementation_client_side_arrays") {
   sources = gles2_implementation_source_files
 
   defines = [
     "GLES2_IMPL_IMPLEMENTATION",
     "GLES2_SUPPORT_CLIENT_SIDE_ARRAYS=1",
@@ skipping 46 matching lines @@
 
   deps = [
     ":client",
     ":gles2_interface",
     "//base",
     "//base/third_party/dynamic_annotations",
     "//gpu/command_buffer/common",
   ]
 }