Update deps in extensions/browser/BUILD.gn to pass gn check.

extensions/browser/BUILD.gn

Index: extensions/browser/BUILD.gn
diff --git a/extensions/browser/BUILD.gn b/extensions/browser/BUILD.gn
index 963ab0b95ec8300a742c7b52bf07bc57557f38a2..a23b309e05f091be7ba20b53f1646df8e9f207ea 100644
--- a/extensions/browser/BUILD.gn
+++ b/extensions/browser/BUILD.gn
@@ -5,7 +5,41 @@
 import("//build/config/features.gni")
 import("//extensions/features/features.gni")
 
-source_set("browser") {
+group("browser") {
+  public_deps = [
+    "//extensions/browser:browser_context_keyed_service_factories",
+    "//extensions/browser:browser_sources",
+    "//extensions/browser/api:api_registration",
+  ]
+
+  if (enable_extensions) {

[Devlin, 2017/02/13 16:07:21]

what includes extensions/* when enable_extensions is false?  We have a good
number of asserts to try and make sure this doesn't happen, e.g.
https://chromium.googlesource.com/chromium/src/+/28ff32abc83ae8939f56702151b3....
 Where does this fail?

[michaelpg, 2017/02/13 21:08:42]

Dunno, maybe nowhere. But I don't want to remove this existing check (moved from
below) in combination with this re-grouping. 

In fact there's already a TODO for this (also moved from below). It may already
be toDONE, but there are other configs I can't check right now (Win/Mac).

[Devlin, 2017/02/13 22:15:12]

Hmm... okay.  But let's look into this soon.

+    # Includes all API implementations and the ExtensionsApiClient
+    # interface. Moving an API from src/chrome to src/extensions implies
+    # it can be cleanly disabled with enable_extensions=false.
+    # TODO: Eventually the entire extensions module should not be built
+    # when enable_extensions=false.
+    public_deps += [ "//extensions/browser/api" ]
+  }
+}
+
+# Isolate the browser's dependency on APIs to the class that instantiates them.

[Devlin, 2017/02/13 16:07:21]

Nit: This isn't a class.  Also, not all of these are API-related.  Maybe just
// Isolate the instantiation of BrowserContextKeyedServices.
?

[michaelpg, 2017/02/13 21:08:42]

Done.

+source_set("browser_context_keyed_service_factories") {
+  visibility = [ ":*" ]
+
+  sources = [
+    "browser_context_keyed_service_factories.cc",
+    "browser_context_keyed_service_factories.h",
+  ]
+
+  deps = [
+    "//extensions/browser:browser_sources",
+    "//extensions/browser/api:api",
+  ]
+}
+
+source_set("browser_sources") {
+  visibility = [ "./*" ]
+
   sources = []
 
   deps = [
@@ -22,8 +56,8 @@ source_set("browser") {
     "//components/web_modal",
     "//components/zoom",
     "//content/public/browser",
+    "//content/public/common",
     "//crypto:platform",
-    "//extensions/browser/api:api_registration",
     "//extensions/common",
     "//extensions/common/api",
     "//extensions/features",
@@ -40,14 +74,7 @@ source_set("browser") {
   ]
 
   if (enable_extensions) {
-    # Includes all API implementations and the ExtensionsApiClient
-    # interface. Moving an API from src/chrome to src/extensions implies
-    # it can be cleanly disabled with enable_extensions=false.
-    # TODO: Eventually the entire extensions module should not be built
-    # when enable_extensions=false.
     sources = [
-      # NOTE: When moving an API out of Chrome be sure to verify that the
-      # Android build still compiles. See conditions below.
       "api_activity_monitor.cc",
       "api_activity_monitor.h",
       "app_sorting.h",
@@ -58,8 +85,6 @@ source_set("browser") {
       "blob_holder.h",
       "blocked_action_type.h",
       "browser_context_keyed_api_factory.h",
-      "browser_context_keyed_service_factories.cc",
-      "browser_context_keyed_service_factories.h",
       "component_extension_resource_manager.h",
       "computed_hashes.cc",
       "computed_hashes.h",
@@ -254,7 +279,6 @@ source_set("browser") {
     ]
 
     public_deps = [
-      "//extensions/browser/api",
       "//extensions/browser/app_window",
       "//extensions/browser/guest_view",
       "//extensions/browser/install",
@@ -264,26 +288,16 @@ source_set("browser") {
     ]
 
     deps += [
-      "//components/browsing_data/content",
-      "//components/onc",
-      "//components/storage_monitor",
+      "//components/crx_file",
+      "//components/prefs",
+      "//components/sync",
       "//components/update_client",
       "//components/variations",
       "//crypto:platform",
-      "//device/base",
-      "//device/bluetooth",
-      "//device/hid",
-      "//device/power_save_blocker",
-      "//device/serial",
-      "//device/usb",
       "//extensions:extensions_browser_resources",
       "//extensions/common:mojo",
       "//services/service_manager/public/cpp",
     ]
-
-    if (is_chromeos) {
-      deps += [ "//chromeos" ]
-    }
   }
 }