Commit bf18dfa8 authored by stuartmorgan's avatar stuartmorgan Committed by Commit bot

Enable chrome.gyp:browser_ui on iOS

Re-enabled building of the browser_ui target on iOS (lost when
unit_tests was turned down). As with browser, removes non_android
sources and puts everything in non_mobile_sources.

See also https://codereview.chromium.org/785233002

BUG=432994

Review URL: https://codereview.chromium.org/801643006

Cr-Commit-Position: refs/heads/master@{#308448}
parent 80093bd0
...@@ -44,6 +44,7 @@ ...@@ -44,6 +44,7 @@
['OS=="ios"', { ['OS=="ios"', {
'dependencies': [ 'dependencies': [
'../chrome/chrome.gyp:browser', '../chrome/chrome.gyp:browser',
'../chrome/chrome.gyp:browser_ui',
'../ios/ios.gyp:*', '../ios/ios.gyp:*',
# NOTE: This list of targets is present because # NOTE: This list of targets is present because
# mojo_base.gyp:mojo_base cannot be built on iOS, as # mojo_base.gyp:mojo_base cannot be built on iOS, as
......
...@@ -216,7 +216,7 @@ static_library("ui") { ...@@ -216,7 +216,7 @@ static_library("ui") {
} else { # Not athena. } else { # Not athena.
if (!is_android) { if (!is_android) {
sources += rebase_path( sources += rebase_path(
gypi_values.chrome_browser_ui_non_athena_non_android_sources, gypi_values.chrome_browser_ui_non_athena_non_mobile_sources,
".", ".",
"//chrome") "//chrome")
} }
...@@ -327,10 +327,6 @@ static_library("ui") { ...@@ -327,10 +327,6 @@ static_library("ui") {
"//chrome") "//chrome")
defines += [ "CHROME_BUILD_ID=" + android_chrome_build_id ] defines += [ "CHROME_BUILD_ID=" + android_chrome_build_id ]
} else { # Non-Android.
sources += rebase_path(gypi_values.chrome_browser_ui_non_android_sources,
".",
"//chrome")
} }
if (is_mac) { if (is_mac) {
......
...@@ -1345,7 +1345,7 @@ ...@@ -1345,7 +1345,7 @@
'browser/ui/auto_login_infobar_delegate.h', 'browser/ui/auto_login_infobar_delegate.h',
'browser/ui/screen_capture_notification_ui_stub.cc', 'browser/ui/screen_capture_notification_ui_stub.cc',
], ],
'chrome_browser_ui_non_android_sources': [ 'chrome_browser_ui_non_mobile_sources': [
'browser/ui/apps/app_info_dialog.h', 'browser/ui/apps/app_info_dialog.h',
'browser/ui/apps/apps_metro_handler_win.cc', 'browser/ui/apps/apps_metro_handler_win.cc',
'browser/ui/apps/apps_metro_handler_win.h', 'browser/ui/apps/apps_metro_handler_win.h',
...@@ -1379,6 +1379,8 @@ ...@@ -1379,6 +1379,8 @@
'browser/ui/browser.h', 'browser/ui/browser.h',
'browser/ui/browser_command_controller.cc', 'browser/ui/browser_command_controller.cc',
'browser/ui/browser_command_controller.h', 'browser/ui/browser_command_controller.h',
'browser/ui/browser_commands.cc',
'browser/ui/browser_commands.h',
'browser/ui/browser_content_setting_bubble_model_delegate.cc', 'browser/ui/browser_content_setting_bubble_model_delegate.cc',
'browser/ui/browser_content_setting_bubble_model_delegate.h', 'browser/ui/browser_content_setting_bubble_model_delegate.h',
'browser/ui/browser_finder.cc', 'browser/ui/browser_finder.cc',
...@@ -1391,6 +1393,7 @@ ...@@ -1391,6 +1393,7 @@
'browser/ui/browser_otr_state.h', 'browser/ui/browser_otr_state.h',
'browser/ui/browser_tab_restore_service_delegate.cc', 'browser/ui/browser_tab_restore_service_delegate.cc',
'browser/ui/browser_tab_restore_service_delegate.h', 'browser/ui/browser_tab_restore_service_delegate.h',
'browser/ui/browser_tab_restorer.cc',
'browser/ui/browser_tab_strip_model_delegate.cc', 'browser/ui/browser_tab_strip_model_delegate.cc',
'browser/ui/browser_tab_strip_model_delegate.h', 'browser/ui/browser_tab_strip_model_delegate.h',
'browser/ui/browser_tabrestore.cc', 'browser/ui/browser_tabrestore.cc',
...@@ -1401,6 +1404,8 @@ ...@@ -1401,6 +1404,8 @@
'browser/ui/browser_toolbar_model_delegate.h', 'browser/ui/browser_toolbar_model_delegate.h',
'browser/ui/browser_view_prefs.cc', 'browser/ui/browser_view_prefs.cc',
'browser/ui/browser_view_prefs.h', 'browser/ui/browser_view_prefs.h',
'browser/ui/browser_window_state.cc',
'browser/ui/browser_window_state.h',
'browser/ui/chrome_pages.cc', 'browser/ui/chrome_pages.cc',
'browser/ui/chrome_pages.h', 'browser/ui/chrome_pages.h',
'browser/ui/chrome_style.cc', 'browser/ui/chrome_style.cc',
...@@ -1504,6 +1509,8 @@ ...@@ -1504,6 +1509,8 @@
'browser/ui/sad_tab_helper.cc', 'browser/ui/sad_tab_helper.cc',
'browser/ui/sad_tab_helper.h', 'browser/ui/sad_tab_helper.h',
'browser/ui/sad_tab_types.h', 'browser/ui/sad_tab_types.h',
'browser/ui/scoped_tabbed_browser_displayer.cc',
'browser/ui/scoped_tabbed_browser_displayer.h',
'browser/ui/search/search_delegate.cc', 'browser/ui/search/search_delegate.cc',
'browser/ui/search/search_delegate.h', 'browser/ui/search/search_delegate.h',
'browser/ui/search/search_ui.cc', 'browser/ui/search/search_ui.cc',
...@@ -1534,6 +1541,8 @@ ...@@ -1534,6 +1541,8 @@
'browser/ui/startup/startup_types.h', 'browser/ui/startup/startup_types.h',
'browser/ui/sync/browser_synced_window_delegate.cc', 'browser/ui/sync/browser_synced_window_delegate.cc',
'browser/ui/sync/browser_synced_window_delegate.h', 'browser/ui/sync/browser_synced_window_delegate.h',
'browser/ui/sync/sync_promo_ui.cc',
'browser/ui/sync/sync_promo_ui.h',
'browser/ui/tab_contents/tab_contents_iterator.cc', 'browser/ui/tab_contents/tab_contents_iterator.cc',
'browser/ui/tab_contents/tab_contents_iterator.h', 'browser/ui/tab_contents/tab_contents_iterator.h',
'browser/ui/tab_modal_confirm_dialog_delegate.cc', 'browser/ui/tab_modal_confirm_dialog_delegate.cc',
...@@ -1569,15 +1578,15 @@ ...@@ -1569,15 +1578,15 @@
'browser/ui/toolbar/encoding_menu_controller.h', 'browser/ui/toolbar/encoding_menu_controller.h',
'browser/ui/toolbar/recent_tabs_sub_menu_model.cc', 'browser/ui/toolbar/recent_tabs_sub_menu_model.cc',
'browser/ui/toolbar/recent_tabs_sub_menu_model.h', 'browser/ui/toolbar/recent_tabs_sub_menu_model.h',
'browser/ui/toolbar/toolbar_action_view_controller.h',
'browser/ui/toolbar/toolbar_action_view_delegate.h',
'browser/ui/toolbar/toolbar_actions_bar.cc', 'browser/ui/toolbar/toolbar_actions_bar.cc',
'browser/ui/toolbar/toolbar_actions_bar.h', 'browser/ui/toolbar/toolbar_actions_bar.h',
'browser/ui/toolbar/toolbar_actions_bar_delegate.h', 'browser/ui/toolbar/toolbar_actions_bar_delegate.h',
'browser/ui/toolbar/toolbar_action_view_controller.h',
'browser/ui/toolbar/toolbar_action_view_delegate.h',
'browser/ui/toolbar/wrench_menu_badge_controller.cc',
'browser/ui/toolbar/wrench_menu_badge_controller.h',
'browser/ui/toolbar/wrench_icon_painter.cc', 'browser/ui/toolbar/wrench_icon_painter.cc',
'browser/ui/toolbar/wrench_icon_painter.h', 'browser/ui/toolbar/wrench_icon_painter.h',
'browser/ui/toolbar/wrench_menu_badge_controller.cc',
'browser/ui/toolbar/wrench_menu_badge_controller.h',
'browser/ui/toolbar/wrench_menu_model.cc', 'browser/ui/toolbar/wrench_menu_model.cc',
'browser/ui/toolbar/wrench_menu_model.h', 'browser/ui/toolbar/wrench_menu_model.h',
'browser/ui/uma_browsing_activity_observer.cc', 'browser/ui/uma_browsing_activity_observer.cc',
...@@ -1594,6 +1603,8 @@ ...@@ -1594,6 +1603,8 @@
'browser/ui/webui/bookmarks_ui.h', 'browser/ui/webui/bookmarks_ui.h',
'browser/ui/webui/chrome_web_contents_handler.cc', 'browser/ui/webui/chrome_web_contents_handler.cc',
'browser/ui/webui/chrome_web_contents_handler.h', 'browser/ui/webui/chrome_web_contents_handler.h',
'browser/ui/webui/devtools_ui.cc',
'browser/ui/webui/devtools_ui.h',
'browser/ui/webui/downloads_dom_handler.cc', 'browser/ui/webui/downloads_dom_handler.cc',
'browser/ui/webui/downloads_dom_handler.h', 'browser/ui/webui/downloads_dom_handler.h',
'browser/ui/webui/downloads_ui.cc', 'browser/ui/webui/downloads_ui.cc',
...@@ -1807,19 +1818,6 @@ ...@@ -1807,19 +1818,6 @@
'browser/ui/window_sizer/window_sizer.cc', 'browser/ui/window_sizer/window_sizer.cc',
'browser/ui/window_sizer/window_sizer.h', 'browser/ui/window_sizer/window_sizer.h',
], ],
'chrome_browser_ui_non_mobile_sources': [
'browser/ui/browser_commands.cc',
'browser/ui/browser_commands.h',
'browser/ui/browser_tab_restorer.cc',
'browser/ui/browser_window_state.cc',
'browser/ui/browser_window_state.h',
'browser/ui/scoped_tabbed_browser_displayer.cc',
'browser/ui/scoped_tabbed_browser_displayer.h',
'browser/ui/sync/sync_promo_ui.cc',
'browser/ui/sync/sync_promo_ui.h',
'browser/ui/webui/devtools_ui.cc',
'browser/ui/webui/devtools_ui.h',
],
'chrome_browser_ui_print_preview_sources': [ 'chrome_browser_ui_print_preview_sources': [
'browser/ui/webui/print_preview/print_preview_handler.cc', 'browser/ui/webui/print_preview/print_preview_handler.cc',
'browser/ui/webui/print_preview/print_preview_handler.h', 'browser/ui/webui/print_preview/print_preview_handler.h',
...@@ -2500,8 +2498,8 @@ ...@@ -2500,8 +2498,8 @@
'browser/ui/ash/session_state_delegate_views.cc', 'browser/ui/ash/session_state_delegate_views.cc',
'browser/ui/ash/session_state_delegate_views.h', 'browser/ui/ash/session_state_delegate_views.h',
], ],
# Used when not using android and not using Athena. # Used when not using mobile and not using Athena.
'chrome_browser_ui_non_athena_non_android_sources': [ 'chrome_browser_ui_non_athena_non_mobile_sources': [
'browser/ui/extensions/application_launch_web_app.cc', 'browser/ui/extensions/application_launch_web_app.cc',
'browser/ui/extensions/extension_install_ui_factory.cc', 'browser/ui/extensions/extension_install_ui_factory.cc',
'browser/ui/webui/chromeos/keyboard_overlay_ui.cc', 'browser/ui/webui/chromeos/keyboard_overlay_ui.cc',
...@@ -2720,7 +2718,13 @@ ...@@ -2720,7 +2718,13 @@
], ],
}, },
}, { # OS == "ios" }, { # OS == "ios"
'sources': [ '<@(chrome_browser_ui_ios_sources)' ], 'sources': [
'<@(chrome_browser_ui_ios_sources)'
],
'sources!': [
'browser/ui/external_protocol_dialog_delegate.cc',
'browser/ui/startup/default_browser_prompt.cc',
],
'dependencies': [ 'dependencies': [
'../net/net.gyp:net', '../net/net.gyp:net',
], ],
...@@ -2813,8 +2817,8 @@ ...@@ -2813,8 +2817,8 @@
], ],
}, { # use_athena==0 }, { # use_athena==0
'conditions': [ 'conditions': [
['OS!="android"', { ['OS!="android" and OS!="ios"', {
'sources': [ '<@(chrome_browser_ui_non_athena_non_android_sources)' ], 'sources': [ '<@(chrome_browser_ui_non_athena_non_mobile_sources)' ],
}], }],
['use_ash==1', { ['use_ash==1', {
'sources': [ '<@(chrome_browser_ui_ash_non_athena_sources)' ], 'sources': [ '<@(chrome_browser_ui_ash_non_athena_sources)' ],
...@@ -2910,8 +2914,6 @@ ...@@ -2910,8 +2914,6 @@
'chrome_browser_ui_views.gyp:browser_ui_views', 'chrome_browser_ui_views.gyp:browser_ui_views',
], ],
'sources': [ '<@(chrome_browser_ui_android_sources)' ], 'sources': [ '<@(chrome_browser_ui_android_sources)' ],
}, { # Non-Android
'sources': [ '<@(chrome_browser_ui_non_android_sources)' ],
}], }],
['OS=="mac"', { ['OS=="mac"', {
'sources': [ '<@(chrome_browser_ui_mac_sources)' ], 'sources': [ '<@(chrome_browser_ui_mac_sources)' ],
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment