MacViews: Initial chrome_browser_ui.gypi support for views on mac

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': {
     'chrome_browser_ui_non_ios_sources': [
       'browser/ui/accelerator_utils.h',
       'browser/ui/android/android_about_app_info.cc',
       'browser/ui/android/android_about_app_info.h',
@@ skipping 1804 matching lines @@
       'browser/ui/sync/one_click_signin_bubble_links_delegate.cc',
       'browser/ui/sync/one_click_signin_bubble_links_delegate.h',
       'browser/ui/sync/one_click_signin_helper.cc',
       'browser/ui/sync/one_click_signin_helper.h',
       'browser/ui/sync/one_click_signin_histogram.h',
       'browser/ui/sync/one_click_signin_sync_starter.cc',
       'browser/ui/sync/one_click_signin_sync_starter.h',
       'browser/ui/views/sync/one_click_signin_bubble_view.cc',
       'browser/ui/views/sync/one_click_signin_bubble_view.h',
     ],
+    'chrome_browser_ui_task_manager_non_mac_sources': [

[Andre, 2014/07/18 20:08:57]

I think TaskManagerView is now ready for MacViews.
Can we make task_manager_view.cc part of chrome_browser_ui_views_sources?
(conditional on enable_task_manager=1)
We may need to add ifdef to avoid duplicate ShowTaskManager.

+      'browser/ui/views/task_manager_view.cc',
+    ],
     'chrome_browser_ui_task_manager_sources': [
-      'browser/ui/views/task_manager_view.cc',
       'browser/ui/cocoa/task_manager_mac.h',
       'browser/ui/cocoa/task_manager_mac.mm',

[Andre, 2014/07/18 20:08:57]

These 2 files are also part of chrome_browser_ui_non_ios_sources.
Delete them from there?
That may break the TaskManagerMac code in AppControllerMac if
enable_task_manager is 0. We can change that code to call
chrome::OpenTaskManager instead.

[tapted, 2014/07/23 03:20:31]

I'll think up some way to consolidate these.. For 3 files, task manager sure
does appear a lot. The gn build rules have much neater filtering - I might be
able to mimic that.

[tapted, 2014/07/29 08:04:10]

Done (task_manager_mac.* effectively appeared double, so we can just drop one).

     ],
     'chrome_browser_ui_nacl_sources': [
       'browser/ui/webui/nacl_ui.cc',
       'browser/ui/webui/nacl_ui.h',
     ],
     'chrome_browser_ui_plugin_sources': [
       'browser/ui/hung_plugin_tab_helper.cc',
       'browser/ui/hung_plugin_tab_helper.h',
     ],
     'chrome_browser_ui_policy_sources': [
       'browser/ui/webui/policy_ui.cc',
       'browser/ui/webui/policy_ui.h',
     ],
-    # Cross-platform views sources. We assume app list is enabled.
-    'chrome_browser_ui_views_sources': [
-      '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',
+    # Cross-platform (except Mac) views sources. We assume app list is enabled.
+    'chrome_browser_ui_views_non_mac_sources': [
       'browser/ui/views/accelerator_utils_aura.cc',
-      'browser/ui/views/accessibility/accessibility_event_router_views.cc',
-      'browser/ui/views/accessibility/accessibility_event_router_views.h',
       'browser/ui/views/accessibility/invert_bubble_view.cc',
       'browser/ui/views/accessibility/invert_bubble_view.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',
       'browser/ui/views/apps/app_info_dialog/app_info_header_panel.cc',
       'browser/ui/views/apps/app_info_dialog/app_info_header_panel.h',
       'browser/ui/views/apps/app_info_dialog/app_info_panel.cc',
       'browser/ui/views/apps/app_info_dialog/app_info_panel.h',
@@ skipping 54 matching lines @@
       'browser/ui/views/bookmarks/bookmark_editor_view.cc',
       'browser/ui/views/bookmarks/bookmark_editor_view.h',
       'browser/ui/views/bookmarks/bookmark_menu_controller_observer.h',
       'browser/ui/views/bookmarks/bookmark_menu_controller_views.cc',
       'browser/ui/views/bookmarks/bookmark_menu_controller_views.h',
       'browser/ui/views/bookmarks/bookmark_menu_delegate.cc',
       'browser/ui/views/bookmarks/bookmark_menu_delegate.h',
       '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_browser_main_extra_parts_views.cc',
-      'browser/ui/views/chrome_browser_main_extra_parts_views.h',
       'browser/ui/views/chrome_views_delegate.cc',
       'browser/ui/views/chrome_views_delegate_chromeos.cc',
-      'browser/ui/views/chrome_views_delegate.h',
       'browser/ui/views/chrome_web_dialog_view.cc',
       'browser/ui/views/collected_cookies_views.cc',
       'browser/ui/views/collected_cookies_views.h',
       'browser/ui/views/color_chooser_aura.cc',
       'browser/ui/views/color_chooser_aura.h',
       'browser/ui/views/color_chooser_win.cc',
       'browser/ui/views/confirm_bubble_views.cc',
       'browser/ui/views/confirm_bubble_views.h',
       'browser/ui/views/conflicting_module_view_win.cc',
       'browser/ui/views/conflicting_module_view_win.h',
@@ skipping 325 matching lines @@
       'browser/ui/views/validation_message_bubble_view.cc',
       'browser/ui/views/web_contents_modal_dialog_manager_views.cc',
       'browser/ui/views/website_settings/permissions_bubble_view.cc',
       'browser/ui/views/website_settings/permissions_bubble_view.h',
       'browser/ui/views/website_settings/permission_selector_view.cc',
       'browser/ui/views/website_settings/permission_selector_view.h',
       'browser/ui/views/website_settings/permission_selector_view_observer.h',
       'browser/ui/views/website_settings/website_settings_popup_view.cc',
       'browser/ui/views/website_settings/website_settings_popup_view.h',
     ],
+    # Cross-platform views sources also ready for toolkit-views on Mac.
+    'chrome_browser_ui_views_sources': [
+      '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/accessibility/accessibility_event_router_views.cc',
+      'browser/ui/views/accessibility/accessibility_event_router_views.h',
+      'browser/ui/views/chrome_browser_main_extra_parts_views.cc',
+      'browser/ui/views/chrome_browser_main_extra_parts_views.h',
+      'browser/ui/views/chrome_views_delegate.cc',
+      'browser/ui/views/chrome_views_delegate.h',
+    ],
     # Views files for everwhere but ChromeOS.
     'chrome_browser_ui_views_non_chromeos_sources': [
       'browser/ui/views/chrome_views_delegate_aura.cc',
       'browser/ui/views/external_protocol_dialog.cc',
       'browser/ui/views/external_protocol_dialog.h',
       'browser/ui/views/frame/desktop_browser_frame_aura.cc',
       'browser/ui/views/frame/desktop_browser_frame_aura.h',
       'browser/ui/views/frame/opaque_browser_frame_view.cc',
       'browser/ui/views/frame/opaque_browser_frame_view.h',
       'browser/ui/views/message_center/message_center_frame_view.cc',
@@ skipping 346 matching lines @@
           ],
         }],
         ['enable_printing!=0', {
           'dependencies': [
             '../printing/printing.gyp:printing',
           ],
         }],
         ['enable_one_click_signin==1', {
           'sources': [ '<@(chrome_browser_ui_one_click_signin_sources)' ]
         }],
+        ['enable_task_manager==1 and OS!="mac"', {
+          'sources': [ '<@(chrome_browser_ui_task_manager_non_mac_sources)' ],
+        }],
         ['enable_task_manager==1', {
-          'sources': [ '<@(chrome_browser_ui_task_manager_sources)' ],
+          'sources': [ '<@(chrome_browser_ui_task_manager_sources' ],

[Andre, 2014/07/18 20:08:57]

Accidentally deleted the closing parenthesis?

[tapted, 2014/07/23 03:20:31]

Done - good catch (shakes fist@ gyp). It's possible the gyp-parser didn't even
notice this :/

         }],
         ['disable_nacl==0', {
           'sources': [ '<@(chrome_browser_ui_nacl_sources)' ],
           'dependencies': [
             '../native_client/src/trusted/service_runtime/service_runtime.gyp:sel',
           ],
         }],
         ['debug_devtools==1', {
           'defines': [
             'DEBUG_DEVTOOLS=1',
@@ skipping 49 matching lines @@
             ['chromeos == 0', {
               '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)' ],
           'conditions': [
-            ['chromeos == 0', {
+            ['chromeos == 0 and OS!="mac"', {
               '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)' ],
+            }],
           ],
         }],
         ['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 95 matching lines @@
               'outputs': [
                 '<(INTERMEDIATE_DIR)/ui_localizer_table.h',
               ],
               'action': ['<(xib_localizer_tool_path)',
                           '<@(_outputs)',
                           '<@(mac_translated_xibs)'],
             },
           ],
         }, {  # non-Mac.
           'sources': [ '<@(chrome_browser_ui_non_mac_sources)' ],
+          'sources/': [
+            ['exclude', '^browser/ui/cocoa/'],
+          ],
         }],
         ['OS=="win"', {
           'include_dirs': [
             '<(DEPTH)/third_party/wtl/include',
           ],
           'dependencies': [
             'installer_util_strings',
             'launcher_support',
             'metro_utils',
             '../google_update/google_update.gyp:google_update',
@@ skipping 10 matching lines @@
           'sources': [ '<@(chrome_browser_ui_win_sources)' ],
           'conditions': [
             ['win_use_allocator_shim==1', {
               'dependencies': [
                 '<(allocator_target)',
               ],
             }],
           ],
         }, {  # 'OS!="win"
           'conditions': [
-            ['use_aura==1', {
+            ['toolkit_views==1', {
               'dependencies': [
                 '../ui/views/controls/webview/webview.gyp:webview',
                 '../ui/views/views.gyp:views',
               ],
               'include_dirs': [
                 '<(INTERMEDIATE_DIR)',
                 '<(INTERMEDIATE_DIR)/chrome',
               ],
-              'sources/': [
-                ['exclude', '^browser/ui/cocoa/*'],
-              ],
-            }, { # else: use_aura==0
+            }, { # else: toolkit_views==0
               'sources/': [
                 # Exclude all of views.
                 ['exclude', '^browser/ui/views/'],
               ]
             }],
           ],
         }],
         ['desktop_linux==1', {
           'sources': [ '<@(chrome_browser_desktop_linux_sources)' ],
         }],
@@ skipping 55 matching lines @@
         }],
         ['enable_spellcheck==1', {
           'dependencies': [
             '../third_party/hunspell/hunspell.gyp:hunspell',
           ],
         }],
       ],
     },
   ],
 }