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

apps/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/ui.gni")
 import("//extensions/features/features.gni")
 
 assert(!is_android && !is_ios)
 
@@ skipping 19 matching lines @@
       "saved_files_service.h",
       "saved_files_service_factory.cc",
       "saved_files_service_factory.h",
       "switches.cc",
       "switches.h",
     ]
 
     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) {
       deps += [ "//components/user_manager" ]
     }
   }
 
-  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" ]
 }
 
 static_library("test_support") {
   testonly = true
   sources = [
     "test/app_window_waiter.cc",
     "test/app_window_waiter.h",
   ]
 
   public_deps = [
     "//content/public/browser",
   ]
 }