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

Unified Diff: components/browser_sync/profile_sync_service_startup_unittest.cc

Issue 2555433003: [Sync] Plumb initial type set from engine to DTM. (Closed)
Patch Set: Rebase. Created 4 years 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: components/browser_sync/profile_sync_service_startup_unittest.cc
diff --git a/components/browser_sync/profile_sync_service_startup_unittest.cc b/components/browser_sync/profile_sync_service_startup_unittest.cc
index 1ff7cdfab80afc4e265150853d93ef02a99cf8bd..dae10f171f10febab39c461f91a3ae4c5158bad7 100644
--- a/components/browser_sync/profile_sync_service_startup_unittest.cc
+++ b/components/browser_sync/profile_sync_service_startup_unittest.cc
@@ -131,7 +131,7 @@ class ProfileSyncServiceStartupTest : public testing::Test {
DataTypeManagerMock* SetUpDataTypeManager() {
DataTypeManagerMock* data_type_manager = new DataTypeManagerMock();
- EXPECT_CALL(*component_factory_, CreateDataTypeManager(_, _, _, _, _))
+ EXPECT_CALL(*component_factory_, CreateDataTypeManager(_, _, _, _, _, _))
.WillOnce(Return(data_type_manager));
return data_type_manager;
}
@@ -213,7 +213,7 @@ TEST_F(ProfileSyncServiceStartupTest, DISABLED_StartNoCredentials) {
// Should not actually start, rather just clean things up and wait
// to be enabled.
- EXPECT_CALL(*component_factory_, CreateDataTypeManager(_, _, _, _, _))
+ EXPECT_CALL(*component_factory_, CreateDataTypeManager(_, _, _, _, _, _))
.Times(0);
EXPECT_CALL(observer_, OnStateChanged()).Times(AnyNumber());
sync_service_->Initialize();
@@ -275,7 +275,7 @@ TEST_F(ProfileSyncServiceStartupTest, DISABLED_StartInvalidCredentials) {
}
TEST_F(ProfileSyncServiceStartupCrosTest, StartCrosNoCredentials) {
- EXPECT_CALL(*component_factory_, CreateDataTypeManager(_, _, _, _, _))
+ EXPECT_CALL(*component_factory_, CreateDataTypeManager(_, _, _, _, _, _))
.Times(0);
EXPECT_CALL(*component_factory_, CreateSyncEngine(_, _, _, _)).Times(0);
pref_service()->ClearPref(syncer::prefs::kSyncFirstSetupComplete);
@@ -389,7 +389,7 @@ TEST_F(ProfileSyncServiceStartupTest, ManagedStartup) {
// Disable sync through policy.
pref_service()->SetBoolean(syncer::prefs::kSyncManaged, true);
- EXPECT_CALL(*component_factory_, CreateDataTypeManager(_, _, _, _, _))
+ EXPECT_CALL(*component_factory_, CreateDataTypeManager(_, _, _, _, _, _))
.Times(0);
EXPECT_CALL(observer_, OnStateChanged()).Times(AnyNumber());
@@ -424,7 +424,7 @@ TEST_F(ProfileSyncServiceStartupTest, SwitchManaged) {
// not start automatically because IsFirstSetupComplete() will be false.
// A new DataTypeManager should not be created.
Mock::VerifyAndClearExpectations(data_type_manager);
- EXPECT_CALL(*component_factory_, CreateDataTypeManager(_, _, _, _, _))
+ EXPECT_CALL(*component_factory_, CreateDataTypeManager(_, _, _, _, _, _))
.Times(0);
pref_service()->ClearPref(syncer::prefs::kSyncManaged);
EXPECT_FALSE(sync_service_->IsEngineInitialized());

Powered by Google App Engine
This is Rietveld 408576698