Allow generation of a chromium outdirs when enable_extensions=false

chrome/browser/media/router/BUILD.gn

Index: chrome/browser/media/router/BUILD.gn
diff --git a/chrome/browser/media/router/BUILD.gn b/chrome/browser/media/router/BUILD.gn
index 14b5095fc38cd2f75afe25553d57e1e35f34f4a1..389fa63780645b7bc73f9c291ea6b28ea4dac78e 100644
--- a/chrome/browser/media/router/BUILD.gn
+++ b/chrome/browser/media/router/BUILD.gn
@@ -3,6 +3,7 @@
 # found in the LICENSE file.
 #
 
+import("//extensions/features/features.gni")
 import("//testing/test.gni")
 
 static_library("router") {
@@ -106,8 +107,10 @@ static_library("test_support") {
     deps += [
       "//chrome/common/media_router/mojo:media_router",
       "//chrome/common/media_router/mojo:media_router_test_interfaces",
-      "//extensions/common",
     ]
+    if (enable_extensions) {
+      deps += [ "//extensions/common" ]

[michaelpg, 2017/05/23 16:54:40]

media-router sources unconditionally include files from extensions/common.
Either the tests target shouldn't exist if !enable_extensions, or... this kind
of defeats the purpose of deps, I think.

[hugoh_UTC2, 2017/05/24 13:58:06]

When !is_android, static_library("test_support") should only depend on
extensions if extensions_enabled? (But this test target should still exist?)

+    }
     sources += [
       "discovery/mdns/mock_dns_sd_registry.cc",
       "discovery/mdns/mock_dns_sd_registry.h",