Componentize Constrained Window Views

components/constrained_window/BUILD.gn

Index: components/constrained_window/BUILD.gn
diff --git a/components/auto_login_parser/BUILD.gn b/components/constrained_window/BUILD.gn
similarity index 50%
copy from components/auto_login_parser/BUILD.gn
copy to components/constrained_window/BUILD.gn
index b59919502b3e2d99ddd06c5821f6ec5eb25d25b1..1e8f9dd20369946a9a86290e70b78048720c3119 100644
--- a/components/auto_login_parser/BUILD.gn
+++ b/components/constrained_window/BUILD.gn
@@ -2,26 +2,27 @@
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
-source_set("auto_login_parser") {
+static_library("constrained_window") {
   sources = [
-    "auto_login_parser.cc",
-    "auto_login_parser.h",
+    "constrained_window_views.cc",

[msw, 2014/10/22 21:32:43]

Ditto: Shouldn't this list include constrained_window_views_client.h?

[oshima, 2014/10/22 22:39:09]

Done.

+    "constrained_window_views.h",
   ]
 
   deps = [
-    "//base",
-    "//net",
+    "//components/web_modal",
+    "//ui/views",
+    "//skia",

[msw, 2014/10/22 21:32:43]

Ditto: Skia doesn't appear to be used directly, is there a transitive dependency
through views, and if so, would removing this explicit dependency entry be okay?

   ]
 }
 
 source_set("unit_tests") {
   testonly = true
   sources = [
-    "auto_login_parser_unittest.cc",
+    "constrained_window_views_unittest.cc",
   ]
 
   deps = [
-    ":auto_login_parser",
-    "//testing/gtest",
+    ":constrained_window",
+    "//ui/views:test_support",
   ]
 }