Componentize Constrained Window Views

chrome/chrome_browser_ui.gypi

Index: chrome/chrome_browser_ui.gypi
diff --git a/chrome/chrome_browser_ui.gypi b/chrome/chrome_browser_ui.gypi
index d08c6c5db6fa6ef97f1d9e96c4dd2dc275223c23..437a2718d1247aabba82eceb904b00fa842ea2c6 100644
--- a/chrome/chrome_browser_ui.gypi
+++ b/chrome/chrome_browser_ui.gypi
@@ -1930,6 +1930,8 @@
       'browser/ui/views/bookmarks/bookmark_sync_promo_view.cc',
       'browser/ui/views/bookmarks/bookmark_sync_promo_view.h',
       'browser/ui/views/certificate_viewer_win.cc',
+      'browser/ui/views/chrome_constrained_window_views_client.cc',
+      'browser/ui/views/chrome_constrained_window_views_client.h',
       'browser/ui/views/chrome_views_delegate_chromeos.cc',
       'browser/ui/views/chrome_web_dialog_view.cc',
       'browser/ui/views/collected_cookies_views.cc',
@@ -1942,8 +1944,6 @@
       'browser/ui/views/conflicting_module_view_win.cc',
       'browser/ui/views/conflicting_module_view_win.h',
       'browser/ui/views/constrained_web_dialog_delegate_views.cc',
-      'browser/ui/views/constrained_window_views.cc',
-      'browser/ui/views/constrained_window_views.h',
       'browser/ui/views/content_setting_bubble_contents.cc',
       'browser/ui/views/content_setting_bubble_contents.h',
       'browser/ui/views/cookie_info_view.cc',
@@ -2774,6 +2774,7 @@
             ['OS!="mac"', {
               'sources': [ '<@(chrome_browser_ui_views_non_mac_sources)' ],
               'dependencies': [
+                '<(DEPTH)/components/components.gyp:constrained_window',

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

This perhaps should be simply contingent on 'toolkit_views==1', not sure if the
'OS!="mac"' condition is necessary, but perhaps it doesn't matter much for now.
+CC andresantoso for thoughts.

[Andre, 2014/10/22 21:45:03]

Yep, but we probably should do that when contrained_window_views compiles for
MacViews, which it does not yet right now.
So this looks right for now.

                 '<(DEPTH)/extensions/components/extensions_components.gyp:native_app_window',
               ],
             }],