Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1477)

Unified Diff: chrome/chrome_browser_ui.gypi

Issue 1861233003: Prepare for building with enable_app_list=0 on Desktop (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase for r392580 - app_list_shower_views_unittest deleted by that rev Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/chrome_browser_extensions.gypi ('k') | chrome/chrome_tests.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/chrome_browser_ui.gypi
diff --git a/chrome/chrome_browser_ui.gypi b/chrome/chrome_browser_ui.gypi
index 4474db803ccf21f4a251b1fce68f2f9bfa90dbd5..3fc02dcf60263c979ff60ad1a8086f8688a3cad9 100644
--- a/chrome/chrome_browser_ui.gypi
+++ b/chrome/chrome_browser_ui.gypi
@@ -1342,10 +1342,6 @@
'browser/ui/input_method/input_method_engine.h',
'browser/ui/input_method/input_method_engine_base.cc',
'browser/ui/input_method/input_method_engine_base.h',
- 'browser/ui/views/app_list/linux/app_list_linux.cc',
- 'browser/ui/views/app_list/linux/app_list_linux.h',
- 'browser/ui/views/app_list/linux/app_list_service_linux.cc',
- 'browser/ui/views/app_list/linux/app_list_service_linux.h',
'browser/ui/views/apps/chrome_app_window_client_views_linux.cc',
'browser/ui/views/first_run_dialog.cc',
'browser/ui/views/first_run_dialog.h',
@@ -2050,6 +2046,8 @@
'browser/ui/autofill/save_card_bubble_controller_impl.h',
'browser/ui/autofill/save_card_bubble_view.h',
'browser/ui/browser_dialogs_mac.cc',
+ 'browser/ui/views/apps/app_info_dialog/app_info_dialog_container.cc',
+ 'browser/ui/views/apps/app_info_dialog/app_info_dialog_container.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',
@@ -2132,7 +2130,7 @@
'browser/ui/views/sync/one_click_signin_dialog_view.cc',
'browser/ui/views/sync/one_click_signin_dialog_view.h',
],
- # Cross-platform (except Mac) views sources. We assume app list is enabled.
+ # Cross-platform (except Mac) views sources.
'chrome_browser_ui_views_non_mac_sources': [
'browser/ui/views/accelerator_utils_aura.cc',
'browser/ui/views/accessibility/invert_bubble_view.cc',
@@ -2545,9 +2543,6 @@
### Feature-based sources ###
# Variables should generally be alphabetical within this section.
-
- # Note that we assume app list is enabled on all views builds, so the
- # views-specific app list files are in the views section.
'chrome_browser_ui_app_list_sources': [
'browser/ui/app_list/app_context_menu.cc',
'browser/ui/app_list/app_context_menu.h',
@@ -2650,8 +2645,6 @@
'browser/ui/app_list/start_page_service.h',
'browser/ui/app_list/start_page_service_factory.cc',
'browser/ui/app_list/start_page_service_factory.h',
- 'browser/ui/views/app_list/app_list_dialog_container.cc',
- 'browser/ui/views/app_list/app_list_dialog_container.h',
'browser/ui/views/app_list/win/activation_tracker_win.cc',
'browser/ui/views/app_list/win/activation_tracker_win.h',
'browser/ui/views/app_list/win/app_list_controller_delegate_win.cc',
@@ -2665,6 +2658,12 @@
'browser/ui/webui/app_list/start_page_ui.cc',
'browser/ui/webui/app_list/start_page_ui.h',
],
+ 'chrome_browser_ui_app_list_linux_sources': [
+ 'browser/ui/views/app_list/linux/app_list_linux.cc',
+ 'browser/ui/views/app_list/linux/app_list_linux.h',
+ 'browser/ui/views/app_list/linux/app_list_service_linux.cc',
+ 'browser/ui/views/app_list/linux/app_list_service_linux.h',
+ ],
'chrome_browser_ui_app_list_views_sources': [
'browser/ui/app_list/app_list_controller_delegate_views.cc',
'browser/ui/app_list/app_list_controller_delegate_views.h',
@@ -3147,7 +3146,6 @@
'../google_update/google_update.gyp:google_update',
'../third_party/iaccessible2/iaccessible2.gyp:iaccessible2',
'../third_party/isimpledom/isimpledom.gyp:isimpledom',
- '../ui/app_list/app_list.gyp:app_list',
'../ui/base/ime/ui_base_ime.gyp:ui_base_ime',
'../ui/events/events.gyp:dom_keycode_converter',
'../ui/views/controls/webview/webview.gyp:webview',
@@ -3175,6 +3173,11 @@
'browser/ui/webui/help/version_updater_basic.h',
],
}],
+ ['enable_app_list==1', {
+ 'dependencies!': [
+ '../ui/app_list/app_list.gyp:app_list',
+ ],
+ }],
],
}, { # 'OS!="win"
'conditions': [
@@ -3236,6 +3239,11 @@
'../ui/app_list/app_list.gyp:app_list',
],
'conditions': [
+ ['desktop_linux==1', {
+ 'sources': [
+ '<@(chrome_browser_ui_app_list_linux_sources)',
+ ],
+ }],
['OS!="mac"', {
'sources': [
'<@(chrome_browser_ui_app_list_views_sources)',
« no previous file with comments | « chrome/chrome_browser_extensions.gypi ('k') | chrome/chrome_tests.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698