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

Unified Diff: chrome/browser/ui/cocoa/app_menu/app_menu_controller_unittest.mm

Issue 2753753005: [sync] WebContentsObserver based sessions notifications (Closed)
Patch Set: use base:MakeUnique, alphabetize Created 3 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
Index: chrome/browser/ui/cocoa/app_menu/app_menu_controller_unittest.mm
diff --git a/chrome/browser/ui/cocoa/app_menu/app_menu_controller_unittest.mm b/chrome/browser/ui/cocoa/app_menu/app_menu_controller_unittest.mm
index 34d097b01c07536e1e483c48b80e65b7d26cdf76..579a76052e99998c26b57edaa3b5eaf6d99c8623 100644
--- a/chrome/browser/ui/cocoa/app_menu/app_menu_controller_unittest.mm
+++ b/chrome/browser/ui/cocoa/app_menu/app_menu_controller_unittest.mm
@@ -76,13 +76,12 @@ class AppMenuControllerTest : public CocoaProfileTest {
fake_model_.reset(new MockAppMenuModel);
sync_prefs_.reset(new syncer::SyncPrefs(profile()->GetPrefs()));
+ dummy_router_.reset(new DummyRouter());
manager_.reset(new sync_sessions::SessionsSyncManager(
ProfileSyncServiceFactory::GetForProfile(profile())
->GetSyncClient()
->GetSyncSessionsClient(),
- sync_prefs_.get(), local_device_.get(),
- std::unique_ptr<sync_sessions::LocalSessionEventRouter>(
- new DummyRouter()),
+ sync_prefs_.get(), local_device_.get(), dummy_router_.get(),
base::Closure(), base::Closure()));
manager_->MergeDataAndStartSyncing(
syncer::SESSIONS, syncer::SyncDataList(),
@@ -117,6 +116,7 @@ class AppMenuControllerTest : public CocoaProfileTest {
private:
std::unique_ptr<syncer::SyncPrefs> sync_prefs_;
+ std::unique_ptr<DummyRouter> dummy_router_;
std::unique_ptr<sync_sessions::SessionsSyncManager> manager_;
std::unique_ptr<syncer::LocalDeviceInfoProviderMock> local_device_;
};

Powered by Google App Engine
This is Rietveld 408576698