[Sync] Remove SyncableServiceAdapter.

chrome/browser/sync/profile_sync_components_factory_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/command_line.h"
 #include "build/build_config.h"
 #include "chrome/browser/extensions/app_notification_manager.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/settings/settings_frontend.h"
 #include "chrome/browser/prefs/pref_model_associator.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/search_engines/template_url_service.h"
 #include "chrome/browser/search_engines/template_url_service_factory.h"
 #include "chrome/browser/signin/signin_manager.h"
 #include "chrome/browser/signin/signin_manager_factory.h"
 #include "chrome/browser/sync/api/syncable_service.h"
 #include "chrome/browser/sync/glue/app_notification_data_type_controller.h"
 #include "chrome/browser/sync/glue/autofill_data_type_controller.h"
 #include "chrome/browser/sync/glue/autofill_profile_data_type_controller.h"
 #include "chrome/browser/sync/glue/bookmark_change_processor.h"
 #include "chrome/browser/sync/glue/bookmark_data_type_controller.h"
 #include "chrome/browser/sync/glue/bookmark_model_associator.h"
 #include "chrome/browser/sync/glue/data_type_manager_impl.h"
 #include "chrome/browser/sync/glue/extension_data_type_controller.h"
 #include "chrome/browser/sync/glue/extension_setting_data_type_controller.h"
 #include "chrome/browser/sync/glue/generic_change_processor.h"
 #include "chrome/browser/sync/glue/password_change_processor.h"
 #include "chrome/browser/sync/glue/password_data_type_controller.h"
 #include "chrome/browser/sync/glue/password_model_associator.h"
-#include "chrome/browser/sync/glue/preference_data_type_controller.h"
 #include "chrome/browser/sync/glue/search_engine_data_type_controller.h"
 #include "chrome/browser/sync/glue/session_change_processor.h"
 #include "chrome/browser/sync/glue/session_data_type_controller.h"
 #include "chrome/browser/sync/glue/session_model_associator.h"
 #include "chrome/browser/sync/glue/shared_change_processor.h"
 #include "chrome/browser/sync/glue/sync_backend_host.h"
-#include "chrome/browser/sync/glue/syncable_service_adapter.h"
 #include "chrome/browser/sync/glue/theme_change_processor.h"
 #include "chrome/browser/sync/glue/theme_data_type_controller.h"
 #include "chrome/browser/sync/glue/theme_model_associator.h"
 #include "chrome/browser/sync/glue/typed_url_change_processor.h"
 #include "chrome/browser/sync/glue/typed_url_data_type_controller.h"
 #include "chrome/browser/sync/glue/typed_url_model_associator.h"
+#include "chrome/browser/sync/glue/ui_data_type_controller.h"
 #include "chrome/browser/sync/profile_sync_components_factory_impl.h"
 #include "chrome/browser/sync/profile_sync_service.h"
 #include "chrome/browser/webdata/autocomplete_syncable_service.h"
 #include "chrome/browser/webdata/autofill_profile_syncable_service.h"
 #include "chrome/browser/webdata/web_data_service.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/pref_names.h"
 #include "content/public/browser/browser_thread.h"
 
 using browser_sync::AppNotificationDataTypeController;
 using browser_sync::AutofillDataTypeController;
 using browser_sync::AutofillProfileDataTypeController;
 using browser_sync::BookmarkChangeProcessor;
 using browser_sync::BookmarkDataTypeController;
 using browser_sync::BookmarkModelAssociator;
 using browser_sync::DataTypeController;
 using browser_sync::DataTypeManager;
 using browser_sync::DataTypeManagerImpl;
 using browser_sync::ExtensionDataTypeController;
 using browser_sync::ExtensionSettingDataTypeController;
 using browser_sync::GenericChangeProcessor;
 using browser_sync::PasswordChangeProcessor;
 using browser_sync::PasswordDataTypeController;
 using browser_sync::PasswordModelAssociator;
-using browser_sync::PreferenceDataTypeController;
 using browser_sync::SearchEngineDataTypeController;
 using browser_sync::SessionChangeProcessor;
 using browser_sync::SessionDataTypeController;
 using browser_sync::SessionModelAssociator;
 using browser_sync::SharedChangeProcessor;
-using browser_sync::SyncableServiceAdapter;
 using browser_sync::SyncBackendHost;
 using browser_sync::ThemeChangeProcessor;
 using browser_sync::ThemeDataTypeController;
 using browser_sync::ThemeModelAssociator;
 using browser_sync::TypedUrlChangeProcessor;
 using browser_sync::TypedUrlDataTypeController;
 using browser_sync::TypedUrlModelAssociator;
+using browser_sync::UIDataTypeController;
 using browser_sync::DataTypeErrorHandler;
 using content::BrowserThread;
 
 ProfileSyncComponentsFactoryImpl::ProfileSyncComponentsFactoryImpl(
     Profile* profile, CommandLine* command_line)
     : profile_(profile),
       command_line_(command_line) {
 }
 
 void ProfileSyncComponentsFactoryImpl::RegisterDataTypes(
@@ skipping 31 matching lines @@
   // disabled.
   if (!command_line_->HasSwitch(switches::kDisableSyncPasswords)) {
     pss->RegisterDataTypeController(
         new PasswordDataTypeController(this, profile_, pss));
   }
 
   // Preference sync is enabled by default.  Register unless explicitly
   // disabled.
   if (!command_line_->HasSwitch(switches::kDisableSyncPreferences)) {
     pss->RegisterDataTypeController(
-        new PreferenceDataTypeController(this, profile_, pss));
+        new UIDataTypeController(syncable::PREFERENCES, this, profile_, pss));
   }
 
   // Theme sync is enabled by default.  Register unless explicitly disabled.
   if (!command_line_->HasSwitch(switches::kDisableSyncThemes)) {
     pss->RegisterDataTypeController(
         new ThemeDataTypeController(this, profile_, pss));
   }
 
   // TypedUrl sync is enabled by default.  Register unless explicitly disabled,
   // or if saving history is disabled.
@@ skipping 67 matching lines @@
     CreateSharedChangeProcessor() {
   return new SharedChangeProcessor();
 }
 
 base::WeakPtr<SyncableService> ProfileSyncComponentsFactoryImpl::
     GetSyncableServiceForType(syncable::ModelType type) {
   if (!profile_) {  // For tests.
      return base::WeakPtr<SyncableService>();
   }
   switch (type) {
-    case syncable::AUTOFILL: {
-      WebDataService* wds =
-          profile_->GetWebDataService(Profile::IMPLICIT_ACCESS);
-      if (!wds)
-        return base::WeakPtr<SyncableService>();
-      return GetAutocompleteSyncableService(wds);
-    }
+    case syncable::PREFERENCES:
+      return profile_->GetPrefs()->GetSyncableService()->AsWeakPtr();
+    case syncable::AUTOFILL:
     case syncable::AUTOFILL_PROFILE: {
       WebDataService* wds =
           profile_->GetWebDataService(Profile::IMPLICIT_ACCESS);
       if (!wds)
         return base::WeakPtr<SyncableService>();
-      return GetAutofillProfileSyncableService(wds);
+      if (type == syncable::AUTOFILL)
+        return wds->GetAutocompleteSyncableService()->AsWeakPtr();
+      else
+        return wds->GetAutofillProfileSyncableService()->AsWeakPtr();
     }
+    case syncable::APPS:
+    case syncable::EXTENSIONS:
+      return profile_->GetExtensionService()->AsWeakPtr();
+    case syncable::SEARCH_ENGINES:
+      return TemplateURLServiceFactory::GetForProfile(profile_)->AsWeakPtr();
+    case syncable::APP_SETTINGS:
+    case syncable::EXTENSION_SETTINGS:
+      return profile_->GetExtensionService()->settings_frontend()->
+          GetBackendForSync(type)->AsWeakPtr();
+    case syncable::APP_NOTIFICATIONS:
+      return profile_->GetExtensionService()->app_notification_manager()->
+          AsWeakPtr();
     default:
+      // The following datatypes still need to be transitioned to the
+      // SyncableService API:
+      // Bookmarks
+      // Passwords
+      // Sessions
+      // Themes
+      // Typed URLs
       NOTREACHED();
       return base::WeakPtr<SyncableService>();
   }
 }
 
-base::WeakPtr<SyncableService>
-    ProfileSyncComponentsFactoryImpl::GetAutofillProfileSyncableService(
-        WebDataService* web_data_service) const {
-  return web_data_service->GetAutofillProfileSyncableService()->AsWeakPtr();
-}
-
-base::WeakPtr<SyncableService>
-    ProfileSyncComponentsFactoryImpl::GetAutocompleteSyncableService(
-        WebDataService* web_data_service) const {
-  return web_data_service->GetAutocompleteSyncableService()->AsWeakPtr();
-}
-
 ProfileSyncComponentsFactory::SyncComponents
     ProfileSyncComponentsFactoryImpl::CreateBookmarkSyncComponents(
         ProfileSyncService* profile_sync_service,
         DataTypeErrorHandler* error_handler) {
   BookmarkModel* bookmark_model =
       profile_sync_service->profile()->GetBookmarkModel();
   sync_api::UserShare* user_share = profile_sync_service->GetUserShare();
   // TODO(akalin): We may want to propagate this switch up eventually.
 #if defined(OS_ANDROID)
   const bool kExpectMobileBookmarksFolder = true;
 #else
   const bool kExpectMobileBookmarksFolder = false;
 #endif
   BookmarkModelAssociator* model_associator =
       new BookmarkModelAssociator(bookmark_model,
                                   user_share,
                                   error_handler,
                                   kExpectMobileBookmarksFolder);
   BookmarkChangeProcessor* change_processor =
       new BookmarkChangeProcessor(model_associator,
                                   error_handler);
   return SyncComponents(model_associator, change_processor);
 }
 
 ProfileSyncComponentsFactory::SyncComponents
-    ProfileSyncComponentsFactoryImpl::CreateExtensionOrAppSettingSyncComponents(
-        syncable::ModelType type,
-        ProfileSyncService* profile_sync_service,
-        DataTypeErrorHandler* error_handler) {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
-  DCHECK(type == syncable::EXTENSION_SETTINGS ||
-         type == syncable::APP_SETTINGS);
-  SyncableService* settings_service = profile_->GetExtensionService()->
-      settings_frontend()->GetBackendForSync(type);
-  sync_api::UserShare* user_share = profile_sync_service->GetUserShare();
-  GenericChangeProcessor* change_processor =
-      new GenericChangeProcessor(error_handler,
-                                 settings_service->AsWeakPtr(),
-                                 user_share);
-  browser_sync::SyncableServiceAdapter* sync_service_adapter =
-      new browser_sync::SyncableServiceAdapter(type,
-                                               settings_service,
-                                               change_processor);
-  return SyncComponents(sync_service_adapter, change_processor);
-}
-
-ProfileSyncComponentsFactory::SyncComponents
-    ProfileSyncComponentsFactoryImpl::CreateExtensionOrAppSyncComponents(
-        syncable::ModelType type,
-        ProfileSyncService* profile_sync_service,
-        DataTypeErrorHandler* error_handler) {
-  base::WeakPtr<SyncableService> sync_service =
-      profile_sync_service->profile()->GetExtensionService()->AsWeakPtr();
-  sync_api::UserShare* user_share = profile_sync_service->GetUserShare();
-  GenericChangeProcessor* change_processor =
-      new GenericChangeProcessor(error_handler,
-                                 sync_service,
-                                 user_share);
-  browser_sync::SyncableServiceAdapter* sync_service_adapter =
-      new browser_sync::SyncableServiceAdapter(type,
-                                               sync_service,
-                                               change_processor);
-  return SyncComponents(sync_service_adapter, change_processor);
-}
-
-ProfileSyncComponentsFactory::SyncComponents
     ProfileSyncComponentsFactoryImpl::CreatePasswordSyncComponents(
         ProfileSyncService* profile_sync_service,
         PasswordStore* password_store,
         DataTypeErrorHandler* error_handler) {
   PasswordModelAssociator* model_associator =
       new PasswordModelAssociator(profile_sync_service,
                                   password_store);
   PasswordChangeProcessor* change_processor =
       new PasswordChangeProcessor(model_associator,
                                   password_store,
                                   error_handler);
   return SyncComponents(model_associator, change_processor);
 }
 
 ProfileSyncComponentsFactory::SyncComponents
-    ProfileSyncComponentsFactoryImpl::CreatePreferenceSyncComponents(
-        ProfileSyncService* profile_sync_service,
-        DataTypeErrorHandler* error_handler) {
-  base::WeakPtr<SyncableService> pref_sync_service =
-      profile_->GetPrefs()->GetSyncableService()->AsWeakPtr();
-  sync_api::UserShare* user_share = profile_sync_service->GetUserShare();
-  GenericChangeProcessor* change_processor =
-      new GenericChangeProcessor(error_handler,
-                                 pref_sync_service,
-                                 user_share);
-  SyncableServiceAdapter* sync_service_adapter =
-      new SyncableServiceAdapter(syncable::PREFERENCES,
-                                 pref_sync_service,
-                                 change_processor);
-  return SyncComponents(sync_service_adapter, change_processor);
-}
-
-ProfileSyncComponentsFactory::SyncComponents
     ProfileSyncComponentsFactoryImpl::CreateThemeSyncComponents(
         ProfileSyncService* profile_sync_service,
         DataTypeErrorHandler* error_handler) {
   ThemeModelAssociator* model_associator =
       new ThemeModelAssociator(profile_sync_service);
   ThemeChangeProcessor* change_processor =
       new ThemeChangeProcessor(error_handler);
   return SyncComponents(model_associator, change_processor);
 }
 
@@ skipping 16 matching lines @@
 ProfileSyncComponentsFactory::SyncComponents
     ProfileSyncComponentsFactoryImpl::CreateSessionSyncComponents(
        ProfileSyncService* profile_sync_service,
         DataTypeErrorHandler* error_handler) {
   SessionModelAssociator* model_associator =
       new SessionModelAssociator(profile_sync_service);
   SessionChangeProcessor* change_processor =
       new SessionChangeProcessor(error_handler, model_associator);
   return SyncComponents(model_associator, change_processor);
 }
-
-ProfileSyncComponentsFactory::SyncComponents
-    ProfileSyncComponentsFactoryImpl::CreateSearchEngineSyncComponents(
-        ProfileSyncService* profile_sync_service,
-        DataTypeErrorHandler* error_handler) {
-  base::WeakPtr<SyncableService> se_sync_service =
-      TemplateURLServiceFactory::GetForProfile(profile_)->AsWeakPtr();
-  DCHECK(se_sync_service);
-  sync_api::UserShare* user_share = profile_sync_service->GetUserShare();
-  GenericChangeProcessor* change_processor =
-      new GenericChangeProcessor(error_handler,
-                                 se_sync_service,
-                                 user_share);
-  SyncableServiceAdapter* sync_service_adapter =
-      new SyncableServiceAdapter(syncable::SEARCH_ENGINES,
-                                 se_sync_service,
-                                 change_processor);
-  return SyncComponents(sync_service_adapter, change_processor);
-}
-
-ProfileSyncComponentsFactory::SyncComponents
-    ProfileSyncComponentsFactoryImpl::CreateAppNotificationSyncComponents(
-        ProfileSyncService* profile_sync_service,
-        browser_sync::DataTypeErrorHandler* error_handler) {
-  base::WeakPtr<SyncableService> notif_sync_service =
-      profile_->GetExtensionService()->app_notification_manager()->AsWeakPtr();
-  DCHECK(notif_sync_service);
-  sync_api::UserShare* user_share = profile_sync_service->GetUserShare();
-  GenericChangeProcessor* change_processor =
-      new GenericChangeProcessor(error_handler,
-                                 notif_sync_service,
-                                 user_share);
-  SyncableServiceAdapter* sync_service_adapter =
-      new SyncableServiceAdapter(syncable::APP_NOTIFICATIONS,
-                                 notif_sync_service,
-                                 change_processor);
-  return SyncComponents(sync_service_adapter, change_processor);
-}