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

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

Issue 9395058: [Sync] Remove SyncableServiceAdapter. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 8 years, 9 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
« no previous file with comments | « chrome/browser/sync/profile_sync_service_autofill_unittest.cc ('k') | chrome/chrome_browser.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/profile_sync_service_preference_unittest.cc
diff --git a/chrome/browser/sync/profile_sync_service_preference_unittest.cc b/chrome/browser/sync/profile_sync_service_preference_unittest.cc
index 14ee6727355ef5f873e423b64ec0812f918f8669..6e1420310ec3df5326b4cb6b49e2da9e08a7fe9d 100644
--- a/chrome/browser/sync/profile_sync_service_preference_unittest.cc
+++ b/chrome/browser/sync/profile_sync_service_preference_unittest.cc
@@ -21,8 +21,7 @@
#include "chrome/browser/sync/abstract_profile_sync_service_test.h"
#include "chrome/browser/sync/api/sync_data.h"
#include "chrome/browser/sync/glue/generic_change_processor.h"
-#include "chrome/browser/sync/glue/preference_data_type_controller.h"
-#include "chrome/browser/sync/glue/syncable_service_adapter.h"
+#include "chrome/browser/sync/glue/ui_data_type_controller.h"
#include "chrome/browser/sync/glue/sync_backend_host.h"
#include "chrome/browser/sync/internal_api/change_record.h"
#include "chrome/browser/sync/internal_api/read_node.h"
@@ -42,9 +41,8 @@
using base::JSONReader;
using browser_sync::GenericChangeProcessor;
-using browser_sync::PreferenceDataTypeController;
+using browser_sync::UIDataTypeController;
using browser_sync::SyncBackendHost;
-using browser_sync::SyncableServiceAdapter;
using sync_api::ChangeRecord;
using testing::_;
using testing::Invoke;
@@ -52,19 +50,10 @@ using testing::Return;
typedef std::map<const std::string, const Value*> PreferenceValues;
-ACTION_P5(BuildPrefSyncComponents, profile_sync_service, pref_sync_service,
- data_type_controller, model_associator_ptr, change_processor_ptr) {
- sync_api::UserShare* user_share = profile_sync_service->GetUserShare();
- *change_processor_ptr = new GenericChangeProcessor(
- data_type_controller,
- pref_sync_service->AsWeakPtr(),
- user_share);
- *model_associator_ptr = new browser_sync::SyncableServiceAdapter(
- syncable::PREFERENCES,
- pref_sync_service,
- *change_processor_ptr);
- return ProfileSyncComponentsFactory::SyncComponents(*model_associator_ptr,
- *change_processor_ptr);
+ACTION_P(CreateAndSaveChangeProcessor, change_processor) {
+ sync_api::UserShare* user_share = arg0->GetUserShare();
+ *change_processor = new GenericChangeProcessor(arg1, arg2, user_share);
+ return *change_processor;
}
// TODO(zea): Refactor to remove the ProfileSyncService usage.
@@ -136,22 +125,17 @@ class ProfileSyncServicePreferenceTest
prefs_->GetSyncableService());
if (!pref_sync_service_)
return false;
-
- dtc_ = new PreferenceDataTypeController(factory,
- profile_.get(),
- service_.get());
-
- EXPECT_CALL(*factory, CreatePreferenceSyncComponents(_, _)).
- WillOnce(BuildPrefSyncComponents(service_.get(),
- pref_sync_service_,
- dtc_,
- &model_associator_,
- &change_processor_));
+ EXPECT_CALL(*factory, GetSyncableServiceForType(syncable::PREFERENCES)).
+ WillOnce(Return(pref_sync_service_->AsWeakPtr()));
EXPECT_CALL(*factory, CreateDataTypeManager(_, _)).
WillOnce(ReturnNewDataTypeManager());
-
-
+ dtc_ = new UIDataTypeController(syncable::PREFERENCES,
+ factory,
+ profile_.get(),
+ service_.get());
+ EXPECT_CALL(*factory, CreateGenericChangeProcessor(_, _, _)).
+ WillOnce(CreateAndSaveChangeProcessor(&change_processor_));
service_->RegisterDataTypeController(dtc_);
profile_->GetTokenService()->IssueAuthTokenForTest(
GaiaConstants::kSyncService, "token");
@@ -209,9 +193,8 @@ class ProfileSyncServicePreferenceTest
scoped_ptr<TestingProfile> profile_;
TestingPrefService* prefs_;
- PreferenceDataTypeController* dtc_;
+ UIDataTypeController* dtc_;
PrefModelAssociator* pref_sync_service_;
- SyncableServiceAdapter* model_associator_;
GenericChangeProcessor* change_processor_;
std::string example_url0_;
« no previous file with comments | « chrome/browser/sync/profile_sync_service_autofill_unittest.cc ('k') | chrome/chrome_browser.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698