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

Unified Diff: chrome/browser/sync/profile_sync_factory_impl.cc

Issue 8375047: Separate the syncing of extension settings and app settings into separate data (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 9 years, 2 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/sync/profile_sync_factory_impl.cc
diff --git a/chrome/browser/sync/profile_sync_factory_impl.cc b/chrome/browser/sync/profile_sync_factory_impl.cc
index 007acbbe74d15f400fbbbd7194b40bc5ac5789bf..169eb0e8e2c3eff8139774a685dc2dde21d5a916 100644
--- a/chrome/browser/sync/profile_sync_factory_impl.cc
+++ b/chrome/browser/sync/profile_sync_factory_impl.cc
@@ -166,7 +166,11 @@ void ProfileSyncFactoryImpl::RegisterDataTypes(ProfileSyncService* pss) {
// explicitly enabled.
if (command_line_->HasSwitch(switches::kEnableSyncExtensionSettings)) {
pss->RegisterDataTypeController(
- new ExtensionSettingDataTypeController(this, profile_, pss));
+ new ExtensionSettingDataTypeController(
+ syncable::EXTENSION_SETTINGS, this, profile_, pss));
+ pss->RegisterDataTypeController(
+ new ExtensionSettingDataTypeController(
+ syncable::APP_SETTINGS, this, profile_, pss));
}
if (!command_line_->HasSwitch(switches::kDisableSyncAutofillProfile)) {
@@ -269,19 +273,22 @@ ProfileSyncFactoryImpl::CreateBookmarkSyncComponents(
}
ProfileSyncFactory::SyncComponents
-ProfileSyncFactoryImpl::CreateExtensionSettingSyncComponents(
- ExtensionSettingsBackend* extension_settings_backend,
+ProfileSyncFactoryImpl::CreateExtensionOrAppSettingSyncComponents(
+ syncable::ModelType type,
+ SyncableService* settings_service,
ProfileSyncService* profile_sync_service,
UnrecoverableErrorHandler* error_handler) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
+ DCHECK(type == syncable::EXTENSION_SETTINGS ||
+ type == syncable::APP_SETTINGS);
sync_api::UserShare* user_share = profile_sync_service->GetUserShare();
GenericChangeProcessor* change_processor =
new GenericChangeProcessor(error_handler,
- extension_settings_backend->AsWeakPtr(),
+ settings_service->AsWeakPtr(),
user_share);
browser_sync::SyncableServiceAdapter* sync_service_adapter =
- new browser_sync::SyncableServiceAdapter(syncable::EXTENSION_SETTINGS,
- extension_settings_backend,
+ new browser_sync::SyncableServiceAdapter(type,
+ settings_service,
change_processor);
return SyncComponents(sync_service_adapter, change_processor);
}

Powered by Google App Engine
This is Rietveld 408576698