| Index: chrome/browser/sync/sessions/sessions_sync_manager_unittest.cc
|
| diff --git a/chrome/browser/sync/sessions/sessions_sync_manager_unittest.cc b/chrome/browser/sync/sessions/sessions_sync_manager_unittest.cc
|
| index 7119cb4f39a92857957eb9e8ceafd7074f7cf27e..9d55db903450d378398a7f53bf937fecc667dd31 100644
|
| --- a/chrome/browser/sync/sessions/sessions_sync_manager_unittest.cc
|
| +++ b/chrome/browser/sync/sessions/sessions_sync_manager_unittest.cc
|
| @@ -45,11 +45,10 @@ using sessions::SerializedNavigationEntryTestHelper;
|
| using sync_driver::DeviceInfo;
|
| using sync_driver::LocalDeviceInfoProvider;
|
| using sync_driver::LocalDeviceInfoProviderMock;
|
| -using sync_driver::SyncedSession;
|
| using syncer::SyncChange;
|
| using syncer::SyncData;
|
|
|
| -namespace browser_sync {
|
| +namespace sync_sessions {
|
|
|
| namespace {
|
|
|
| @@ -210,11 +209,10 @@ int CountIfTagMatches(const syncer::SyncChangeList& changes,
|
| });
|
| }
|
|
|
| -int CountIfTagMatches(
|
| - const std::vector<const sync_driver::SyncedSession*>& sessions,
|
| - const std::string& tag) {
|
| +int CountIfTagMatches(const std::vector<const SyncedSession*>& sessions,
|
| + const std::string& tag) {
|
| return std::count_if(sessions.begin(), sessions.end(),
|
| - [&tag](const sync_driver::SyncedSession* session) {
|
| + [&tag](const SyncedSession* session) {
|
| return session->session_tag == tag;
|
| });
|
| }
|
| @@ -245,10 +243,9 @@ std::unique_ptr<LocalSessionEventRouter> NewDummyRouter() {
|
|
|
| // Provides ability to override SyncedWindowDelegatesGetter.
|
| // All other calls are passed through to the original SyncSessionsClient.
|
| -class SyncSessionsClientShim : public sync_sessions::SyncSessionsClient {
|
| +class SyncSessionsClientShim : public SyncSessionsClient {
|
| public:
|
| - SyncSessionsClientShim(
|
| - sync_sessions::SyncSessionsClient* sync_sessions_client)
|
| + explicit SyncSessionsClientShim(SyncSessionsClient* sync_sessions_client)
|
| : sync_sessions_client_(sync_sessions_client),
|
| synced_window_getter_(nullptr) {}
|
| ~SyncSessionsClientShim() override {}
|
| @@ -269,8 +266,7 @@ class SyncSessionsClientShim : public sync_sessions::SyncSessionsClient {
|
| return sync_sessions_client_->ShouldSyncURL(url);
|
| }
|
|
|
| - browser_sync::SyncedWindowDelegatesGetter* GetSyncedWindowDelegatesGetter()
|
| - override {
|
| + SyncedWindowDelegatesGetter* GetSyncedWindowDelegatesGetter() override {
|
| // The idea here is to allow the test code override the default
|
| // SyncedWindowDelegatesGetter provided by |sync_sessions_client_|.
|
| // If |synced_window_getter_| is explicitly set, return it; otherwise return
|
| @@ -280,19 +276,19 @@ class SyncSessionsClientShim : public sync_sessions::SyncSessionsClient {
|
| : sync_sessions_client_->GetSyncedWindowDelegatesGetter();
|
| }
|
|
|
| - std::unique_ptr<browser_sync::LocalSessionEventRouter>
|
| - GetLocalSessionEventRouter() override {
|
| + std::unique_ptr<LocalSessionEventRouter> GetLocalSessionEventRouter()
|
| + override {
|
| return sync_sessions_client_->GetLocalSessionEventRouter();
|
| }
|
|
|
| void set_synced_window_getter(
|
| - browser_sync::SyncedWindowDelegatesGetter* synced_window_getter) {
|
| + SyncedWindowDelegatesGetter* synced_window_getter) {
|
| synced_window_getter_ = synced_window_getter;
|
| }
|
|
|
| private:
|
| - sync_sessions::SyncSessionsClient* const sync_sessions_client_;
|
| - browser_sync::SyncedWindowDelegatesGetter* synced_window_getter_;
|
| + SyncSessionsClient* const sync_sessions_client_;
|
| + SyncedWindowDelegatesGetter* synced_window_getter_;
|
| };
|
|
|
| } // namespace
|
| @@ -392,18 +388,18 @@ class SessionsSyncManagerTest
|
| return list;
|
| }
|
|
|
| - sync_sessions::SyncSessionsClient* GetSyncSessionsClient() {
|
| + SyncSessionsClient* GetSyncSessionsClient() {
|
| return sessions_client_shim_.get();
|
| }
|
|
|
| sync_driver::SyncPrefs* sync_prefs() { return sync_prefs_.get(); }
|
|
|
| - browser_sync::SyncedWindowDelegatesGetter* get_synced_window_getter() {
|
| + SyncedWindowDelegatesGetter* get_synced_window_getter() {
|
| return manager()->synced_window_delegates_getter();
|
| }
|
|
|
| void set_synced_window_getter(
|
| - browser_sync::SyncedWindowDelegatesGetter* synced_window_getter) {
|
| + SyncedWindowDelegatesGetter* synced_window_getter) {
|
| sessions_client_shim_->set_synced_window_getter(synced_window_getter);
|
| }
|
|
|
| @@ -578,7 +574,7 @@ class SyncedTabDelegateFake : public SyncedTabDelegate {
|
| int GetSyncId() const override { return sync_id_; }
|
| void SetSyncId(int sync_id) override { sync_id_ = sync_id; }
|
|
|
| - bool ShouldSync(sync_sessions::SyncSessionsClient* sessions_client) override {
|
| + bool ShouldSync(SyncSessionsClient* sessions_client) override {
|
| return false;
|
| }
|
|
|
| @@ -1781,7 +1777,7 @@ TEST_F(SessionsSyncManagerTest, MergeDeletesBadHash) {
|
| EXPECT_EQ(1, CountIfTagMatches(output, bad_header_tag));
|
| EXPECT_EQ(1, CountIfTagMatches(output, bad_tab_tag));
|
|
|
| - std::vector<const sync_driver::SyncedSession*> sessions;
|
| + std::vector<const SyncedSession*> sessions;
|
| manager()->session_tracker_.LookupAllForeignSessions(
|
| &sessions, SyncedSessionTracker::RAW);
|
| ASSERT_EQ(2U, sessions.size());
|
| @@ -2606,4 +2602,4 @@ TEST_F(SessionsSyncManagerTest, GetForeignSessionTabs) {
|
| }
|
| }
|
|
|
| -} // namespace browser_sync
|
| +} // namespace sync_sessions
|
|
|