GN: Fix Android component build

chrome/BUILD.gn

 # Copyright 2014 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/features.gni")
 import("//build/config/locales.gni")
 import("//build/config/ui.gni")
 import("//chrome/chrome_repack_locales.gni")
 import("//chrome/version.gni")
 
@@ skipping 549 matching lines @@
   ]
 
   deps = [
     "//chrome/browser",
     "//chrome/browser/ui",
     "//chrome/plugin",
     "//chrome/renderer",
     "//chrome/utility",
     "//components/enhanced_bookmarks",
     "//content/public/app:browser",
+    "//content"

[jamesr, 2014/10/20 23:11:24]

could you explain this? things in //chrome/ should only depend on the public
parts of //content, not the internals

[cjhopman, 2014/10/21 00:19:06]

This matches the gyp build: chrome/chrome_android.gypi:33

I'm no expert here, but it appears to be that, when chrome_multiple_dll is
false, //content/app should depend on //content/gpu. Making that change resolves
all the undefined references (but I don't know content/ well enough to say that
that is correct).

This also only actually affects the static_library build since in the component
build any dependency on the public parts of content becomes a dependency on
//content.

Also, //content is a bundling of all the public targets and //content/gpu, to
me, it doesn't look like an internal target (so maybe we should comment there
or, even better, restrict its visibility).

   ]
 }
 
 }