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

Unified Diff: chrome/browser/extensions/api/storage/settings_apitest.cc

Issue 184043031: Split up extensions storage API implementations for sync and local storage. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review comments (split_sync) Created 6 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/browser/extensions/api/storage/settings_apitest.cc
diff --git a/chrome/browser/extensions/api/storage/settings_apitest.cc b/chrome/browser/extensions/api/storage/settings_apitest.cc
index 98fa1a5143dc463fa8a6b58692e9c0171aba1bf2..8de1b3044651a086103401f7fcea085bd2fcd2fb 100644
--- a/chrome/browser/extensions/api/storage/settings_apitest.cc
+++ b/chrome/browser/extensions/api/storage/settings_apitest.cc
@@ -114,18 +114,19 @@ class ExtensionSettingsApiTest : public ExtensionApiTest {
settings_namespace, normal_action, incognito_action, NULL, true);
}
+ syncer::SyncableService* GetSyncableService() {
+ return settings_sync_util::GetSyncableService(browser()->profile(),
+ kModelType);
+ }
+
void InitSync(syncer::SyncChangeProcessor* sync_processor) {
base::MessageLoop::current()->RunUntilIdle();
- SettingsFrontend* frontend = SettingsFrontend::Get(browser()->profile());
- InitSyncWithSyncableService(sync_processor,
- frontend->GetBackendForSync(kModelType));
+ InitSyncWithSyncableService(sync_processor, GetSyncableService());
}
void SendChanges(const syncer::SyncChangeList& change_list) {
base::MessageLoop::current()->RunUntilIdle();
- SettingsFrontend* frontend = SettingsFrontend::Get(browser()->profile());
- SendChangesToSyncableService(change_list,
- frontend->GetBackendForSync(kModelType));
+ SendChangesToSyncableService(change_list, GetSyncableService());
}
#if defined(ENABLE_CONFIGURATION_POLICY)

Powered by Google App Engine
This is Rietveld 408576698