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

Unified Diff: chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc

Issue 1408643002: [Sync] Componentize synced_tab_delegate (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix GN, self review Created 5 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/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc
diff --git a/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc b/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc
index ce402e2335146a990b89dfa65ffc12bb2a5dd036..8f330aba0005ce701d0727b2c66d6356ea7453d5 100644
--- a/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc
+++ b/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc
@@ -31,6 +31,7 @@
#include "components/sessions/core/session_types.h"
#include "components/sync_driver/glue/synced_session.h"
#include "components/sync_driver/local_device_info_provider_mock.h"
+#include "components/sync_sessions/fake_sync_sessions_client.h"
#include "content/public/browser/browser_thread.h"
#include "content/public/test/test_utils.h"
#include "grit/generated_resources.h"
@@ -125,7 +126,7 @@ class RecentTabsSubMenuModelTest
sync_pb::SyncEnums_DeviceType_TYPE_LINUX,
"device_id")) {
manager_.reset(new browser_sync::SessionsSyncManager(
- &testing_profile_, local_device_.get(),
+ &fake_sync_sessions_client_, &testing_profile_, local_device_.get(),
scoped_ptr<browser_sync::LocalSessionEventRouter>(new DummyRouter()),
scoped_ptr<browser_sync::SyncedWindowDelegatesGetter>(
new browser_sync::BrowserSyncedWindowDelegatesGetter())));
@@ -162,6 +163,7 @@ class RecentTabsSubMenuModelTest
TestingProfile testing_profile_;
testing::NiceMock<ProfileSyncServiceMock> sync_service_;
+ sync_sessions::FakeSyncSessionsClient fake_sync_sessions_client_;
scoped_ptr<browser_sync::SessionsSyncManager> manager_;
scoped_ptr<sync_driver::LocalDeviceInfoProviderMock> local_device_;
};

Powered by Google App Engine
This is Rietveld 408576698