diff options
| author | stevenjb@chromium.org <stevenjb@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-11-19 18:10:49 +0000 |
|---|---|---|
| committer | stevenjb@chromium.org <stevenjb@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-11-19 18:10:49 +0000 |
| commit | c2a7429976443bbca712e9e23012f32416a980de (patch) | |
| tree | dd3bd8c6bef622f300638470439ecf8e7959a209 /chrome/chrome_tests.gypi | |
| parent | b645bd4b8f85f9a059b42bd4f85f37ddd987bc02 (diff) | |
| download | chromium_src-c2a7429976443bbca712e9e23012f32416a980de.zip chromium_src-c2a7429976443bbca712e9e23012f32416a980de.tar.gz chromium_src-c2a7429976443bbca712e9e23012f32416a980de.tar.bz2 | |
Introduce AppListSyncableService, owns AppListModel and builder
This introduces AppListSyncableService, but does not implement any
syncing yet. The purpose is to move ownership of AppListModel so that
we can maintain and sync the model state independent of the view. It also
made sense to move ExtensionAppModelBuilder ownership to
AppListSyncableService.
This required a fair bit of (smallish) re-factoring. Here is a simplified
diagram that may help:
https://docs.google.com/a/google.com/drawings/d/1NgaQzdqddTh1W7altZ6P6t4dNxZDRO0e_HgJqa8V36A/edit
BUG=315887
For minor ash/shell/ changes:
R=jennyz@chromium.org, tapted@chromium.org, xiyuan@chromium.org
TBR=jamescook@chromium.org
Review URL: https://codereview.chromium.org/66023003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@236020 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/chrome_tests.gypi')
| -rw-r--r-- | chrome/chrome_tests.gypi | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/chrome/chrome_tests.gypi b/chrome/chrome_tests.gypi index 1ee78dc..b8cd040 100644 --- a/chrome/chrome_tests.gypi +++ b/chrome/chrome_tests.gypi @@ -1396,10 +1396,6 @@ 'browser/ui/app_list/app_list_service_mac_browsertest.mm', 'browser/ui/app_list/search/people/people_provider_browsertest.cc', 'browser/ui/app_list/search/webstore/webstore_provider_browsertest.cc', - 'browser/ui/app_list/test/app_list_service_test_api.h', - 'browser/ui/app_list/test/app_list_service_test_api_ash.cc', - 'browser/ui/app_list/test/app_list_service_test_api_ash.h', - 'browser/ui/app_list/test/app_list_service_test_api_mac.mm', 'browser/ui/ash/accelerator_commands_browsertest.cc', 'browser/ui/ash/caps_lock_delegate_chromeos_browsertest.cc', 'browser/ui/ash/launcher/chrome_launcher_controller_browsertest.cc', @@ -1462,8 +1458,6 @@ 'browser/ui/tab_modal_confirm_dialog_browsertest.h', 'browser/ui/toolbar/test_toolbar_model.cc', 'browser/ui/toolbar/test_toolbar_model.h', - 'browser/ui/views/app_list/linux/test/app_list_service_test_api_linux.cc', - 'browser/ui/views/app_list/win/test/app_list_service_test_api_win.cc', 'browser/ui/views/avatar_menu_button_browsertest.cc', 'browser/ui/views/frame/app_non_client_frame_view_ash_browsertest.cc', 'browser/ui/views/frame/browser_non_client_frame_view_ash_browsertest.cc', @@ -1754,7 +1748,6 @@ 'browser/signin/signin_browsertest.cc', # chromeos does not use cross-platform panels 'browser/ui/panels/panel_extension_browsertest.cc', - 'browser/ui/views/app_list/linux/test/app_list_service_test_api_linux.cc', # chromeos does not use the desktop user manager 'browser/ui/webui/signin/user_manager_ui_browsertest.cc', ], |