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

Unified Diff: chrome/chrome_browser_ui.gypi

Issue 12207104: Refactor app_list_util.h into AppListService abstract base. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix some build problems maybe Created 7 years, 10 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
Index: chrome/chrome_browser_ui.gypi
diff --git a/chrome/chrome_browser_ui.gypi b/chrome/chrome_browser_ui.gypi
index 9623381aa5d620ac311ee44814b6d0a020a63619..2d2ca4e01116b29df9002fe203e2b81760115b50 100644
--- a/chrome/chrome_browser_ui.gypi
+++ b/chrome/chrome_browser_ui.gypi
@@ -100,8 +100,13 @@
'browser/ui/android/window_android_helper.h',
'browser/ui/app_list/app_list_controller_delegate.cc',
'browser/ui/app_list/app_list_controller_delegate.h',
- 'browser/ui/app_list/app_list_util.cc',
- 'browser/ui/app_list/app_list_util.h',
+ 'browser/ui/app_list/app_list_service.cc',
+ 'browser/ui/app_list/app_list_service.h',
+ 'browser/ui/app_list/app_list_service_ash.h',
+ 'browser/ui/app_list/app_list_service_cocoa.h',
+ 'browser/ui/app_list/app_list_service_disabled.cc',
benwells 2013/02/19 03:49:42 You don't really need the _disabled.cc here, which
tapted 2013/02/19 04:25:21 There's a "buut" on line 66 ;)
+ 'browser/ui/app_list/app_list_service_selector.cc',
+ 'browser/ui/app_list/app_list_service_win.h',
'browser/ui/app_list/app_list_view_delegate.cc',
'browser/ui/app_list/app_list_view_delegate.h',
'browser/ui/app_list/apps_model_builder.cc',
@@ -127,6 +132,7 @@
'browser/ui/ash/app_icon_loader_impl.h',
'browser/ui/ash/app_list/app_list_controller_ash.h',
'browser/ui/ash/app_list/app_list_controller_ash.cc',
+ 'browser/ui/ash/app_list/app_list_service_ash.cc',
'browser/ui/ash/app_list/app_sync_ui_state_watcher.cc',
'browser/ui/ash/app_list/app_sync_ui_state_watcher.h',
'browser/ui/ash/app_sync_ui_state.cc',
@@ -2818,10 +2824,16 @@
'dependencies': [
'../ui/app_list/app_list.gyp:app_list',
],
+ 'sources!': [
+ 'browser/ui/app_list/app_list_service_disabled.cc',
+ ]
}, { # else: enable_app_list==0
'sources/': [
['exclude', '^browser/ui/views/app_list/'],
['exclude', '^browser/ui/app_list/'],
+ ['include', '^browser/ui/app_list/app_list_service.h'],
+ ['include', '^browser/ui/app_list/app_list_service.cc'],
+ ['include', '^browser/ui/app_list/app_list_service_disabled.cc'],
]
}],
['enable_message_center==1', {

Powered by Google App Engine
This is Rietveld 408576698