extensions: Check |enable_extensions| flag before depending on extension code.

chrome/browser/ui/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/crypto.gni")
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 
 gypi_values = exec_script("//build/gypi_to_gn.py",
                           [ rebase_path("../../chrome_browser_ui.gypi") ],
@@ skipping 188 matching lines @@
                            ".",
                            "//chrome")
     deps += [ "//components/constrained_window" ]
     if (!is_chromeos) {
       sources +=
           rebase_path(gypi_values.chrome_browser_ui_views_non_chromeos_sources,
                       ".",
                       "//chrome")
     }
     if (!is_mac) {
-      sources +=
-          rebase_path(gypi_values.chrome_browser_ui_views_non_mac_sources,
-                      ".",
-                      "//chrome")
-      deps += [ "//extensions/components/native_app_window" ]
+      if (!enable_extensions) {
+        set_sources_assignment_filter(
+            sources_assignment_filter + [ "*/extensions/*" ])

[brettw, 2015/06/07 22:48:00]

Our rule in the GN build is to only use the filter for platform stuff, not
features.

Can you separate out the views extension sources into a separate variable and
conditionally include them?

[sadrul, 2015/06/09 21:08:35]

Done.

+        sources +=
+            rebase_path(gypi_values.chrome_browser_ui_views_non_mac_sources,
+                        ".",
+                        "//chrome")
+        set_sources_assignment_filter(sources_assignment_filter)
+      } else {
+        sources +=
+            rebase_path(gypi_values.chrome_browser_ui_views_non_mac_sources,
+                        ".",
+                        "//chrome")
+        deps += [ "//extensions/components/native_app_window" ]
+      }
       if (is_chromeos) {
         # TODO(GYP): crbug.com/481629. These files should probably not be
         # a part of ui_views_non_mac_sources at all.
         sources -= [
           "views/frame/opaque_browser_frame_view.cc",
           "views/frame/opaque_browser_frame_view.h",
           "views/frame/opaque_browser_frame_view_linux.cc",
           "views/frame/opaque_browser_frame_view_linux.h",
         ]
       }
@@ skipping 289 matching lines @@
     "//chrome/browser",
     "//content/public/browser",
     "//content/public/common",
     "//content/test:test_support",
     "//net:test_support",
     "//skia",
     "//testing/gtest",
     "//ui/base",
   ]
 }