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

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

Issue 8065016: [Sync] Refactor non-frontend DTC to handle new API properly. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase and self review 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_service_autofill_unittest.cc
diff --git a/chrome/browser/sync/profile_sync_service_autofill_unittest.cc b/chrome/browser/sync/profile_sync_service_autofill_unittest.cc
index 2a58e4bbb8e01af6fa2b95de5a17c311ae538560..6292f6422d1d9b3a8f46f3d2768e8b295cbd61c3 100644
--- a/chrome/browser/sync/profile_sync_service_autofill_unittest.cc
+++ b/chrome/browser/sync/profile_sync_service_autofill_unittest.cc
@@ -29,7 +29,7 @@
#include "chrome/browser/sync/glue/autofill_profile_data_type_controller.h"
#include "chrome/browser/sync/glue/autofill_profile_syncable_service.h"
#include "chrome/browser/sync/glue/data_type_controller.h"
-#include "chrome/browser/sync/glue/generic_change_processor.h"
+#include "chrome/browser/sync/glue/shared_change_processor.h"
#include "chrome/browser/sync/glue/syncable_service_adapter.h"
#include "chrome/browser/sync/internal_api/read_node.h"
#include "chrome/browser/sync/internal_api/read_transaction.h"
@@ -62,7 +62,7 @@ using browser_sync::AutofillModelAssociator;
using browser_sync::AutofillProfileDataTypeController;
using browser_sync::AutofillProfileSyncableService;
using browser_sync::DataTypeController;
-using browser_sync::GenericChangeProcessor;
+using browser_sync::SharedChangeProcessor;
using browser_sync::SyncableServiceAdapter;
using browser_sync::GROUP_DB;
using browser_sync::kAutofillTag;
@@ -179,26 +179,22 @@ ACTION_P3(MakeAutofillSyncComponents, service, wd, dtc) {
change_processor);
}
+ACTION(MakeChangeProcessor) {
+ return new SharedChangeProcessor(arg0);
+}
+
ACTION_P3(MakeAutofillProfileSyncComponents, service, wd, dtc) {
EXPECT_TRUE(BrowserThread::CurrentlyOn(BrowserThread::DB));
if (!BrowserThread::CurrentlyOn(BrowserThread::DB))
- return ProfileSyncFactory::SyncComponents(NULL, NULL);
+ return NULL;
AutofillProfileSyncableService* sync_service =
new AutofillProfileSyncableService(wd, service->profile());
- sync_api::UserShare* user_share = service->GetUserShare();
- GenericChangeProcessor* change_processor =
- new GenericChangeProcessor(sync_service, dtc, user_share);
- SyncableServiceAdapter* sync_service_adapter =
- new SyncableServiceAdapter(syncable::AUTOFILL_PROFILE,
- sync_service,
- change_processor);
- return ProfileSyncFactory::SyncComponents(sync_service_adapter,
- change_processor);
+ return sync_service;
}
class AbstractAutofillFactory {
public:
- virtual AutofillDataTypeController* CreateDataTypeController(
+ virtual DataTypeController* CreateDataTypeController(
ProfileSyncFactory* factory,
ProfileMock* profile,
ProfileSyncService* service) = 0;
@@ -211,12 +207,11 @@ class AbstractAutofillFactory {
class AutofillEntryFactory : public AbstractAutofillFactory {
public:
- browser_sync::AutofillDataTypeController* CreateDataTypeController(
+ browser_sync::DataTypeController* CreateDataTypeController(
ProfileSyncFactory* factory,
ProfileMock* profile,
ProfileSyncService* service) {
- return new AutofillDataTypeController(factory,
- profile);
+ return new AutofillDataTypeController(factory, profile);
}
void SetExpectation(ProfileSyncFactoryMock* factory,
@@ -230,19 +225,20 @@ class AutofillEntryFactory : public AbstractAutofillFactory {
class AutofillProfileFactory : public AbstractAutofillFactory {
public:
- browser_sync::AutofillDataTypeController* CreateDataTypeController(
+ browser_sync::DataTypeController* CreateDataTypeController(
ProfileSyncFactory* factory,
ProfileMock* profile,
ProfileSyncService* service) {
- return new AutofillProfileDataTypeController(factory,
- profile);
+ return new AutofillProfileDataTypeController(factory, profile);
}
void SetExpectation(ProfileSyncFactoryMock* factory,
ProfileSyncService* service,
WebDatabase* wd,
DataTypeController* dtc) {
- EXPECT_CALL(*factory, CreateAutofillProfileSyncComponents(_,_,_)).
+ EXPECT_CALL(*factory, CreateSharedChangeProcessor(_)).
+ WillOnce(MakeChangeProcessor());
+ EXPECT_CALL(*factory, CreateAutofillProfileSyncComponents(_,_)).
WillOnce(MakeAutofillProfileSyncComponents(service, wd, dtc));
}
};
@@ -316,7 +312,7 @@ class ProfileSyncServiceAutofillTest : public AbstractProfileSyncServiceTest {
task));
EXPECT_CALL(profile_, GetProfileSyncService()).WillRepeatedly(
Return(service_.get()));
- AutofillDataTypeController* data_type_controller =
+ DataTypeController* data_type_controller =
factory->CreateDataTypeController(&factory_,
&profile_,
service_.get());

Powered by Google App Engine
This is Rietveld 408576698