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

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 test broken by rebase 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..c3050416152cc3a270579f03b5721659c862725f 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_driver/sync_client.h"
#include "content/public/browser/browser_thread.h"
#include "content/public/test/test_utils.h"
#include "grit/generated_resources.h"
@@ -125,10 +126,9 @@ class RecentTabsSubMenuModelTest
sync_pb::SyncEnums_DeviceType_TYPE_LINUX,
"device_id")) {
manager_.reset(new browser_sync::SessionsSyncManager(
+ sync_service_.GetSyncClient()->GetSyncSessionsClient(),
&testing_profile_, local_device_.get(),
- scoped_ptr<browser_sync::LocalSessionEventRouter>(new DummyRouter()),
- scoped_ptr<browser_sync::SyncedWindowDelegatesGetter>(
- new browser_sync::BrowserSyncedWindowDelegatesGetter())));
+ scoped_ptr<browser_sync::LocalSessionEventRouter>(new DummyRouter())));
manager_->MergeDataAndStartSyncing(
syncer::SESSIONS,
syncer::SyncDataList(),

Powered by Google App Engine
This is Rietveld 408576698