Create separate BUILD.gn for //apps/ui/views

apps/BUILD.gn

Index: apps/BUILD.gn
diff --git a/apps/BUILD.gn b/apps/BUILD.gn
index e83a7bf2714569a2e0470e8e86d8ee35a3fc8f03..fae99193fa0d25e67101b71bb21c38df9f150879 100644
--- a/apps/BUILD.gn
+++ b/apps/BUILD.gn
@@ -37,13 +37,12 @@ static_library("apps") {
     configs += [ "//build/config/compiler:wexit_time_destructors" ]
 
     deps = [
-      "//chrome/app/theme:theme_resources",
       "//components/keyed_service/content",
       "//content/public/browser",
       "//content/public/common",
+      "//extensions/browser",

[hugoh_UTC2, 2017/05/22 15:38:25]

Somewhere here, I believe this dependency causes a little bit trouble for my
team as we sometimes compile with !enable_extensions.

Is it possible to add the |if (enable_extensions)| checks back again? 

[Nico, 2017/05/22 17:51:41]

Sure, just send me a patch.

       "//extensions/common",
       "//extensions/common/api",
-      "//skia",
     ]
 
     if (is_chromeos) {
@@ -51,21 +50,6 @@ static_library("apps") {
     }
   }
 
-  if (toolkit_views) {
-    sources += [
-      "ui/views/app_window_frame_view.cc",
-      "ui/views/app_window_frame_view.h",
-    ]
-    deps += [
-      "//cc/paint",
-      "//ui/strings",
-      "//ui/views",
-    ]
-    if (enable_extensions) {
-      deps += [ "//extensions/browser" ]
-    }
-  }
-
   # TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
   configs += [ "//build/config/compiler:no_size_t_to_int_warning" ]
 }