Index: chrome/browser/sync/sessions2/sessions_sync_manager_unittest.cc |
diff --git a/chrome/browser/sync/sessions2/sessions_sync_manager_unittest.cc b/chrome/browser/sync/sessions2/sessions_sync_manager_unittest.cc |
index 6183e67db4586ea81508443cb5844531097810a5..3f21000e04c932bfcebdc686e3008be0ac8a53d2 100644 |
--- a/chrome/browser/sync/sessions2/sessions_sync_manager_unittest.cc |
+++ b/chrome/browser/sync/sessions2/sessions_sync_manager_unittest.cc |
@@ -102,10 +102,7 @@ class SessionsSyncManagerTest |
virtual void SetUp() OVERRIDE { |
BrowserWithTestWindowTest::SetUp(); |
- manager_.reset(new SessionsSyncManager( |
- profile(), |
- scoped_ptr<SyncPrefs>(new SyncPrefs(profile()->GetPrefs())), |
- this)); |
+ manager_.reset(new SessionsSyncManager(profile(), this)); |
} |
virtual void TearDown() OVERRIDE { |
@@ -116,14 +113,14 @@ class SessionsSyncManagerTest |
virtual scoped_ptr<DeviceInfo> GetLocalDeviceInfo() const OVERRIDE { |
return scoped_ptr<DeviceInfo>( |
- new DeviceInfo(GetCacheGuid(), |
+ new DeviceInfo(GetLocalSyncCacheGUID(), |
"Wayne Gretzky's Hacking Box", |
"Chromium 10k", |
"Chrome 10k", |
sync_pb::SyncEnums_DeviceType_TYPE_LINUX)); |
} |
- virtual std::string GetCacheGuid() const OVERRIDE { |
+ virtual std::string GetLocalSyncCacheGUID() const OVERRIDE { |
return "cache_guid"; |
} |
@@ -522,10 +519,7 @@ TEST_F(SessionsSyncManagerTest, MergeLocalSessionNoTabs) { |
SyncData d(SyncData::CreateRemoteData(1, data.GetSpecifics(), base::Time())); |
syncer::SyncDataList in(&d, &d + 1); |
out.clear(); |
- SessionsSyncManager manager2( |
- profile(), |
- scoped_ptr<SyncPrefs>(new SyncPrefs(profile()->GetPrefs())), |
- this); |
+ SessionsSyncManager manager2(profile(), this); |
syncer::SyncMergeResult result = manager2.MergeDataAndStartSyncing( |
syncer::SESSIONS, in, |
scoped_ptr<syncer::SyncChangeProcessor>( |
@@ -754,7 +748,7 @@ TEST_F(SessionsSyncManagerTest, DeleteForeignSession) { |
std::vector<const SyncedSession*> foreign_sessions; |
ASSERT_FALSE(manager()->GetAllForeignSessions(&foreign_sessions)); |
- manager()->DeleteForeignSession(tag, &changes); |
+ manager()->DeleteForeignSessionInternal(tag, &changes); |
ASSERT_FALSE(manager()->GetAllForeignSessions(&foreign_sessions)); |
EXPECT_TRUE(changes.empty()); |
@@ -775,7 +769,7 @@ TEST_F(SessionsSyncManagerTest, DeleteForeignSession) { |
ASSERT_EQ(1U, foreign_sessions.size()); |
// Now delete the foreign session. |
- manager()->DeleteForeignSession(tag, &changes); |
+ manager()->DeleteForeignSessionInternal(tag, &changes); |
EXPECT_FALSE(manager()->GetAllForeignSessions(&foreign_sessions)); |
EXPECT_EQ(5U, changes.size()); |
@@ -918,10 +912,7 @@ TEST_F(SessionsSyncManagerTest, SaveUnassociatedNodesForReassociation) { |
SyncData d(SyncData::CreateRemoteData(1, entity, base::Time())); |
syncer::SyncDataList in(&d, &d + 1); |
changes.clear(); |
- SessionsSyncManager manager2( |
- profile(), |
- scoped_ptr<SyncPrefs>(new SyncPrefs(profile()->GetPrefs())), |
- this); |
+ SessionsSyncManager manager2(profile(), this); |
syncer::SyncMergeResult result = manager2.MergeDataAndStartSyncing( |
syncer::SESSIONS, in, |
scoped_ptr<syncer::SyncChangeProcessor>( |