[Mac] Include MacViews native app windows in regular builds.

chrome/chrome_browser_ui.gypi

 # Copyright 2012 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.
 
 {
   'variables': {
     # There are two sections of sources:
     # * "Platform"-based sources (OS, toolkit, etc.)
     # * Feature/directory-based sources (e.g., history, printing)
 
@@ skipping 1933 matching lines @@
       'browser/ui/webui/system_info_ui.h',
       'browser/ui/webui/theme_handler.cc',
       'browser/ui/webui/theme_handler.h',
       'browser/ui/webui/uber/uber_ui.cc',
       'browser/ui/webui/uber/uber_ui.h',
       'browser/ui/window_sizer/window_sizer.cc',
       'browser/ui/window_sizer/window_sizer.h',
     ],
     # Cross-platform views sources also ready for toolkit-views on Mac.
     'chrome_browser_ui_views_sources': [
+
+      'browser/ui/views/apps/app_window_native_widget_mac.h',
+      'browser/ui/views/apps/app_window_native_widget_mac.mm',
+      'browser/ui/views/apps/chrome_native_app_window_views_mac.h',
+      'browser/ui/views/apps/chrome_native_app_window_views_mac.mm',
+      'browser/ui/views/apps/native_app_window_frame_view_mac.h',
+      'browser/ui/views/apps/native_app_window_frame_view_mac.mm',
+      'browser/ui/views/extensions/extension_keybinding_registry_views.cc',

[tapted, 2015/06/04 04:48:00]

I don't think this will work. Cocoa's BrowserWindowController only initializes a
ExtensionKeybindingRegistryCocoa. I think NativeAppWindowViews will need to be
refactored to target their abstract base class instead (i.e.
ExtensionsKeybindingRegistry). 

I think extension_keybinding_registry_views.cc will only funciton correctly with
a toolkit-views browser (i.e. it should stay in the mac_views_browser build
only)

[jackhou1, 2015/06/04 06:06:40]

extension_keybinding_registry_views.cc doesn't seem to depend on much from the
toolkit-views browser. I figured it would be better to use that and make it work
since we'd get to exercise some code that'll be in the mac-views browser. I
don't know much about how the views keybinding registry works, but it seems
possible to have both implementations.

I'm not sure if refactoring ChromeNativeAppWindowViews to use
ExtensionKeybindingRegistryCocoa is straightforward either. With
NativeAppWindowCocoa, events come in via ChromeEventProcessingWindow which calls
NativeAppWindowCocoa::HandledByExtensionCommand, which directly calls
ExtensionKeybindingRegistryCocoa::ProcessKeyEvent.

[tapted, 2015/06/05 02:09:08]

Hm - yeah I think it will be OK. A "registry" sounds a bit like a singleton, but
that doesn't seem to be the case.

Another concern might be that ExtensionKeybindingRegistryCocoa::ProcessKeyEvent
emits, e.g.,  extensions::NOTIFICATION_EXTENSION_COMMAND_PAGE_ACTION_MAC but the
views one does not. However, I'm pretty sure that code path doesn't matter for
packaged apps either way. i.e. only keybindings on the browser window should be
invoking browser actions.

So this lg - I don't think anything will break, but there might be some test
cases to try out.

+      'browser/ui/views/extensions/extension_keybinding_registry_views.h',
+      'browser/ui/views/frame/native_widget_mac_frameless_nswindow.h',
+      'browser/ui/views/frame/native_widget_mac_frameless_nswindow.mm',

[tapted, 2015/06/04 04:48:00]

sort into the ui/views stuff below

[jackhou1, 2015/06/04 06:06:40]

Done.

+
+

[tapted, 2015/06/04 04:48:00]

nit: no blank lines

[jackhou1, 2015/06/04 06:06:40]

Done.

       'browser/ui/app_list/app_list_controller_delegate_views.cc',
       'browser/ui/app_list/app_list_controller_delegate_views.h',
       'browser/ui/app_list/app_list_service_views.cc',
       'browser/ui/app_list/app_list_service_views.h',
       'browser/ui/app_list/app_list_shower_views.cc',
       'browser/ui/app_list/app_list_shower_views.h',
       'browser/ui/views/apps/app_info_dialog/app_info_dialog_views.cc',
       'browser/ui/views/apps/app_info_dialog/app_info_dialog_views.h',
       'browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc',
       'browser/ui/views/apps/app_info_dialog/app_info_footer_panel.h',
@@ skipping 152 matching lines @@
       'browser/ui/views/extensions/device_permissions_dialog_view.h',
       'browser/ui/views/extensions/extension_action_platform_delegate_views.cc',
       'browser/ui/views/extensions/extension_action_platform_delegate_views.h',
       'browser/ui/views/extensions/extension_dialog.cc',
       'browser/ui/views/extensions/extension_dialog.h',
       'browser/ui/views/extensions/extension_dialog_observer.cc',
       'browser/ui/views/extensions/extension_dialog_observer.h',
       'browser/ui/views/extensions/extension_install_dialog_view.cc',
       'browser/ui/views/extensions/extension_installed_bubble_view.cc',
       'browser/ui/views/extensions/extension_installed_bubble_view.h',
-      'browser/ui/views/extensions/extension_keybinding_registry_views.cc',
-      'browser/ui/views/extensions/extension_keybinding_registry_views.h',
       'browser/ui/views/extensions/extension_message_bubble_view.cc',
       'browser/ui/views/extensions/extension_message_bubble_view.h',
       'browser/ui/views/extensions/extension_popup.cc',
       'browser/ui/views/extensions/extension_popup.h',
       'browser/ui/views/extensions/extension_popup_aura.cc',
       'browser/ui/views/extensions/extension_popup_aura.h',
       'browser/ui/views/extensions/extension_toolbar_icon_surfacing_bubble_views.cc',
       'browser/ui/views/extensions/extension_toolbar_icon_surfacing_bubble_views.h',
       'browser/ui/views/extensions/extension_uninstall_dialog_view.cc',
       'browser/ui/views/extensions/extension_view_views.cc',
@@ skipping 44 matching lines @@
       'browser/ui/views/frame/immersive_mode_controller_factory_mac.cc',
       'browser/ui/views/frame/immersive_mode_controller_stub.cc',
       'browser/ui/views/frame/immersive_mode_controller_stub.h',
       'browser/ui/views/frame/minimize_button_metrics_win.cc',
       'browser/ui/views/frame/minimize_button_metrics_win.h',
       'browser/ui/views/frame/native_browser_frame.h',
       'browser/ui/views/frame/native_browser_frame_factory.cc',
       'browser/ui/views/frame/native_browser_frame_factory.h',
       'browser/ui/views/frame/native_browser_frame_factory_chromeos.cc',
       'browser/ui/views/frame/native_browser_frame_factory_mac.cc',
-      'browser/ui/views/frame/native_widget_mac_frameless_nswindow.h',
-      'browser/ui/views/frame/native_widget_mac_frameless_nswindow.mm',
       'browser/ui/views/frame/opaque_browser_frame_view.cc',
       'browser/ui/views/frame/opaque_browser_frame_view.h',
       'browser/ui/views/frame/opaque_browser_frame_view_layout.cc',
       'browser/ui/views/frame/opaque_browser_frame_view_layout.h',
       'browser/ui/views/frame/opaque_browser_frame_view_layout_delegate.h',
       'browser/ui/views/frame/opaque_browser_frame_view_linux.cc',
       'browser/ui/views/frame/opaque_browser_frame_view_linux.h',
       'browser/ui/views/frame/opaque_browser_frame_view_platform_specific.cc',
       'browser/ui/views/frame/opaque_browser_frame_view_platform_specific.h',
       'browser/ui/views/frame/system_menu_insertion_delegate_win.cc',
@@ skipping 207 matching lines @@
       'browser/ui/views/website_settings/permission_selector_view_observer.h',
       'browser/ui/views/website_settings/permissions_bubble_view.cc',
       'browser/ui/views/website_settings/permissions_bubble_view.h',
       'browser/ui/views/website_settings/website_settings_popup_view.cc',
       'browser/ui/views/website_settings/website_settings_popup_view.h',
     ],
     # MacViews sources that we still want to keep behind a compile-time flag.
     # TODO(jackhou): Move items to chrome_browser_ui_views_sources when they
     # migrate from mac_views_browser to a chrome://flag.
     'chrome_browser_ui_views_mac_experimental_sources': [
-      'browser/ui/views/apps/app_window_native_widget_mac.h',
-      'browser/ui/views/apps/app_window_native_widget_mac.mm',
       'browser/ui/views/apps/chrome_app_window_client_views_mac.mm',

[tapted, 2015/06/04 04:48:00]

ah! it already exists. So I think we just move this file into the build, have it
support Cocoa windows, delete this entire gyp section, and finally delete
chrome_app_window_client_cocoa.mm

-      'browser/ui/views/apps/chrome_native_app_window_views_mac.h',
-      'browser/ui/views/apps/chrome_native_app_window_views_mac.mm',
-      'browser/ui/views/apps/native_app_window_frame_view_mac.h',
-      'browser/ui/views/apps/native_app_window_frame_view_mac.mm',
     ],
     # Windows-only. Assume ash/aura/views.
     'chrome_browser_ui_win_sources': [
       'browser/ui/network_profile_bubble.cc',
       'browser/ui/network_profile_bubble.h',
       'browser/ui/views/color_chooser_dialog.cc',
       'browser/ui/views/color_chooser_dialog.h',
       'browser/ui/views/frame/browser_desktop_window_tree_host.h',
       'browser/ui/views/frame/browser_desktop_window_tree_host_win.cc',
       'browser/ui/views/frame/browser_desktop_window_tree_host_win.h',
@@ skipping 457 matching lines @@
               'sources': [ '<@(chrome_browser_ui_ash_non_chromeos)' ],
             }],
           ],
         }, { # use_ash==0
           'sources': [ '<@(chrome_browser_ui_non_ash_sources)' ],
         }],
         ['toolkit_views==1', {
           'sources': [ '<@(chrome_browser_ui_views_sources)' ],
           'dependencies': [
             '<(DEPTH)/components/components.gyp:constrained_window',
+            '<(DEPTH)/extensions/components/extensions_components.gyp:native_app_window',
           ],
           'conditions': [
             ['chromeos == 0 and (OS!="mac" or mac_views_browser==1)', {
               'sources': [ '<@(chrome_browser_ui_views_non_chromeos_sources)' ],
             }],
             ['use_ash == 1', {
               'sources': [ '<@(chrome_browser_ui_ash_views_sources)' ],
             }],
             ['OS!="mac"', {
               'sources': [ '<@(chrome_browser_ui_views_non_mac_sources)' ],
-              'dependencies': [
-                '<(DEPTH)/extensions/components/extensions_components.gyp:native_app_window',
-              ],
             }],
           ],
         }],
         ['use_aura==1 and chromeos==0 and use_ozone==0 and OS=="linux"', {
           'dependencies': [
             # gtk2 is the only component that can interact with gtk2 in our new
             # world.
             'browser/ui/libgtk2ui/libgtk2ui.gyp:gtk2ui',
             '../build/linux/system.gyp:gio',
           ],
@@ skipping 52 matching lines @@
             '../third_party/google_toolbox_for_mac/google_toolbox_for_mac.gyp:google_toolbox_for_mac',
             '../third_party/molokocacao/molokocacao.gyp:molokocacao',
             '../third_party/mozilla/mozilla.gyp:mozilla',
           ],
           'conditions': [
             ['mac_views_browser==1', {
               'sources': [
                 '<@(chrome_browser_ui_views_mac_experimental_sources)',
                 '<@(chrome_browser_ui_views_non_mac_sources)',
               ],
-              'dependencies': [
-                '<(DEPTH)/extensions/components/extensions_components.gyp:native_app_window',
-              ],
             }, {
               'sources': [ '<@(chrome_browser_ui_cocoa_sources)' ],
             }],
           ],
           'link_settings': {
             'libraries': [
               '$(SDKROOT)/System/Library/Frameworks/Quartz.framework',
             ],
           },
           'actions': [
@@ skipping 191 matching lines @@
           ],
           'sources': [
             '<@(chrome_browser_ui_non_mobile_sources)',
             '<@(chrome_browser_ui_omnibox_non_mobile_sources)',
           ],
         }],
       ],
     },
   ],
 }