| 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
|
| deleted file mode 100644
|
| index 886c2a4bf72a1cf615ff1244ab5abf6a85fbddbd..0000000000000000000000000000000000000000
|
| --- a/chrome/browser/sync/sessions/sessions_sync_manager_unittest.cc
|
| +++ /dev/null
|
| @@ -1,2612 +0,0 @@
|
| -// Copyright 2014 The Chromium Authors. All rights reserved.
|
| -// Use of this source code is governed by a BSD-style license that can be
|
| -// found in the LICENSE file.
|
| -
|
| -#include "components/sync_sessions/sessions_sync_manager.h"
|
| -
|
| -#include <stdint.h>
|
| -
|
| -#include <utility>
|
| -
|
| -#include "base/memory/ptr_util.h"
|
| -#include "base/strings/string_util.h"
|
| -#include "build/build_config.h"
|
| -#include "chrome/browser/sessions/session_tab_helper.h"
|
| -#include "chrome/browser/sync/chrome_sync_client.h"
|
| -#include "chrome/browser/sync/sessions/notification_service_sessions_router.h"
|
| -#include "chrome/browser/ui/sync/browser_synced_window_delegates_getter.h"
|
| -#include "chrome/browser/ui/sync/tab_contents_synced_tab_delegate.h"
|
| -#include "chrome/browser/ui/tabs/tab_strip_model.h"
|
| -#include "chrome/test/base/browser_with_test_window_test.h"
|
| -#include "components/sessions/content/content_serialized_navigation_builder.h"
|
| -#include "components/sessions/core/serialized_navigation_entry_test_helper.h"
|
| -#include "components/sync/device_info/local_device_info_provider_mock.h"
|
| -#include "components/sync/driver/sync_api_component_factory.h"
|
| -#include "components/sync/model/attachments/attachment_id.h"
|
| -#include "components/sync/model/attachments/attachment_service_proxy_for_test.h"
|
| -#include "components/sync/model/sync_error_factory_mock.h"
|
| -#include "components/sync_sessions/session_sync_test_helper.h"
|
| -#include "components/sync_sessions/sync_sessions_client.h"
|
| -#include "components/sync_sessions/synced_tab_delegate.h"
|
| -#include "components/sync_sessions/synced_window_delegate.h"
|
| -#include "components/sync_sessions/synced_window_delegates_getter.h"
|
| -#include "content/public/browser/navigation_entry.h"
|
| -#include "content/public/browser/web_contents.h"
|
| -#include "testing/gmock/include/gmock/gmock.h"
|
| -#include "testing/gtest/include/gtest/gtest.h"
|
| -
|
| -using content::WebContents;
|
| -using sessions::SerializedNavigationEntry;
|
| -using sessions::SerializedNavigationEntryTestHelper;
|
| -using syncer::DeviceInfo;
|
| -using syncer::LocalDeviceInfoProvider;
|
| -using syncer::LocalDeviceInfoProviderMock;
|
| -using syncer::SyncChange;
|
| -using syncer::SyncData;
|
| -
|
| -namespace sync_sessions {
|
| -
|
| -namespace {
|
| -
|
| -class SessionNotificationObserver {
|
| - public:
|
| - SessionNotificationObserver()
|
| - : notified_of_update_(false), notified_of_refresh_(false) {}
|
| - void NotifyOfUpdate() { notified_of_update_ = true; }
|
| - void NotifyOfRefresh() { notified_of_refresh_ = true; }
|
| -
|
| - bool notified_of_update() const { return notified_of_update_; }
|
| - bool notified_of_refresh() const { return notified_of_refresh_; }
|
| -
|
| - void Reset() {
|
| - notified_of_update_ = false;
|
| - notified_of_refresh_ = false;
|
| - }
|
| -
|
| - private:
|
| - bool notified_of_update_;
|
| - bool notified_of_refresh_;
|
| -};
|
| -
|
| -class SyncedWindowDelegateOverride : public SyncedWindowDelegate {
|
| - public:
|
| - explicit SyncedWindowDelegateOverride(const SyncedWindowDelegate* wrapped)
|
| - : wrapped_(wrapped) {
|
| - }
|
| - ~SyncedWindowDelegateOverride() override {}
|
| -
|
| - bool HasWindow() const override { return wrapped_->HasWindow(); }
|
| -
|
| - SessionID::id_type GetSessionId() const override {
|
| - return session_id_override_ >= 0 ? session_id_override_
|
| - : wrapped_->GetSessionId();
|
| - }
|
| -
|
| - int GetTabCount() const override { return wrapped_->GetTabCount(); }
|
| -
|
| - int GetActiveIndex() const override { return wrapped_->GetActiveIndex(); }
|
| -
|
| - bool IsApp() const override { return wrapped_->IsApp(); }
|
| -
|
| - bool IsTypeTabbed() const override { return wrapped_->IsTypeTabbed(); }
|
| -
|
| - bool IsTypePopup() const override { return wrapped_->IsTypePopup(); }
|
| -
|
| - bool IsTabPinned(const SyncedTabDelegate* tab) const override {
|
| - return wrapped_->IsTabPinned(tab);
|
| - }
|
| -
|
| - SyncedTabDelegate* GetTabAt(int index) const override {
|
| - if (tab_overrides_.find(index) != tab_overrides_.end())
|
| - return tab_overrides_.find(index)->second;
|
| -
|
| - return wrapped_->GetTabAt(index);
|
| - }
|
| -
|
| - void OverrideSessionId(SessionID::id_type id) { session_id_override_ = id; }
|
| -
|
| - void OverrideTabAt(int index,
|
| - SyncedTabDelegate* delegate,
|
| - SessionID::id_type tab_id) {
|
| - tab_overrides_[index] = delegate;
|
| - tab_id_overrides_[index] = tab_id;
|
| - }
|
| -
|
| - SessionID::id_type GetTabIdAt(int index) const override {
|
| - if (tab_id_overrides_.find(index) != tab_id_overrides_.end())
|
| - return tab_id_overrides_.find(index)->second;
|
| - return wrapped_->GetTabIdAt(index);
|
| - }
|
| -
|
| - bool IsSessionRestoreInProgress() const override {
|
| - return wrapped_->IsSessionRestoreInProgress();
|
| - }
|
| -
|
| - bool ShouldSync() const override { return wrapped_->ShouldSync(); }
|
| -
|
| - private:
|
| - std::map<int, SyncedTabDelegate*> tab_overrides_;
|
| - std::map<int, SessionID::id_type> tab_id_overrides_;
|
| - const SyncedWindowDelegate* const wrapped_;
|
| - SessionID::id_type session_id_override_ = -1;
|
| -};
|
| -
|
| -class TestSyncedWindowDelegatesGetter : public SyncedWindowDelegatesGetter {
|
| - public:
|
| - TestSyncedWindowDelegatesGetter(
|
| - const std::set<const SyncedWindowDelegate*>& delegates)
|
| - : delegates_(delegates) {}
|
| -
|
| - std::set<const SyncedWindowDelegate*> GetSyncedWindowDelegates() override {
|
| - return delegates_;
|
| - }
|
| -
|
| - const SyncedWindowDelegate* FindById(SessionID::id_type id) override {
|
| - for (auto* window : delegates_) {
|
| - if (window->GetSessionId() == id)
|
| - return window;
|
| - }
|
| - return nullptr;
|
| - }
|
| -
|
| - private:
|
| - const std::set<const SyncedWindowDelegate*> delegates_;
|
| -};
|
| -
|
| -class TestSyncProcessorStub : public syncer::SyncChangeProcessor {
|
| - public:
|
| - explicit TestSyncProcessorStub(syncer::SyncChangeList* output)
|
| - : output_(output) {}
|
| - syncer::SyncError ProcessSyncChanges(
|
| - const tracked_objects::Location& from_here,
|
| - const syncer::SyncChangeList& change_list) override {
|
| - if (error_.IsSet()) {
|
| - syncer::SyncError error = error_;
|
| - error_ = syncer::SyncError();
|
| - return error;
|
| - }
|
| -
|
| - if (output_)
|
| - output_->insert(output_->end(), change_list.begin(), change_list.end());
|
| - NotifyLocalChangeObservers();
|
| -
|
| - return syncer::SyncError();
|
| - }
|
| -
|
| - syncer::SyncDataList GetAllSyncData(syncer::ModelType type) const override {
|
| - return sync_data_to_return_;
|
| - }
|
| -
|
| - void AddLocalChangeObserver(syncer::LocalChangeObserver* observer) override {
|
| - local_change_observers_.AddObserver(observer);
|
| - }
|
| - void RemoveLocalChangeObserver(
|
| - syncer::LocalChangeObserver* observer) override {
|
| - local_change_observers_.RemoveObserver(observer);
|
| - }
|
| -
|
| - void NotifyLocalChangeObservers() {
|
| - const syncer::SyncChange empty_change;
|
| - for (syncer::LocalChangeObserver& observer : local_change_observers_)
|
| - observer.OnLocalChange(nullptr, empty_change);
|
| - }
|
| -
|
| - void FailProcessSyncChangesWith(const syncer::SyncError& error) {
|
| - error_ = error;
|
| - }
|
| -
|
| - void SetSyncDataToReturn(const syncer::SyncDataList& data) {
|
| - sync_data_to_return_ = data;
|
| - }
|
| -
|
| - private:
|
| - syncer::SyncError error_;
|
| - syncer::SyncChangeList* output_;
|
| - syncer::SyncDataList sync_data_to_return_;
|
| - base::ObserverList<syncer::LocalChangeObserver> local_change_observers_;
|
| -};
|
| -
|
| -void ExpectAllOfChangesType(const syncer::SyncChangeList& changes,
|
| - const syncer::SyncChange::SyncChangeType type) {
|
| - for (const syncer::SyncChange& change : changes) {
|
| - EXPECT_EQ(type, change.change_type());
|
| - }
|
| -}
|
| -
|
| -int CountIfTagMatches(const syncer::SyncChangeList& changes,
|
| - const std::string& tag) {
|
| - return std::count_if(
|
| - changes.begin(), changes.end(), [&tag](const syncer::SyncChange& change) {
|
| - return change.sync_data().GetSpecifics().session().session_tag() == tag;
|
| - });
|
| -}
|
| -
|
| -int CountIfTagMatches(const std::vector<const SyncedSession*>& sessions,
|
| - const std::string& tag) {
|
| - return std::count_if(sessions.begin(), sessions.end(),
|
| - [&tag](const SyncedSession* session) {
|
| - return session->session_tag == tag;
|
| - });
|
| -}
|
| -
|
| -// Creates a field trial with the specified |trial_name| and |group_name| and
|
| -// registers an associated |variation_id| for it for the given |service|.
|
| -void CreateAndActivateFieldTrial(const std::string& trial_name,
|
| - const std::string& group_name,
|
| - variations::VariationID variation_id,
|
| - variations::IDCollectionKey service) {
|
| - base::FieldTrialList::CreateFieldTrial(trial_name, group_name);
|
| - variations::AssociateGoogleVariationID(service, trial_name, group_name,
|
| - variation_id);
|
| - // Access the trial to activate it.
|
| - base::FieldTrialList::FindFullName(trial_name);
|
| -}
|
| -
|
| -class DummyRouter : public LocalSessionEventRouter {
|
| - public:
|
| - ~DummyRouter() override {}
|
| - void StartRoutingTo(LocalSessionEventHandler* handler) override {}
|
| - void Stop() override {}
|
| -};
|
| -
|
| -std::unique_ptr<LocalSessionEventRouter> NewDummyRouter() {
|
| - return std::unique_ptr<LocalSessionEventRouter>(new DummyRouter());
|
| -}
|
| -
|
| -// Provides ability to override SyncedWindowDelegatesGetter.
|
| -// All other calls are passed through to the original SyncSessionsClient.
|
| -class SyncSessionsClientShim : public SyncSessionsClient {
|
| - public:
|
| - explicit SyncSessionsClientShim(SyncSessionsClient* sync_sessions_client)
|
| - : sync_sessions_client_(sync_sessions_client),
|
| - synced_window_getter_(nullptr) {}
|
| - ~SyncSessionsClientShim() override {}
|
| -
|
| - bookmarks::BookmarkModel* GetBookmarkModel() override {
|
| - return sync_sessions_client_->GetBookmarkModel();
|
| - }
|
| -
|
| - favicon::FaviconService* GetFaviconService() override {
|
| - return sync_sessions_client_->GetFaviconService();
|
| - }
|
| -
|
| - history::HistoryService* GetHistoryService() override {
|
| - return sync_sessions_client_->GetHistoryService();
|
| - }
|
| -
|
| - bool ShouldSyncURL(const GURL& url) const override {
|
| - return sync_sessions_client_->ShouldSyncURL(url);
|
| - }
|
| -
|
| - 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
|
| - // the default one provided by |sync_sessions_client_|.
|
| - return synced_window_getter_
|
| - ? synced_window_getter_
|
| - : sync_sessions_client_->GetSyncedWindowDelegatesGetter();
|
| - }
|
| -
|
| - std::unique_ptr<LocalSessionEventRouter> GetLocalSessionEventRouter()
|
| - override {
|
| - return sync_sessions_client_->GetLocalSessionEventRouter();
|
| - }
|
| -
|
| - void set_synced_window_getter(
|
| - SyncedWindowDelegatesGetter* synced_window_getter) {
|
| - synced_window_getter_ = synced_window_getter;
|
| - }
|
| -
|
| - private:
|
| - SyncSessionsClient* const sync_sessions_client_;
|
| - SyncedWindowDelegatesGetter* synced_window_getter_;
|
| -};
|
| -
|
| -} // namespace
|
| -
|
| -class SessionsSyncManagerTest
|
| - : public BrowserWithTestWindowTest {
|
| - protected:
|
| - SessionsSyncManagerTest() : test_processor_(nullptr) {
|
| - local_device_ = base::MakeUnique<LocalDeviceInfoProviderMock>(
|
| - "cache_guid", "Wayne Gretzky's Hacking Box", "Chromium 10k",
|
| - "Chrome 10k", sync_pb::SyncEnums_DeviceType_TYPE_LINUX, "device_id");
|
| - }
|
| -
|
| - void SetUp() override {
|
| - BrowserWithTestWindowTest::SetUp();
|
| - sync_client_ = base::MakeUnique<browser_sync::ChromeSyncClient>(profile());
|
| - sessions_client_shim_ = base::MakeUnique<SyncSessionsClientShim>(
|
| - sync_client_->GetSyncSessionsClient());
|
| - NotificationServiceSessionsRouter* router(
|
| - new NotificationServiceSessionsRouter(
|
| - profile(), GetSyncSessionsClient(),
|
| - syncer::SyncableService::StartSyncFlare()));
|
| - sync_prefs_ = base::MakeUnique<syncer::SyncPrefs>(profile()->GetPrefs());
|
| - manager_ = base::MakeUnique<SessionsSyncManager>(
|
| - GetSyncSessionsClient(), sync_prefs_.get(), local_device_.get(),
|
| - std::unique_ptr<LocalSessionEventRouter>(router),
|
| - base::Bind(&SessionNotificationObserver::NotifyOfUpdate,
|
| - base::Unretained(&observer_)),
|
| - base::Bind(&SessionNotificationObserver::NotifyOfRefresh,
|
| - base::Unretained(&observer_)));
|
| - }
|
| -
|
| - void TearDown() override {
|
| - test_processor_ = nullptr;
|
| - helper()->Reset();
|
| - sync_prefs_.reset();
|
| - manager_.reset();
|
| - BrowserWithTestWindowTest::TearDown();
|
| - }
|
| -
|
| - const DeviceInfo* GetLocalDeviceInfo() {
|
| - return local_device_->GetLocalDeviceInfo();
|
| - }
|
| -
|
| - SessionsSyncManager* manager() { return manager_.get(); }
|
| - SessionSyncTestHelper* helper() { return &helper_; }
|
| - LocalDeviceInfoProvider* local_device() { return local_device_.get(); }
|
| - SessionNotificationObserver* observer() { return &observer_; }
|
| -
|
| - void InitWithSyncDataTakeOutput(const syncer::SyncDataList& initial_data,
|
| - syncer::SyncChangeList* output) {
|
| - test_processor_ = new TestSyncProcessorStub(output);
|
| - syncer::SyncMergeResult result = manager_->MergeDataAndStartSyncing(
|
| - syncer::SESSIONS, initial_data,
|
| - std::unique_ptr<syncer::SyncChangeProcessor>(test_processor_),
|
| - std::unique_ptr<syncer::SyncErrorFactory>(
|
| - new syncer::SyncErrorFactoryMock()));
|
| - EXPECT_FALSE(result.error().IsSet());
|
| - }
|
| -
|
| - void InitWithNoSyncData() {
|
| - InitWithSyncDataTakeOutput(syncer::SyncDataList(), nullptr);
|
| - }
|
| -
|
| - void TriggerProcessSyncChangesError() {
|
| - test_processor_->FailProcessSyncChangesWith(syncer::SyncError(
|
| - FROM_HERE, syncer::SyncError::DATATYPE_ERROR, "Error",
|
| - syncer::SESSIONS));
|
| - }
|
| -
|
| - void SetSyncData(const syncer::SyncDataList& data) {
|
| - test_processor_->SetSyncDataToReturn(data);
|
| - }
|
| -
|
| - syncer::SyncChangeList* FilterOutLocalHeaderChanges(
|
| - syncer::SyncChangeList* list) {
|
| - syncer::SyncChangeList::iterator it = list->begin();
|
| - bool found = false;
|
| - while (it != list->end()) {
|
| - if (it->sync_data().IsLocal() &&
|
| - syncer::SyncDataLocal(it->sync_data()).GetTag() ==
|
| - manager_->current_machine_tag()) {
|
| - EXPECT_TRUE(SyncChange::ACTION_ADD == it->change_type() ||
|
| - SyncChange::ACTION_UPDATE == it->change_type());
|
| - it = list->erase(it);
|
| - found = true;
|
| - } else {
|
| - ++it;
|
| - }
|
| - }
|
| - EXPECT_TRUE(found);
|
| - return list;
|
| - }
|
| -
|
| - SyncSessionsClient* GetSyncSessionsClient() {
|
| - return sessions_client_shim_.get();
|
| - }
|
| -
|
| - syncer::SyncPrefs* sync_prefs() { return sync_prefs_.get(); }
|
| -
|
| - SyncedWindowDelegatesGetter* get_synced_window_getter() {
|
| - return manager()->synced_window_delegates_getter();
|
| - }
|
| -
|
| - void set_synced_window_getter(
|
| - SyncedWindowDelegatesGetter* synced_window_getter) {
|
| - sessions_client_shim_->set_synced_window_getter(synced_window_getter);
|
| - }
|
| -
|
| - syncer::SyncChange MakeRemoteChange(
|
| - const sync_pb::SessionSpecifics& specifics,
|
| - SyncChange::SyncChangeType type) const {
|
| - return syncer::SyncChange(FROM_HERE, type, CreateRemoteData(specifics));
|
| - }
|
| -
|
| - void AddTabsToChangeList(const std::vector<sync_pb::SessionSpecifics>& batch,
|
| - SyncChange::SyncChangeType type,
|
| - syncer::SyncChangeList* change_list) const {
|
| - for (const auto& specifics : batch) {
|
| - change_list->push_back(
|
| - syncer::SyncChange(FROM_HERE, type, CreateRemoteData(specifics)));
|
| - }
|
| - }
|
| -
|
| - void AddToSyncDataList(const sync_pb::SessionSpecifics& specifics,
|
| - syncer::SyncDataList* list,
|
| - base::Time mtime) const {
|
| - list->push_back(CreateRemoteData(specifics, mtime));
|
| - }
|
| -
|
| - void AddTabsToSyncDataList(const std::vector<sync_pb::SessionSpecifics>& tabs,
|
| - syncer::SyncDataList* list) const {
|
| - for (size_t i = 0; i < tabs.size(); i++) {
|
| - AddToSyncDataList(tabs[i], list, base::Time::FromInternalValue(i + 1));
|
| - }
|
| - }
|
| -
|
| - syncer::SyncData CreateRemoteData(const sync_pb::SessionSpecifics& specifics,
|
| - base::Time mtime = base::Time()) const {
|
| - sync_pb::EntitySpecifics entity;
|
| - entity.mutable_session()->CopyFrom(specifics);
|
| - return CreateRemoteData(entity, mtime);
|
| - }
|
| -
|
| - syncer::SyncData CreateRemoteData(const sync_pb::EntitySpecifics& entity,
|
| - base::Time mtime = base::Time()) const {
|
| - // The server ID is never relevant to these tests, so just use 1.
|
| - return SyncData::CreateRemoteData(
|
| - 1, entity, mtime, syncer::AttachmentIdList(),
|
| - syncer::AttachmentServiceProxyForTest::Create(),
|
| - SessionsSyncManager::TagHashFromSpecifics(entity.session()));
|
| - }
|
| -
|
| - private:
|
| - std::unique_ptr<browser_sync::ChromeSyncClient> sync_client_;
|
| - std::unique_ptr<SyncSessionsClientShim> sessions_client_shim_;
|
| - std::unique_ptr<syncer::SyncPrefs> sync_prefs_;
|
| - SessionNotificationObserver observer_;
|
| - std::unique_ptr<SessionsSyncManager> manager_;
|
| - SessionSyncTestHelper helper_;
|
| - TestSyncProcessorStub* test_processor_;
|
| - std::unique_ptr<LocalDeviceInfoProviderMock> local_device_;
|
| -};
|
| -
|
| -// Test that the SyncSessionManager can properly fill in a SessionHeader.
|
| -TEST_F(SessionsSyncManagerTest, PopulateSessionHeader) {
|
| - sync_pb::SessionHeader header_s;
|
| - header_s.set_client_name("Client 1");
|
| - header_s.set_device_type(sync_pb::SyncEnums_DeviceType_TYPE_WIN);
|
| -
|
| - SyncedSession session;
|
| - base::Time time = base::Time::Now();
|
| - SessionsSyncManager::PopulateSessionHeaderFromSpecifics(
|
| - header_s, time, &session);
|
| - ASSERT_EQ("Client 1", session.session_name);
|
| - ASSERT_EQ(SyncedSession::TYPE_WIN, session.device_type);
|
| - ASSERT_EQ(time, session.modified_time);
|
| -}
|
| -
|
| -// Test translation between protobuf types and chrome session types.
|
| -TEST_F(SessionsSyncManagerTest, PopulateSessionWindow) {
|
| - sync_pb::SessionWindow window_s;
|
| - window_s.add_tab(0);
|
| - window_s.set_browser_type(sync_pb::SessionWindow_BrowserType_TYPE_TABBED);
|
| - window_s.set_selected_tab_index(1);
|
| -
|
| - std::string tag = "tag";
|
| - SyncedSession* session = manager()->session_tracker_.GetSession(tag);
|
| - manager()->session_tracker_.PutWindowInSession(tag, 0);
|
| - manager()->BuildSyncedSessionFromSpecifics(tag, window_s, base::Time(),
|
| - session->windows[0].get());
|
| - ASSERT_EQ(1U, session->windows[0]->tabs.size());
|
| - ASSERT_EQ(1, session->windows[0]->selected_tab_index);
|
| - ASSERT_EQ(sessions::SessionWindow::TYPE_TABBED, session->windows[0]->type);
|
| - ASSERT_EQ(1U, manager()->session_tracker_.num_synced_sessions());
|
| - ASSERT_EQ(1U,
|
| - manager()->session_tracker_.num_synced_tabs(std::string("tag")));
|
| -}
|
| -
|
| -namespace {
|
| -
|
| -class SyncedTabDelegateFake : public SyncedTabDelegate {
|
| - public:
|
| - SyncedTabDelegateFake()
|
| - : current_entry_index_(0), is_supervised_(false), sync_id_(-1) {}
|
| - ~SyncedTabDelegateFake() override {}
|
| -
|
| - bool IsInitialBlankNavigation() const override {
|
| - // This differs from NavigationControllerImpl, which has an initial blank
|
| - // NavigationEntry.
|
| - return GetEntryCount() == 0;
|
| - }
|
| - int GetCurrentEntryIndex() const override { return current_entry_index_; }
|
| - void set_current_entry_index(int i) {
|
| - current_entry_index_ = i;
|
| - }
|
| -
|
| - void AppendEntry(std::unique_ptr<content::NavigationEntry> entry) {
|
| - entries_.push_back(std::move(entry));
|
| - }
|
| -
|
| - GURL GetVirtualURLAtIndex(int i) const override {
|
| - if (static_cast<size_t>(i) >= entries_.size())
|
| - return GURL();
|
| - return entries_[i]->GetVirtualURL();
|
| - }
|
| -
|
| - GURL GetFaviconURLAtIndex(int i) const override { return GURL(); }
|
| -
|
| - ui::PageTransition GetTransitionAtIndex(int i) const override {
|
| - if (static_cast<size_t>(i) >= entries_.size())
|
| - return ui::PAGE_TRANSITION_LINK;
|
| - return entries_[i]->GetTransitionType();
|
| - }
|
| -
|
| - void GetSerializedNavigationAtIndex(
|
| - int i,
|
| - sessions::SerializedNavigationEntry* serialized_entry) const override {
|
| - if (static_cast<size_t>(i) >= entries_.size())
|
| - return;
|
| - *serialized_entry =
|
| - sessions::ContentSerializedNavigationBuilder::FromNavigationEntry(
|
| - i, *entries_[i]);
|
| - }
|
| -
|
| - int GetEntryCount() const override { return entries_.size(); }
|
| -
|
| - SessionID::id_type GetWindowId() const override {
|
| - return SessionID::id_type();
|
| - }
|
| -
|
| - SessionID::id_type GetSessionId() const override {
|
| - return SessionID::id_type();
|
| - }
|
| -
|
| - bool IsBeingDestroyed() const override { return false; }
|
| - std::string GetExtensionAppId() const override { return std::string(); }
|
| - bool ProfileIsSupervised() const override { return is_supervised_; }
|
| - void set_is_supervised(bool is_supervised) { is_supervised_ = is_supervised; }
|
| - const std::vector<std::unique_ptr<const sessions::SerializedNavigationEntry>>*
|
| - GetBlockedNavigations() const override {
|
| - return &blocked_navigations_;
|
| - }
|
| - void set_blocked_navigations(
|
| - std::vector<const content::NavigationEntry*>* navs) {
|
| - for (auto* entry : *navs) {
|
| - auto serialized_entry =
|
| - base::MakeUnique<sessions::SerializedNavigationEntry>(
|
| - sessions::ContentSerializedNavigationBuilder::FromNavigationEntry(
|
| - blocked_navigations_.size(), *entry));
|
| - blocked_navigations_.push_back(std::move(serialized_entry));
|
| - }
|
| - }
|
| - bool IsPlaceholderTab() const override { return true; }
|
| -
|
| - // Session sync related methods.
|
| - int GetSyncId() const override { return sync_id_; }
|
| - void SetSyncId(int sync_id) override { sync_id_ = sync_id; }
|
| -
|
| - bool ShouldSync(SyncSessionsClient* sessions_client) override {
|
| - return false;
|
| - }
|
| -
|
| - void reset() {
|
| - current_entry_index_ = 0;
|
| - sync_id_ = -1;
|
| - entries_.clear();
|
| - }
|
| -
|
| - private:
|
| - int current_entry_index_;
|
| - bool is_supervised_;
|
| - int sync_id_;
|
| - std::vector<std::unique_ptr<const sessions::SerializedNavigationEntry>>
|
| - blocked_navigations_;
|
| - std::vector<std::unique_ptr<content::NavigationEntry>> entries_;
|
| -};
|
| -
|
| -} // namespace
|
| -
|
| -static const base::Time kTime0 = base::Time::FromInternalValue(100);
|
| -static const base::Time kTime1 = base::Time::FromInternalValue(110);
|
| -static const base::Time kTime2 = base::Time::FromInternalValue(120);
|
| -static const base::Time kTime3 = base::Time::FromInternalValue(130);
|
| -static const base::Time kTime4 = base::Time::FromInternalValue(140);
|
| -static const base::Time kTime5 = base::Time::FromInternalValue(150);
|
| -static const base::Time kTime6 = base::Time::FromInternalValue(160);
|
| -static const base::Time kTime7 = base::Time::FromInternalValue(170);
|
| -static const base::Time kTime8 = base::Time::FromInternalValue(180);
|
| -static const base::Time kTime9 = base::Time::FromInternalValue(190);
|
| -
|
| -// Populate the mock tab delegate with some data and navigation
|
| -// entries and make sure that setting a SessionTab from it preserves
|
| -// those entries (and clobbers any existing data).
|
| -TEST_F(SessionsSyncManagerTest, SetSessionTabFromDelegate) {
|
| - // Create a tab with three valid entries.
|
| - SyncedTabDelegateFake tab;
|
| - std::unique_ptr<content::NavigationEntry> entry1(
|
| - content::NavigationEntry::Create());
|
| - GURL url1("http://www.google.com/");
|
| - entry1->SetVirtualURL(url1);
|
| - entry1->SetTimestamp(kTime1);
|
| - entry1->SetHttpStatusCode(200);
|
| - std::unique_ptr<content::NavigationEntry> entry2(
|
| - content::NavigationEntry::Create());
|
| - GURL url2("http://www.noodle.com/");
|
| - entry2->SetVirtualURL(url2);
|
| - entry2->SetTimestamp(kTime2);
|
| - entry2->SetHttpStatusCode(201);
|
| - std::unique_ptr<content::NavigationEntry> entry3(
|
| - content::NavigationEntry::Create());
|
| - GURL url3("http://www.doodle.com/");
|
| - entry3->SetVirtualURL(url3);
|
| - entry3->SetTimestamp(kTime3);
|
| - entry3->SetHttpStatusCode(202);
|
| -
|
| - tab.AppendEntry(std::move(entry1));
|
| - tab.AppendEntry(std::move(entry2));
|
| - tab.AppendEntry(std::move(entry3));
|
| - tab.set_current_entry_index(2);
|
| -
|
| - sessions::SessionTab session_tab;
|
| - session_tab.window_id.set_id(1);
|
| - session_tab.tab_id.set_id(1);
|
| - session_tab.tab_visual_index = 1;
|
| - session_tab.current_navigation_index = 1;
|
| - session_tab.pinned = true;
|
| - session_tab.extension_app_id = "app id";
|
| - session_tab.user_agent_override = "override";
|
| - session_tab.timestamp = kTime5;
|
| - session_tab.navigations.push_back(
|
| - SerializedNavigationEntryTestHelper::CreateNavigation(
|
| - "http://www.example.com", "Example"));
|
| - session_tab.session_storage_persistent_id = "persistent id";
|
| - manager()->SetSessionTabFromDelegate(tab, kTime4, &session_tab);
|
| -
|
| - EXPECT_EQ(0, session_tab.window_id.id());
|
| - EXPECT_EQ(0, session_tab.tab_id.id());
|
| - EXPECT_EQ(0, session_tab.tab_visual_index);
|
| - EXPECT_EQ(2, session_tab.current_navigation_index);
|
| - EXPECT_FALSE(session_tab.pinned);
|
| - EXPECT_TRUE(session_tab.extension_app_id.empty());
|
| - EXPECT_TRUE(session_tab.user_agent_override.empty());
|
| - EXPECT_EQ(kTime4, session_tab.timestamp);
|
| - ASSERT_EQ(3u, session_tab.navigations.size());
|
| - EXPECT_EQ(url1, session_tab.navigations[0].virtual_url());
|
| - EXPECT_EQ(url2, session_tab.navigations[1].virtual_url());
|
| - EXPECT_EQ(url3, session_tab.navigations[2].virtual_url());
|
| - EXPECT_EQ(kTime1, session_tab.navigations[0].timestamp());
|
| - EXPECT_EQ(kTime2, session_tab.navigations[1].timestamp());
|
| - EXPECT_EQ(kTime3, session_tab.navigations[2].timestamp());
|
| - EXPECT_EQ(200, session_tab.navigations[0].http_status_code());
|
| - EXPECT_EQ(201, session_tab.navigations[1].http_status_code());
|
| - EXPECT_EQ(202, session_tab.navigations[2].http_status_code());
|
| - EXPECT_EQ(SerializedNavigationEntry::STATE_INVALID,
|
| - session_tab.navigations[0].blocked_state());
|
| - EXPECT_EQ(SerializedNavigationEntry::STATE_INVALID,
|
| - session_tab.navigations[1].blocked_state());
|
| - EXPECT_EQ(SerializedNavigationEntry::STATE_INVALID,
|
| - session_tab.navigations[2].blocked_state());
|
| - EXPECT_TRUE(session_tab.session_storage_persistent_id.empty());
|
| -}
|
| -
|
| -// Ensure the current_navigation_index gets set properly when the navigation
|
| -// stack gets trucated to +/- 6 entries.
|
| -TEST_F(SessionsSyncManagerTest, SetSessionTabFromDelegateNavigationIndex) {
|
| - SyncedTabDelegateFake tab;
|
| - std::unique_ptr<content::NavigationEntry> entry0(
|
| - content::NavigationEntry::Create());
|
| - GURL url0("http://www.google.com/");
|
| - entry0->SetVirtualURL(url0);
|
| - entry0->SetTimestamp(kTime0);
|
| - entry0->SetHttpStatusCode(200);
|
| - std::unique_ptr<content::NavigationEntry> entry1(
|
| - content::NavigationEntry::Create());
|
| - GURL url1("http://www.zoogle.com/");
|
| - entry1->SetVirtualURL(url1);
|
| - entry1->SetTimestamp(kTime1);
|
| - entry1->SetHttpStatusCode(200);
|
| - std::unique_ptr<content::NavigationEntry> entry2(
|
| - content::NavigationEntry::Create());
|
| - GURL url2("http://www.noogle.com/");
|
| - entry2->SetVirtualURL(url2);
|
| - entry2->SetTimestamp(kTime2);
|
| - entry2->SetHttpStatusCode(200);
|
| - std::unique_ptr<content::NavigationEntry> entry3(
|
| - content::NavigationEntry::Create());
|
| - GURL url3("http://www.doogle.com/");
|
| - entry3->SetVirtualURL(url3);
|
| - entry3->SetTimestamp(kTime3);
|
| - entry3->SetHttpStatusCode(200);
|
| - std::unique_ptr<content::NavigationEntry> entry4(
|
| - content::NavigationEntry::Create());
|
| - GURL url4("http://www.yoogle.com/");
|
| - entry4->SetVirtualURL(url4);
|
| - entry4->SetTimestamp(kTime4);
|
| - entry4->SetHttpStatusCode(200);
|
| - std::unique_ptr<content::NavigationEntry> entry5(
|
| - content::NavigationEntry::Create());
|
| - GURL url5("http://www.foogle.com/");
|
| - entry5->SetVirtualURL(url5);
|
| - entry5->SetTimestamp(kTime5);
|
| - entry5->SetHttpStatusCode(200);
|
| - std::unique_ptr<content::NavigationEntry> entry6(
|
| - content::NavigationEntry::Create());
|
| - GURL url6("http://www.boogle.com/");
|
| - entry6->SetVirtualURL(url6);
|
| - entry6->SetTimestamp(kTime6);
|
| - entry6->SetHttpStatusCode(200);
|
| - std::unique_ptr<content::NavigationEntry> entry7(
|
| - content::NavigationEntry::Create());
|
| - GURL url7("http://www.moogle.com/");
|
| - entry7->SetVirtualURL(url7);
|
| - entry7->SetTimestamp(kTime7);
|
| - entry7->SetHttpStatusCode(200);
|
| - std::unique_ptr<content::NavigationEntry> entry8(
|
| - content::NavigationEntry::Create());
|
| - GURL url8("http://www.poogle.com/");
|
| - entry8->SetVirtualURL(url8);
|
| - entry8->SetTimestamp(kTime8);
|
| - entry8->SetHttpStatusCode(200);
|
| - std::unique_ptr<content::NavigationEntry> entry9(
|
| - content::NavigationEntry::Create());
|
| - GURL url9("http://www.roogle.com/");
|
| - entry9->SetVirtualURL(url9);
|
| - entry9->SetTimestamp(kTime9);
|
| - entry9->SetHttpStatusCode(200);
|
| -
|
| - tab.AppendEntry(std::move(entry0));
|
| - tab.AppendEntry(std::move(entry1));
|
| - tab.AppendEntry(std::move(entry2));
|
| - tab.AppendEntry(std::move(entry3));
|
| - tab.AppendEntry(std::move(entry4));
|
| - tab.AppendEntry(std::move(entry5));
|
| - tab.AppendEntry(std::move(entry6));
|
| - tab.AppendEntry(std::move(entry7));
|
| - tab.AppendEntry(std::move(entry8));
|
| - tab.AppendEntry(std::move(entry9));
|
| - tab.set_current_entry_index(8);
|
| -
|
| - sessions::SessionTab session_tab;
|
| - manager()->SetSessionTabFromDelegate(tab, kTime9, &session_tab);
|
| -
|
| - EXPECT_EQ(6, session_tab.current_navigation_index);
|
| - ASSERT_EQ(8u, session_tab.navigations.size());
|
| - EXPECT_EQ(url2, session_tab.navigations[0].virtual_url());
|
| - EXPECT_EQ(url3, session_tab.navigations[1].virtual_url());
|
| - EXPECT_EQ(url4, session_tab.navigations[2].virtual_url());
|
| -}
|
| -
|
| -// Ensure the current_navigation_index gets set to the end of the navigation
|
| -// stack if the current navigation is invalid.
|
| -TEST_F(SessionsSyncManagerTest, SetSessionTabFromDelegateCurrentInvalid) {
|
| - SyncedTabDelegateFake tab;
|
| - std::unique_ptr<content::NavigationEntry> entry0(
|
| - content::NavigationEntry::Create());
|
| - entry0->SetVirtualURL(GURL("http://www.google.com"));
|
| - entry0->SetTimestamp(kTime0);
|
| - entry0->SetHttpStatusCode(200);
|
| - std::unique_ptr<content::NavigationEntry> entry1(
|
| - content::NavigationEntry::Create());
|
| - entry1->SetVirtualURL(GURL(""));
|
| - entry1->SetTimestamp(kTime1);
|
| - entry1->SetHttpStatusCode(200);
|
| - std::unique_ptr<content::NavigationEntry> entry2(
|
| - content::NavigationEntry::Create());
|
| - entry2->SetVirtualURL(GURL("http://www.noogle.com"));
|
| - entry2->SetTimestamp(kTime2);
|
| - entry2->SetHttpStatusCode(200);
|
| - std::unique_ptr<content::NavigationEntry> entry3(
|
| - content::NavigationEntry::Create());
|
| - entry3->SetVirtualURL(GURL("http://www.doogle.com"));
|
| - entry3->SetTimestamp(kTime3);
|
| - entry3->SetHttpStatusCode(200);
|
| -
|
| - tab.AppendEntry(std::move(entry0));
|
| - tab.AppendEntry(std::move(entry1));
|
| - tab.AppendEntry(std::move(entry2));
|
| - tab.AppendEntry(std::move(entry3));
|
| - tab.set_current_entry_index(1);
|
| -
|
| - sessions::SessionTab session_tab;
|
| - manager()->SetSessionTabFromDelegate(tab, kTime9, &session_tab);
|
| -
|
| - EXPECT_EQ(2, session_tab.current_navigation_index);
|
| - ASSERT_EQ(3u, session_tab.navigations.size());
|
| -}
|
| -
|
| -// Tests that variation ids are set correctly.
|
| -TEST_F(SessionsSyncManagerTest, SetVariationIds) {
|
| - // Create two trials with a group which has a variation id for Chrome Sync
|
| - // and one with a variation id for another service.
|
| - const variations::VariationID kVariationId1 = 3300200;
|
| - const variations::VariationID kVariationId2 = 3300300;
|
| - const variations::VariationID kVariationId3 = 3300400;
|
| -
|
| - base::FieldTrialList field_trial_list(nullptr);
|
| - CreateAndActivateFieldTrial("trial name 1", "group name", kVariationId1,
|
| - variations::CHROME_SYNC_SERVICE);
|
| - CreateAndActivateFieldTrial("trial name 2", "group name", kVariationId2,
|
| - variations::CHROME_SYNC_SERVICE);
|
| - CreateAndActivateFieldTrial("trial name 3", "group name", kVariationId3,
|
| - variations::GOOGLE_WEB_PROPERTIES);
|
| -
|
| - sessions::SessionTab session_tab;
|
| - manager()->SetVariationIds(&session_tab);
|
| -
|
| - ASSERT_EQ(2u, session_tab.variation_ids.size());
|
| - EXPECT_EQ(kVariationId1, session_tab.variation_ids[0]);
|
| - EXPECT_EQ(kVariationId2, session_tab.variation_ids[1]);
|
| -}
|
| -
|
| -// Tests that for supervised users blocked navigations are recorded and marked
|
| -// as such, while regular navigations are marked as allowed.
|
| -TEST_F(SessionsSyncManagerTest, BlockedNavigations) {
|
| - SyncedTabDelegateFake tab;
|
| - std::unique_ptr<content::NavigationEntry> entry1(
|
| - content::NavigationEntry::Create());
|
| - GURL url1("http://www.google.com/");
|
| - entry1->SetVirtualURL(url1);
|
| - entry1->SetTimestamp(kTime1);
|
| - tab.AppendEntry(std::move(entry1));
|
| -
|
| - std::unique_ptr<content::NavigationEntry> entry2(
|
| - content::NavigationEntry::Create());
|
| - GURL url2("http://blocked.com/foo");
|
| - entry2->SetVirtualURL(url2);
|
| - entry2->SetTimestamp(kTime2);
|
| - std::unique_ptr<content::NavigationEntry> entry3(
|
| - content::NavigationEntry::Create());
|
| - GURL url3("http://evil.com/");
|
| - entry3->SetVirtualURL(url3);
|
| - entry3->SetTimestamp(kTime3);
|
| - ScopedVector<const content::NavigationEntry> blocked_navigations;
|
| - blocked_navigations.push_back(std::move(entry2));
|
| - blocked_navigations.push_back(std::move(entry3));
|
| -
|
| - tab.set_is_supervised(true);
|
| - tab.set_blocked_navigations(&blocked_navigations.get());
|
| -
|
| - sessions::SessionTab session_tab;
|
| - session_tab.window_id.set_id(1);
|
| - session_tab.tab_id.set_id(1);
|
| - session_tab.tab_visual_index = 1;
|
| - session_tab.current_navigation_index = 1;
|
| - session_tab.pinned = true;
|
| - session_tab.extension_app_id = "app id";
|
| - session_tab.user_agent_override = "override";
|
| - session_tab.timestamp = kTime5;
|
| - session_tab.navigations.push_back(
|
| - SerializedNavigationEntryTestHelper::CreateNavigation(
|
| - "http://www.example.com", "Example"));
|
| - session_tab.session_storage_persistent_id = "persistent id";
|
| - manager()->SetSessionTabFromDelegate(tab, kTime4, &session_tab);
|
| -
|
| - EXPECT_EQ(0, session_tab.window_id.id());
|
| - EXPECT_EQ(0, session_tab.tab_id.id());
|
| - EXPECT_EQ(0, session_tab.tab_visual_index);
|
| - EXPECT_EQ(0, session_tab.current_navigation_index);
|
| - EXPECT_FALSE(session_tab.pinned);
|
| - EXPECT_TRUE(session_tab.extension_app_id.empty());
|
| - EXPECT_TRUE(session_tab.user_agent_override.empty());
|
| - EXPECT_EQ(kTime4, session_tab.timestamp);
|
| - ASSERT_EQ(3u, session_tab.navigations.size());
|
| - EXPECT_EQ(url1, session_tab.navigations[0].virtual_url());
|
| - EXPECT_EQ(url2, session_tab.navigations[1].virtual_url());
|
| - EXPECT_EQ(url3, session_tab.navigations[2].virtual_url());
|
| - EXPECT_EQ(kTime1, session_tab.navigations[0].timestamp());
|
| - EXPECT_EQ(kTime2, session_tab.navigations[1].timestamp());
|
| - EXPECT_EQ(kTime3, session_tab.navigations[2].timestamp());
|
| - EXPECT_EQ(SerializedNavigationEntry::STATE_ALLOWED,
|
| - session_tab.navigations[0].blocked_state());
|
| - EXPECT_EQ(SerializedNavigationEntry::STATE_BLOCKED,
|
| - session_tab.navigations[1].blocked_state());
|
| - EXPECT_EQ(SerializedNavigationEntry::STATE_BLOCKED,
|
| - session_tab.navigations[2].blocked_state());
|
| - EXPECT_TRUE(session_tab.session_storage_persistent_id.empty());
|
| -}
|
| -
|
| -// Tests that the local session header objects is created properly in
|
| -// presence of no other session activity, once and only once.
|
| -TEST_F(SessionsSyncManagerTest, MergeLocalSessionNoTabs) {
|
| - syncer::SyncChangeList out;
|
| - InitWithSyncDataTakeOutput(syncer::SyncDataList(), &out);
|
| - EXPECT_FALSE(manager()->current_machine_tag().empty());
|
| -
|
| - EXPECT_EQ(2U, out.size());
|
| - EXPECT_TRUE(out[0].IsValid());
|
| - EXPECT_EQ(SyncChange::ACTION_ADD, out[0].change_type());
|
| - const SyncData data(out[0].sync_data());
|
| - EXPECT_EQ(manager()->current_machine_tag(),
|
| - syncer::SyncDataLocal(data).GetTag());
|
| - const sync_pb::SessionSpecifics& specifics(data.GetSpecifics().session());
|
| - EXPECT_EQ(manager()->current_machine_tag(), specifics.session_tag());
|
| - EXPECT_TRUE(specifics.has_header());
|
| - const sync_pb::SessionHeader& header_s = specifics.header();
|
| - EXPECT_TRUE(header_s.has_device_type());
|
| - EXPECT_EQ(GetLocalDeviceInfo()->client_name(), header_s.client_name());
|
| - EXPECT_EQ(0, header_s.window_size());
|
| -
|
| - EXPECT_TRUE(out[1].IsValid());
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[1].change_type());
|
| - const SyncData data_2(out[1].sync_data());
|
| - EXPECT_EQ(manager()->current_machine_tag(),
|
| - syncer::SyncDataLocal(data_2).GetTag());
|
| - const sync_pb::SessionSpecifics& specifics2(data_2.GetSpecifics().session());
|
| - EXPECT_EQ(manager()->current_machine_tag(), specifics2.session_tag());
|
| - EXPECT_TRUE(specifics2.has_header());
|
| - const sync_pb::SessionHeader& header_s2 = specifics2.header();
|
| - EXPECT_EQ(0, header_s2.window_size());
|
| -
|
| - // Now take that header node and feed it in as input.
|
| - SyncData d = CreateRemoteData(data.GetSpecifics());
|
| - syncer::SyncDataList in(&d, &d + 1);
|
| - out.clear();
|
| - SessionsSyncManager manager2(GetSyncSessionsClient(), sync_prefs(),
|
| - local_device(), NewDummyRouter(),
|
| - base::Closure(), base::Closure());
|
| - syncer::SyncMergeResult result = manager2.MergeDataAndStartSyncing(
|
| - syncer::SESSIONS, in, std::unique_ptr<syncer::SyncChangeProcessor>(
|
| - new TestSyncProcessorStub(&out)),
|
| - std::unique_ptr<syncer::SyncErrorFactory>(
|
| - new syncer::SyncErrorFactoryMock()));
|
| - ASSERT_FALSE(result.error().IsSet());
|
| -
|
| - EXPECT_EQ(1U, out.size());
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[0].change_type());
|
| - EXPECT_TRUE(out[0].sync_data().GetSpecifics().session().has_header());
|
| -}
|
| -
|
| -// Ensure model association associates the pre-existing tabs.
|
| -TEST_F(SessionsSyncManagerTest, SwappedOutOnRestore) {
|
| - AddTab(browser(), GURL("http://foo1"));
|
| - NavigateAndCommitActiveTab(GURL("http://foo2"));
|
| - AddTab(browser(), GURL("http://bar1"));
|
| - NavigateAndCommitActiveTab(GURL("http://bar2"));
|
| - AddTab(browser(), GURL("http://baz1"));
|
| - NavigateAndCommitActiveTab(GURL("http://baz2"));
|
| - const int kRestoredTabId = 1337;
|
| - const int kNewTabId = 2468;
|
| -
|
| - syncer::SyncDataList in;
|
| - syncer::SyncChangeList out;
|
| - InitWithSyncDataTakeOutput(in, &out);
|
| -
|
| - // Should be one header add, 3 tab add/update pairs, one header update.
|
| - ASSERT_EQ(8U, out.size());
|
| -
|
| - // For input, we set up:
|
| - // * one "normal" fully loaded tab
|
| - // * one "frozen" tab with no WebContents and a tab_id change
|
| - // * one "frozen" tab with no WebContents and no tab_id change
|
| - sync_pb::EntitySpecifics t0_entity = out[2].sync_data().GetSpecifics();
|
| - sync_pb::EntitySpecifics t1_entity = out[4].sync_data().GetSpecifics();
|
| - sync_pb::EntitySpecifics t2_entity = out[6].sync_data().GetSpecifics();
|
| - t1_entity.mutable_session()->mutable_tab()->set_tab_id(kRestoredTabId);
|
| - in.push_back(CreateRemoteData(t0_entity));
|
| - in.push_back(CreateRemoteData(t1_entity));
|
| - in.push_back(CreateRemoteData(t2_entity));
|
| - out.clear();
|
| - manager()->StopSyncing(syncer::SESSIONS);
|
| -
|
| - const std::set<const SyncedWindowDelegate*>& windows =
|
| - manager()->synced_window_delegates_getter()->GetSyncedWindowDelegates();
|
| - ASSERT_EQ(1U, windows.size());
|
| - SyncedTabDelegateFake t1_override, t2_override;
|
| - t1_override.SetSyncId(1); // No WebContents by default.
|
| - t2_override.SetSyncId(2); // No WebContents by default.
|
| - SyncedWindowDelegateOverride window_override(*windows.begin());
|
| - window_override.OverrideTabAt(1, &t1_override, kNewTabId);
|
| - window_override.OverrideTabAt(2, &t2_override,
|
| - t2_entity.session().tab().tab_id());
|
| - std::set<const SyncedWindowDelegate*> delegates;
|
| - delegates.insert(&window_override);
|
| - std::unique_ptr<TestSyncedWindowDelegatesGetter> getter(
|
| - new TestSyncedWindowDelegatesGetter(delegates));
|
| - set_synced_window_getter(getter.get());
|
| -
|
| - syncer::SyncMergeResult result = manager()->MergeDataAndStartSyncing(
|
| - syncer::SESSIONS, in, std::unique_ptr<syncer::SyncChangeProcessor>(
|
| - new TestSyncProcessorStub(&out)),
|
| - std::unique_ptr<syncer::SyncErrorFactory>(
|
| - new syncer::SyncErrorFactoryMock()));
|
| -
|
| - // There should be two changes, one for the fully associated tab, and
|
| - // one for the tab_id update to t1. t2 shouldn't need to be updated.
|
| - ASSERT_EQ(2U, FilterOutLocalHeaderChanges(&out)->size());
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[0].change_type());
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[1].change_type());
|
| - EXPECT_EQ(kNewTabId,
|
| - out[1].sync_data().GetSpecifics().session().tab().tab_id());
|
| -
|
| - // Verify TabLinks.
|
| - SessionsSyncManager::TabLinksMap tab_map = manager()->local_tab_map_;
|
| - ASSERT_EQ(3U, tab_map.size());
|
| - int t2_tab_id = t2_entity.session().tab().tab_id();
|
| - EXPECT_EQ(2, tab_map.find(t2_tab_id)->second->tab_node_id());
|
| - EXPECT_EQ(1, tab_map.find(kNewTabId)->second->tab_node_id());
|
| - int t0_tab_id = out[0].sync_data().GetSpecifics().session().tab().tab_id();
|
| - EXPECT_EQ(0, tab_map.find(t0_tab_id)->second->tab_node_id());
|
| - // TODO(tim): Once bug 337057 is fixed, we can issue an OnLocalTabModified
|
| - // from here (using an override similar to above) to return a new tab id
|
| - // and verify that we don't see any node creations in the SyncChangeProcessor
|
| - // (similar to how SessionsSyncManagerTest.OnLocalTabModified works.)
|
| -}
|
| -
|
| -// Ensure model association updates the window ID for tabs whose window's ID has
|
| -// changed.
|
| -TEST_F(SessionsSyncManagerTest, WindowIdUpdatedOnRestore) {
|
| - const int kNewWindowId = 1337;
|
| - syncer::SyncDataList in;
|
| - syncer::SyncChangeList out;
|
| -
|
| - // Set up one tab and start sync with it.
|
| - AddTab(browser(), GURL("http://foo1"));
|
| - NavigateAndCommitActiveTab(GURL("http://foo2"));
|
| - InitWithSyncDataTakeOutput(in, &out);
|
| -
|
| - // Should be one header add, 1 tab add/update pair, and one header update.
|
| - ASSERT_EQ(4U, out.size());
|
| - const sync_pb::EntitySpecifics t0_entity = out[2].sync_data().GetSpecifics();
|
| -
|
| - in.push_back(CreateRemoteData(t0_entity));
|
| - out.clear();
|
| - manager()->StopSyncing(syncer::SESSIONS);
|
| -
|
| - // SyncedTabDelegateFake is a placeholder (no WebContents) by default.
|
| - SyncedTabDelegateFake t0_override;
|
| - t0_override.SetSyncId(t0_entity.session().tab_node_id());
|
| -
|
| - // Set up the window override with the new window ID and placeholder tab.
|
| - const std::set<const SyncedWindowDelegate*>& windows =
|
| - get_synced_window_getter()->GetSyncedWindowDelegates();
|
| - ASSERT_EQ(1U, windows.size());
|
| - SyncedWindowDelegateOverride window_override(*windows.begin());
|
| - window_override.OverrideSessionId(kNewWindowId);
|
| - window_override.OverrideTabAt(0, &t0_override,
|
| - t0_entity.session().tab().tab_id());
|
| -
|
| - // Inject the window override.
|
| - std::set<const SyncedWindowDelegate*> delegates;
|
| - delegates.insert(&window_override);
|
| - std::unique_ptr<TestSyncedWindowDelegatesGetter> getter(
|
| - new TestSyncedWindowDelegatesGetter(delegates));
|
| - set_synced_window_getter(getter.get());
|
| -
|
| - syncer::SyncMergeResult result = manager()->MergeDataAndStartSyncing(
|
| - syncer::SESSIONS, in, std::unique_ptr<syncer::SyncChangeProcessor>(
|
| - new TestSyncProcessorStub(&out)),
|
| - std::unique_ptr<syncer::SyncErrorFactory>(
|
| - new syncer::SyncErrorFactoryMock()));
|
| -
|
| - // There should be one change for t0's window ID update.
|
| - ASSERT_EQ(1U, FilterOutLocalHeaderChanges(&out)->size());
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[0].change_type());
|
| - EXPECT_EQ(kNewWindowId,
|
| - out[0].sync_data().GetSpecifics().session().tab().window_id());
|
| -}
|
| -
|
| -// Tests MergeDataAndStartSyncing with sync data but no local data.
|
| -TEST_F(SessionsSyncManagerTest, MergeWithInitialForeignSession) {
|
| - std::string tag = "tag1";
|
| -
|
| - SessionID::id_type n1[] = {5, 10, 13, 17};
|
| - std::vector<SessionID::id_type> tab_list1(n1, n1 + arraysize(n1));
|
| - std::vector<sync_pb::SessionSpecifics> tabs1;
|
| - sync_pb::SessionSpecifics meta(helper()->BuildForeignSession(
|
| - tag, tab_list1, &tabs1));
|
| - // Add a second window.
|
| - SessionID::id_type n2[] = {7, 15, 18, 20};
|
| - std::vector<SessionID::id_type> tab_list2(n2, n2 + arraysize(n2));
|
| - helper()->AddWindowSpecifics(1, tab_list2, &meta);
|
| -
|
| - // Set up initial data.
|
| - syncer::SyncDataList initial_data;
|
| - initial_data.push_back(CreateRemoteData(meta));
|
| - AddTabsToSyncDataList(tabs1, &initial_data);
|
| -
|
| - for (size_t i = 0; i < tab_list2.size(); ++i) {
|
| - sync_pb::EntitySpecifics entity;
|
| - helper()->BuildTabSpecifics(tag, 0, tab_list2[i],
|
| - entity.mutable_session());
|
| - initial_data.push_back(CreateRemoteData(entity));
|
| - }
|
| -
|
| - syncer::SyncChangeList output;
|
| - InitWithSyncDataTakeOutput(initial_data, &output);
|
| - EXPECT_TRUE(FilterOutLocalHeaderChanges(&output)->empty());
|
| -
|
| - std::vector<const SyncedSession*> foreign_sessions;
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - ASSERT_EQ(1U, foreign_sessions.size());
|
| - std::vector<std::vector<SessionID::id_type> > session_reference;
|
| - session_reference.push_back(tab_list1);
|
| - session_reference.push_back(tab_list2);
|
| - helper()->VerifySyncedSession(tag, session_reference, *(foreign_sessions[0]));
|
| -}
|
| -
|
| -// This is a combination of MergeWithInitialForeignSession and
|
| -// MergeLocalSessionExistingTabs. We repeat some checks performed in each of
|
| -// those tests to ensure the common mixed scenario works.
|
| -TEST_F(SessionsSyncManagerTest, MergeWithLocalAndForeignTabs) {
|
| - // Local.
|
| - AddTab(browser(), GURL("http://foo1"));
|
| - NavigateAndCommitActiveTab(GURL("http://foo2"));
|
| -
|
| - // Foreign.
|
| - std::string tag = "tag1";
|
| - SessionID::id_type n1[] = {5, 10, 13, 17};
|
| - std::vector<SessionID::id_type> tab_list1(n1, n1 + arraysize(n1));
|
| - std::vector<sync_pb::SessionSpecifics> tabs1;
|
| - sync_pb::SessionSpecifics meta(helper()->BuildForeignSession(
|
| - tag, tab_list1, &tabs1));
|
| - syncer::SyncDataList foreign_data;
|
| - foreign_data.push_back(CreateRemoteData(meta));
|
| - AddTabsToSyncDataList(tabs1, &foreign_data);
|
| -
|
| - syncer::SyncChangeList output;
|
| - InitWithSyncDataTakeOutput(foreign_data, &output);
|
| - ASSERT_EQ(4U, output.size());
|
| -
|
| - // Verify the local header.
|
| - EXPECT_TRUE(output[0].IsValid());
|
| - EXPECT_EQ(SyncChange::ACTION_ADD, output[0].change_type());
|
| - const SyncData data(output[0].sync_data());
|
| - EXPECT_EQ(manager()->current_machine_tag(),
|
| - syncer::SyncDataLocal(data).GetTag());
|
| - const sync_pb::SessionSpecifics& specifics(data.GetSpecifics().session());
|
| - EXPECT_EQ(manager()->current_machine_tag(), specifics.session_tag());
|
| - EXPECT_TRUE(specifics.has_header());
|
| - const sync_pb::SessionHeader& header_s = specifics.header();
|
| - EXPECT_TRUE(header_s.has_device_type());
|
| - EXPECT_EQ(GetLocalDeviceInfo()->client_name(), header_s.client_name());
|
| - EXPECT_EQ(0, header_s.window_size());
|
| -
|
| - // Verify the tab node creations and updates with content.
|
| - for (int i = 1; i < 3; i++) {
|
| - EXPECT_TRUE(output[i].IsValid());
|
| - const SyncData data(output[i].sync_data());
|
| - EXPECT_TRUE(base::StartsWith(syncer::SyncDataLocal(data).GetTag(),
|
| - manager()->current_machine_tag(),
|
| - base::CompareCase::SENSITIVE));
|
| - const sync_pb::SessionSpecifics& specifics(data.GetSpecifics().session());
|
| - EXPECT_EQ(manager()->current_machine_tag(), specifics.session_tag());
|
| - }
|
| - EXPECT_EQ(SyncChange::ACTION_ADD, output[1].change_type());
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, output[2].change_type());
|
| - EXPECT_TRUE(output[2].sync_data().GetSpecifics().session().has_tab());
|
| -
|
| - // Verify the header was updated to reflect window state.
|
| - EXPECT_TRUE(output[3].IsValid());
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, output[3].change_type());
|
| - const SyncData data_2(output[3].sync_data());
|
| - EXPECT_EQ(manager()->current_machine_tag(),
|
| - syncer::SyncDataLocal(data_2).GetTag());
|
| - const sync_pb::SessionSpecifics& specifics2(data_2.GetSpecifics().session());
|
| - EXPECT_EQ(manager()->current_machine_tag(), specifics2.session_tag());
|
| - EXPECT_TRUE(specifics2.has_header());
|
| - const sync_pb::SessionHeader& header_s2 = specifics2.header();
|
| - EXPECT_EQ(1, header_s2.window_size());
|
| -
|
| - // Verify foreign data.
|
| - std::vector<const SyncedSession*> foreign_sessions;
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - std::vector<std::vector<SessionID::id_type> > session_reference;
|
| - session_reference.push_back(tab_list1);
|
| - helper()->VerifySyncedSession(tag, session_reference, *(foreign_sessions[0]));
|
| - // There should be one and only one foreign session. If VerifySyncedSession
|
| - // was successful above this EXPECT call ensures the local session didn't
|
| - // get mistakenly added to foreign tracking (Similar to ExistingTabs test).
|
| - EXPECT_EQ(1U, foreign_sessions.size());
|
| -}
|
| -
|
| -// Tests the common scenario. Merge with both local and foreign session data
|
| -// followed by updates flowing from sync and local.
|
| -TEST_F(SessionsSyncManagerTest, UpdatesAfterMixedMerge) {
|
| - // Add local and foreign data.
|
| - AddTab(browser(), GURL("http://foo1"));
|
| - NavigateAndCommitActiveTab(GURL("http://foo2"));
|
| -
|
| - std::string tag1 = "tag1";
|
| - syncer::SyncDataList foreign_data1;
|
| - std::vector<std::vector<SessionID::id_type> > meta1_reference;
|
| - sync_pb::SessionSpecifics meta1;
|
| -
|
| - SessionID::id_type n1[] = {5, 10, 13, 17};
|
| - std::vector<SessionID::id_type> tab_list1(n1, n1 + arraysize(n1));
|
| - meta1_reference.push_back(tab_list1);
|
| - std::vector<sync_pb::SessionSpecifics> tabs1;
|
| - meta1 = helper()->BuildForeignSession(tag1, tab_list1, &tabs1);
|
| - foreign_data1.push_back(CreateRemoteData(meta1));
|
| - AddTabsToSyncDataList(tabs1, &foreign_data1);
|
| -
|
| - syncer::SyncChangeList output1;
|
| - InitWithSyncDataTakeOutput(foreign_data1, &output1);
|
| - ASSERT_EQ(4U, output1.size());
|
| -
|
| - // Add a second window to the foreign session.
|
| - // TODO(tim): Bug 98892. Add local window too when observers are hooked up.
|
| - SessionID::id_type tab_nums2[] = {7, 15, 18, 20};
|
| - std::vector<SessionID::id_type> tab_list2(
|
| - tab_nums2, tab_nums2 + arraysize(tab_nums2));
|
| - meta1_reference.push_back(tab_list2);
|
| - helper()->AddWindowSpecifics(1, tab_list2, &meta1);
|
| - std::vector<sync_pb::SessionSpecifics> tabs2;
|
| - tabs2.resize(tab_list2.size());
|
| - for (size_t i = 0; i < tab_list2.size(); ++i) {
|
| - helper()->BuildTabSpecifics(tag1, 0, tab_list2[i], &tabs2[i]);
|
| - }
|
| -
|
| - syncer::SyncChangeList changes;
|
| - changes.push_back(MakeRemoteChange(meta1, SyncChange::ACTION_UPDATE));
|
| - AddTabsToChangeList(tabs2, SyncChange::ACTION_ADD, &changes);
|
| - manager()->ProcessSyncChanges(FROM_HERE, changes);
|
| - changes.clear();
|
| -
|
| - // Check that the foreign session was associated and retrieve the data.
|
| - std::vector<const SyncedSession*> foreign_sessions;
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - ASSERT_EQ(1U, foreign_sessions.size());
|
| - ASSERT_EQ(4U, foreign_sessions[0]->windows.find(0)->second->tabs.size());
|
| - ASSERT_EQ(4U, foreign_sessions[0]->windows.find(1)->second->tabs.size());
|
| - helper()->VerifySyncedSession(tag1, meta1_reference, *(foreign_sessions[0]));
|
| -
|
| - // Add a new foreign session.
|
| - std::string tag2 = "tag2";
|
| - SessionID::id_type n2[] = {107, 115};
|
| - std::vector<SessionID::id_type> tag2_tab_list(n2, n2 + arraysize(n2));
|
| - std::vector<sync_pb::SessionSpecifics> tag2_tabs;
|
| - sync_pb::SessionSpecifics meta2(helper()->BuildForeignSession(
|
| - tag2, tag2_tab_list, &tag2_tabs));
|
| - changes.push_back(MakeRemoteChange(meta2, SyncChange::ACTION_ADD));
|
| - AddTabsToChangeList(tag2_tabs, SyncChange::ACTION_ADD, &changes);
|
| -
|
| - manager()->ProcessSyncChanges(FROM_HERE, changes);
|
| - changes.clear();
|
| -
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - std::vector<std::vector<SessionID::id_type> > meta2_reference;
|
| - meta2_reference.push_back(tag2_tab_list);
|
| - ASSERT_EQ(2U, foreign_sessions.size());
|
| - ASSERT_EQ(2U, foreign_sessions[1]->windows.find(0)->second->tabs.size());
|
| - helper()->VerifySyncedSession(tag2, meta2_reference, *(foreign_sessions[1]));
|
| - foreign_sessions.clear();
|
| -
|
| - // Remove a tab from a window.
|
| - meta1_reference[0].pop_back();
|
| - tab_list1.pop_back();
|
| - sync_pb::SessionWindow* win = meta1.mutable_header()->mutable_window(0);
|
| - win->clear_tab();
|
| - for (std::vector<int>::const_iterator iter = tab_list1.begin();
|
| - iter != tab_list1.end(); ++iter) {
|
| - win->add_tab(*iter);
|
| - }
|
| - syncer::SyncChangeList removal;
|
| - removal.push_back(MakeRemoteChange(meta1, SyncChange::ACTION_UPDATE));
|
| - AddTabsToChangeList(tabs1, SyncChange::ACTION_UPDATE, &removal);
|
| - manager()->ProcessSyncChanges(FROM_HERE, removal);
|
| -
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - ASSERT_EQ(2U, foreign_sessions.size());
|
| - ASSERT_EQ(3U, foreign_sessions[0]->windows.find(0)->second->tabs.size());
|
| - helper()->VerifySyncedSession(tag1, meta1_reference, *(foreign_sessions[0]));
|
| -}
|
| -
|
| -// Tests that this SyncSessionManager knows how to delete foreign sessions
|
| -// if it wants to.
|
| -TEST_F(SessionsSyncManagerTest, DeleteForeignSession) {
|
| - InitWithNoSyncData();
|
| - std::string tag = "tag1";
|
| - syncer::SyncChangeList changes;
|
| -
|
| - std::vector<const SyncedSession*> foreign_sessions;
|
| - ASSERT_FALSE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - manager()->DeleteForeignSessionInternal(tag, &changes);
|
| - ASSERT_FALSE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - EXPECT_TRUE(changes.empty());
|
| -
|
| - // Fill an instance of session specifics with a foreign session's data.
|
| - std::vector<sync_pb::SessionSpecifics> tabs;
|
| - SessionID::id_type n1[] = {5, 10, 13, 17};
|
| - std::vector<SessionID::id_type> tab_nums1(n1, n1 + arraysize(n1));
|
| - sync_pb::SessionSpecifics meta(helper()->BuildForeignSession(
|
| - tag, tab_nums1, &tabs));
|
| -
|
| - // Update associator with the session's meta node, window, and tabs.
|
| - manager()->UpdateTrackerWithForeignSession(meta, base::Time());
|
| - for (std::vector<sync_pb::SessionSpecifics>::iterator iter = tabs.begin();
|
| - iter != tabs.end(); ++iter) {
|
| - manager()->UpdateTrackerWithForeignSession(*iter, base::Time());
|
| - }
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - ASSERT_EQ(1U, foreign_sessions.size());
|
| -
|
| - // Now delete the foreign session.
|
| - manager()->DeleteForeignSessionInternal(tag, &changes);
|
| - EXPECT_FALSE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| -
|
| - EXPECT_EQ(5U, changes.size());
|
| - std::set<std::string> expected_tags(&tag, &tag + 1);
|
| - for (int i = 0; i < 5; i++)
|
| - expected_tags.insert(TabNodePool::TabIdToTag(tag, i));
|
| -
|
| - for (int i = 0; i < 5; i++) {
|
| - SCOPED_TRACE(changes[i].ToString());
|
| - EXPECT_TRUE(changes[i].IsValid());
|
| - EXPECT_EQ(SyncChange::ACTION_DELETE, changes[i].change_type());
|
| - EXPECT_TRUE(changes[i].sync_data().IsValid());
|
| - EXPECT_EQ(1U,
|
| - expected_tags.erase(
|
| - syncer::SyncDataLocal(changes[i].sync_data()).GetTag()));
|
| - }
|
| -}
|
| -
|
| -// Write a foreign session to a node, with the tabs arriving first, and then
|
| -// retrieve it.
|
| -TEST_F(SessionsSyncManagerTest, WriteForeignSessionToNodeTabsFirst) {
|
| - InitWithNoSyncData();
|
| -
|
| - // Fill an instance of session specifics with a foreign session's data.
|
| - std::string tag = "tag1";
|
| - SessionID::id_type nums1[] = {5, 10, 13, 17};
|
| - std::vector<sync_pb::SessionSpecifics> tabs1;
|
| - std::vector<SessionID::id_type> tab_list1(nums1, nums1 + arraysize(nums1));
|
| - sync_pb::SessionSpecifics meta(helper()->BuildForeignSession(
|
| - tag, tab_list1, &tabs1));
|
| -
|
| - syncer::SyncChangeList adds;
|
| - // Add tabs for first window, then the meta node.
|
| - AddTabsToChangeList(tabs1, SyncChange::ACTION_ADD, &adds);
|
| - adds.push_back(MakeRemoteChange(meta, SyncChange::ACTION_ADD));
|
| - manager()->ProcessSyncChanges(FROM_HERE, adds);
|
| -
|
| - // Check that the foreign session was associated and retrieve the data.
|
| - std::vector<const SyncedSession*> foreign_sessions;
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - ASSERT_EQ(1U, foreign_sessions.size());
|
| - std::vector<std::vector<SessionID::id_type> > session_reference;
|
| - session_reference.push_back(tab_list1);
|
| - helper()->VerifySyncedSession(tag, session_reference, *(foreign_sessions[0]));
|
| -}
|
| -
|
| -// Write a foreign session to a node with some tabs that never arrive.
|
| -TEST_F(SessionsSyncManagerTest, WriteForeignSessionToNodeMissingTabs) {
|
| - InitWithNoSyncData();
|
| -
|
| - // Fill an instance of session specifics with a foreign session's data.
|
| - std::string tag = "tag1";
|
| - SessionID::id_type nums1[] = {5, 10, 13, 17};
|
| - std::vector<sync_pb::SessionSpecifics> tabs1;
|
| - std::vector<SessionID::id_type> tab_list1(nums1, nums1 + arraysize(nums1));
|
| - sync_pb::SessionSpecifics meta(helper()->BuildForeignSession(
|
| - tag, tab_list1, &tabs1));
|
| - // Add a second window, but this time only create two tab nodes, despite the
|
| - // window expecting four tabs.
|
| - SessionID::id_type tab_nums2[] = {7, 15, 18, 20};
|
| - std::vector<SessionID::id_type> tab_list2(
|
| - tab_nums2, tab_nums2 + arraysize(tab_nums2));
|
| - helper()->AddWindowSpecifics(1, tab_list2, &meta);
|
| - std::vector<sync_pb::SessionSpecifics> tabs2;
|
| - tabs2.resize(2);
|
| - for (size_t i = 0; i < 2; ++i) {
|
| - helper()->BuildTabSpecifics(tag, 0, tab_list2[i], &tabs2[i]);
|
| - }
|
| -
|
| - syncer::SyncChangeList changes;
|
| - changes.push_back(MakeRemoteChange(meta, SyncChange::ACTION_ADD));
|
| - AddTabsToChangeList(tabs1, SyncChange::ACTION_ADD, &changes);
|
| - AddTabsToChangeList(tabs2, SyncChange::ACTION_ADD, &changes);
|
| - manager()->ProcessSyncChanges(FROM_HERE, changes);
|
| - changes.clear();
|
| -
|
| - // Check that the foreign session was associated and retrieve the data.
|
| - std::vector<const SyncedSession*> foreign_sessions;
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - ASSERT_EQ(1U, foreign_sessions.size());
|
| - ASSERT_EQ(2U, foreign_sessions[0]->windows.size());
|
| - ASSERT_EQ(4U, foreign_sessions[0]->windows.find(0)->second->tabs.size());
|
| - ASSERT_EQ(4U, foreign_sessions[0]->windows.find(1)->second->tabs.size());
|
| -
|
| - // Close the second window.
|
| - meta.mutable_header()->clear_window();
|
| - helper()->AddWindowSpecifics(0, tab_list1, &meta);
|
| - changes.push_back(MakeRemoteChange(meta, SyncChange::ACTION_UPDATE));
|
| - // Update associator with the session's meta node containing one window.
|
| - manager()->ProcessSyncChanges(FROM_HERE, changes);
|
| -
|
| - // Check that the foreign session was associated and retrieve the data.
|
| - foreign_sessions.clear();
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - ASSERT_EQ(1U, foreign_sessions.size());
|
| - ASSERT_EQ(1U, foreign_sessions[0]->windows.size());
|
| - std::vector<std::vector<SessionID::id_type> > session_reference;
|
| - session_reference.push_back(tab_list1);
|
| - helper()->VerifySyncedSession(tag, session_reference, *(foreign_sessions[0]));
|
| -}
|
| -
|
| -// Tests that the SessionsSyncManager can handle a remote client deleting
|
| -// sync nodes that belong to this local session.
|
| -TEST_F(SessionsSyncManagerTest, ProcessRemoteDeleteOfLocalSession) {
|
| - syncer::SyncChangeList out;
|
| - InitWithSyncDataTakeOutput(syncer::SyncDataList(), &out);
|
| - ASSERT_EQ(2U, out.size());
|
| - sync_pb::EntitySpecifics entity(out[0].sync_data().GetSpecifics());
|
| - SyncData d = CreateRemoteData(entity);
|
| - SetSyncData(syncer::SyncDataList(&d, &d + 1));
|
| - out.clear();
|
| -
|
| - syncer::SyncChangeList changes;
|
| - changes.push_back(
|
| - MakeRemoteChange(entity.session(), SyncChange::ACTION_DELETE));
|
| - manager()->ProcessSyncChanges(FROM_HERE, changes);
|
| - EXPECT_TRUE(manager()->local_tab_pool_out_of_sync_);
|
| - EXPECT_TRUE(out.empty()); // ChangeProcessor shouldn't see any activity.
|
| -
|
| - // This should trigger repair of the TabNodePool.
|
| - const GURL foo1("http://foo/1");
|
| - AddTab(browser(), foo1);
|
| - EXPECT_FALSE(manager()->local_tab_pool_out_of_sync_);
|
| -
|
| - // AddTab triggers two notifications, one for the tab insertion and one for
|
| - // committing the NavigationEntry. The first notification results in a tab
|
| - // we don't associate although we do update the header node. The second
|
| - // notification triggers association of the tab, and the subsequent window
|
| - // update. So we should see 4 changes at the SyncChangeProcessor.
|
| - ASSERT_EQ(4U, out.size());
|
| -
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[0].change_type());
|
| - ASSERT_TRUE(out[0].sync_data().GetSpecifics().session().has_header());
|
| - EXPECT_EQ(SyncChange::ACTION_ADD, out[1].change_type());
|
| - int tab_node_id = out[1].sync_data().GetSpecifics().session().tab_node_id();
|
| - EXPECT_EQ(TabNodePool::TabIdToTag(
|
| - manager()->current_machine_tag(), tab_node_id),
|
| - syncer::SyncDataLocal(out[1].sync_data()).GetTag());
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[2].change_type());
|
| - ASSERT_TRUE(out[2].sync_data().GetSpecifics().session().has_tab());
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[3].change_type());
|
| - ASSERT_TRUE(out[3].sync_data().GetSpecifics().session().has_header());
|
| -
|
| - // Verify the actual content.
|
| - const sync_pb::SessionHeader& session_header =
|
| - out[3].sync_data().GetSpecifics().session().header();
|
| - ASSERT_EQ(1, session_header.window_size());
|
| - EXPECT_EQ(1, session_header.window(0).tab_size());
|
| - const sync_pb::SessionTab& tab1 =
|
| - out[2].sync_data().GetSpecifics().session().tab();
|
| - ASSERT_EQ(1, tab1.navigation_size());
|
| - EXPECT_EQ(foo1.spec(), tab1.navigation(0).virtual_url());
|
| -
|
| - // Verify TabNodePool integrity.
|
| - EXPECT_EQ(1U, manager()->local_tab_pool_.Capacity());
|
| - EXPECT_TRUE(manager()->local_tab_pool_.Empty());
|
| -
|
| - // Verify TabLinks.
|
| - SessionsSyncManager::TabLinksMap tab_map = manager()->local_tab_map_;
|
| - ASSERT_EQ(1U, tab_map.size());
|
| - int tab_id = out[2].sync_data().GetSpecifics().session().tab().tab_id();
|
| - EXPECT_EQ(tab_node_id, tab_map.find(tab_id)->second->tab_node_id());
|
| -}
|
| -
|
| -// Test that receiving a session delete from sync removes the session
|
| -// from tracking.
|
| -TEST_F(SessionsSyncManagerTest, ProcessForeignDelete) {
|
| - InitWithNoSyncData();
|
| - SessionID::id_type n[] = {5};
|
| - std::vector<sync_pb::SessionSpecifics> tabs1;
|
| - std::vector<SessionID::id_type> tab_list(n, n + arraysize(n));
|
| - sync_pb::SessionSpecifics meta(helper()->BuildForeignSession(
|
| - "tag1", tab_list, &tabs1));
|
| -
|
| - syncer::SyncChangeList changes;
|
| - changes.push_back(MakeRemoteChange(meta, SyncChange::ACTION_ADD));
|
| - AddTabsToChangeList(tabs1, SyncChange::ACTION_ADD, &changes);
|
| - manager()->ProcessSyncChanges(FROM_HERE, changes);
|
| -
|
| - std::vector<const SyncedSession*> foreign_sessions;
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - ASSERT_EQ(1U, foreign_sessions.size());
|
| -
|
| - changes.clear();
|
| - foreign_sessions.clear();
|
| - changes.push_back(MakeRemoteChange(meta, SyncChange::ACTION_DELETE));
|
| - manager()->ProcessSyncChanges(FROM_HERE, changes);
|
| -
|
| - EXPECT_FALSE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| -}
|
| -
|
| -TEST_F(SessionsSyncManagerTest, ProcessForeignDeleteTabs) {
|
| - syncer::SyncDataList foreign_data;
|
| - base::Time stale_mtime = base::Time::Now() - base::TimeDelta::FromDays(15);
|
| - std::string session_tag = "tag1";
|
| -
|
| - // 0 will not have ownership changed.
|
| - // 1 will not be updated, but header will stop owning.
|
| - // 2 will be deleted before header stops owning.
|
| - // 3 will be deleted after header stops owning.
|
| - // 4 will be deleted before header update, but header will still try to own.
|
| - // 5 will be deleted after header update, but header will still try to own.
|
| - // 6 starts orphaned and then deleted before header update.
|
| - // 7 starts orphaned and then deleted after header update.
|
| - SessionID::id_type tab_ids[] = {0, 1, 2, 3, 4, 5};
|
| - std::vector<SessionID::id_type> tab_list(tab_ids,
|
| - tab_ids + arraysize(tab_ids));
|
| - std::vector<sync_pb::SessionSpecifics> tabs;
|
| - sync_pb::SessionSpecifics meta(
|
| - helper()->BuildForeignSession(session_tag, tab_list, &tabs));
|
| - AddToSyncDataList(meta, &foreign_data, stale_mtime);
|
| - AddTabsToSyncDataList(tabs, &foreign_data);
|
| - sync_pb::SessionSpecifics orphan6;
|
| - helper()->BuildTabSpecifics(session_tag, 0, 6, &orphan6);
|
| - AddToSyncDataList(orphan6, &foreign_data, stale_mtime);
|
| - sync_pb::SessionSpecifics orphan7;
|
| - helper()->BuildTabSpecifics(session_tag, 0, 7, &orphan7);
|
| - AddToSyncDataList(orphan7, &foreign_data, stale_mtime);
|
| -
|
| - syncer::SyncChangeList output;
|
| - InitWithSyncDataTakeOutput(foreign_data, &output);
|
| - ASSERT_EQ(2U, output.size());
|
| - output.clear();
|
| -
|
| - SessionID::id_type update_ids[] = {0, 4, 5};
|
| - std::vector<SessionID::id_type> update_list(
|
| - update_ids, update_ids + arraysize(update_ids));
|
| - sync_pb::SessionWindow* window = meta.mutable_header()->mutable_window(0);
|
| - window->clear_tab();
|
| - for (int i : update_ids) {
|
| - window->add_tab(i);
|
| - }
|
| -
|
| - syncer::SyncChangeList changes;
|
| - changes.push_back(MakeRemoteChange(tabs[2], SyncChange::ACTION_DELETE));
|
| - changes.push_back(MakeRemoteChange(tabs[4], SyncChange::ACTION_DELETE));
|
| - changes.push_back(MakeRemoteChange(orphan6, SyncChange::ACTION_DELETE));
|
| - changes.push_back(MakeRemoteChange(meta, SyncChange::ACTION_UPDATE));
|
| - changes.push_back(MakeRemoteChange(tabs[3], SyncChange::ACTION_DELETE));
|
| - changes.push_back(MakeRemoteChange(tabs[5], SyncChange::ACTION_DELETE));
|
| - changes.push_back(MakeRemoteChange(orphan7, SyncChange::ACTION_DELETE));
|
| - manager()->ProcessSyncChanges(FROM_HERE, changes);
|
| -
|
| - std::vector<const SyncedSession*> foreign_sessions;
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - ASSERT_EQ(1U, foreign_sessions.size());
|
| - std::vector<std::vector<SessionID::id_type>> session_reference;
|
| - session_reference.push_back(update_list);
|
| - helper()->VerifySyncedSession(session_tag, session_reference,
|
| - *(foreign_sessions[0]));
|
| -
|
| - // Everything except for session, tab0, and tab1 will have no node_id, and
|
| - // should get skipped by garbage collection.
|
| - manager()->DoGarbageCollection();
|
| - ASSERT_EQ(3U, output.size());
|
| -}
|
| -
|
| -TEST_F(SessionsSyncManagerTest, ProcessForeignDeleteTabsWithShadowing) {
|
| - syncer::SyncDataList foreign_data;
|
| - base::Time stale_mtime = base::Time::Now() - base::TimeDelta::FromDays(16);
|
| - std::string session_tag = "tag1";
|
| -
|
| - // Add several tabs that shadow eachother, in that they share tab_ids. They
|
| - // will, thanks to the helper, have unique tab_node_ids.
|
| - sync_pb::SessionSpecifics tab1A;
|
| - helper()->BuildTabSpecifics(session_tag, 0, 1, &tab1A);
|
| - AddToSyncDataList(tab1A, &foreign_data,
|
| - stale_mtime + base::TimeDelta::FromMinutes(1));
|
| -
|
| - sync_pb::SessionSpecifics tab1B;
|
| - helper()->BuildTabSpecifics(session_tag, 0, 1, &tab1B);
|
| - AddToSyncDataList(tab1B, &foreign_data,
|
| - stale_mtime + base::TimeDelta::FromMinutes(2));
|
| -
|
| - sync_pb::SessionSpecifics tab1C;
|
| - helper()->BuildTabSpecifics(session_tag, 0, 1, &tab1C);
|
| - AddToSyncDataList(tab1C, &foreign_data, stale_mtime);
|
| -
|
| - sync_pb::SessionSpecifics tab2A;
|
| - helper()->BuildTabSpecifics(session_tag, 0, 2, &tab2A);
|
| - AddToSyncDataList(tab2A, &foreign_data,
|
| - stale_mtime + base::TimeDelta::FromMinutes(1));
|
| -
|
| - sync_pb::SessionSpecifics tab2B;
|
| - helper()->BuildTabSpecifics(session_tag, 0, 2, &tab2B);
|
| - AddToSyncDataList(tab2B, &foreign_data,
|
| - stale_mtime + base::TimeDelta::FromMinutes(2));
|
| -
|
| - sync_pb::SessionSpecifics tab2C;
|
| - helper()->BuildTabSpecifics(session_tag, 0, 2, &tab2C);
|
| - AddToSyncDataList(tab2C, &foreign_data, stale_mtime);
|
| -
|
| - syncer::SyncChangeList output;
|
| - InitWithSyncDataTakeOutput(foreign_data, &output);
|
| - ASSERT_EQ(2U, output.size());
|
| - output.clear();
|
| -
|
| - // Verify that cleanup post-merge cleanup correctly removes all tabs objects.
|
| - const sessions::SessionTab* tab;
|
| - ASSERT_FALSE(
|
| - manager()->session_tracker_.LookupSessionTab(session_tag, 1, &tab));
|
| - ASSERT_FALSE(
|
| - manager()->session_tracker_.LookupSessionTab(session_tag, 2, &tab));
|
| -
|
| - std::set<int> tab_node_ids;
|
| - manager()->session_tracker_.LookupTabNodeIds(session_tag, &tab_node_ids);
|
| - EXPECT_EQ(6U, tab_node_ids.size());
|
| - EXPECT_TRUE(tab_node_ids.find(tab1A.tab_node_id()) != tab_node_ids.end());
|
| - EXPECT_TRUE(tab_node_ids.find(tab1B.tab_node_id()) != tab_node_ids.end());
|
| - EXPECT_TRUE(tab_node_ids.find(tab1C.tab_node_id()) != tab_node_ids.end());
|
| - EXPECT_TRUE(tab_node_ids.find(tab2A.tab_node_id()) != tab_node_ids.end());
|
| - EXPECT_TRUE(tab_node_ids.find(tab2B.tab_node_id()) != tab_node_ids.end());
|
| - EXPECT_TRUE(tab_node_ids.find(tab2C.tab_node_id()) != tab_node_ids.end());
|
| -
|
| - syncer::SyncChangeList changes;
|
| - changes.push_back(MakeRemoteChange(tab1A, SyncChange::ACTION_DELETE));
|
| - changes.push_back(MakeRemoteChange(tab1B, SyncChange::ACTION_DELETE));
|
| - changes.push_back(MakeRemoteChange(tab2C, SyncChange::ACTION_DELETE));
|
| - manager()->ProcessSyncChanges(FROM_HERE, changes);
|
| -
|
| - tab_node_ids.clear();
|
| - manager()->session_tracker_.LookupTabNodeIds(session_tag, &tab_node_ids);
|
| - EXPECT_EQ(3U, tab_node_ids.size());
|
| - EXPECT_TRUE(tab_node_ids.find(tab1C.tab_node_id()) != tab_node_ids.end());
|
| - EXPECT_TRUE(tab_node_ids.find(tab2A.tab_node_id()) != tab_node_ids.end());
|
| - EXPECT_TRUE(tab_node_ids.find(tab2B.tab_node_id()) != tab_node_ids.end());
|
| -
|
| - manager()->DoGarbageCollection();
|
| - ASSERT_EQ(3U, output.size());
|
| -}
|
| -
|
| -TEST_F(SessionsSyncManagerTest, ProcessForeignDeleteTabsWithReusedNodeIds) {
|
| - syncer::SyncDataList foreign_data;
|
| - base::Time stale_mtime = base::Time::Now() - base::TimeDelta::FromDays(16);
|
| - std::string session_tag = "tag1";
|
| - int tab_node_id_shared = 13;
|
| - int tab_node_id_unique = 14;
|
| -
|
| - sync_pb::SessionSpecifics tab1A;
|
| - helper()->BuildTabSpecifics(session_tag, 0, 1, tab_node_id_shared, &tab1A);
|
| - AddToSyncDataList(tab1A, &foreign_data,
|
| - stale_mtime + base::TimeDelta::FromMinutes(1));
|
| -
|
| - sync_pb::SessionSpecifics tab1B;
|
| - helper()->BuildTabSpecifics(session_tag, 0, 1, tab_node_id_unique, &tab1B);
|
| - AddToSyncDataList(tab1B, &foreign_data,
|
| - stale_mtime + base::TimeDelta::FromMinutes(2));
|
| -
|
| - sync_pb::SessionSpecifics tab2A;
|
| - helper()->BuildTabSpecifics(session_tag, 0, 2, tab_node_id_shared, &tab2A);
|
| - AddToSyncDataList(tab2A, &foreign_data,
|
| - stale_mtime + base::TimeDelta::FromMinutes(1));
|
| -
|
| - syncer::SyncChangeList output;
|
| - InitWithSyncDataTakeOutput(foreign_data, &output);
|
| - ASSERT_EQ(2U, output.size());
|
| - output.clear();
|
| -
|
| - std::set<int> tab_node_ids;
|
| - manager()->session_tracker_.LookupTabNodeIds(session_tag, &tab_node_ids);
|
| - EXPECT_EQ(2U, tab_node_ids.size());
|
| - EXPECT_TRUE(tab_node_ids.find(tab_node_id_shared) != tab_node_ids.end());
|
| - EXPECT_TRUE(tab_node_ids.find(tab_node_id_unique) != tab_node_ids.end());
|
| -
|
| - syncer::SyncChangeList changes;
|
| - changes.push_back(MakeRemoteChange(tab1A, SyncChange::ACTION_DELETE));
|
| - manager()->ProcessSyncChanges(FROM_HERE, changes);
|
| -
|
| - tab_node_ids.clear();
|
| - manager()->session_tracker_.LookupTabNodeIds(session_tag, &tab_node_ids);
|
| - EXPECT_EQ(1U, tab_node_ids.size());
|
| - EXPECT_TRUE(tab_node_ids.find(tab_node_id_unique) != tab_node_ids.end());
|
| -
|
| - manager()->DoGarbageCollection();
|
| - EXPECT_EQ(1U, output.size());
|
| -}
|
| -
|
| -// TODO(shashishekhar): "Move this to TabNodePool unittests."
|
| -TEST_F(SessionsSyncManagerTest, SaveUnassociatedNodesForReassociation) {
|
| - syncer::SyncChangeList changes;
|
| - InitWithNoSyncData();
|
| -
|
| - std::string local_tag = manager()->current_machine_tag();
|
| - // Create a free node and then dissassociate sessions so that it ends up
|
| - // unassociated.
|
| - manager()->local_tab_pool_.GetFreeTabNode(&changes);
|
| -
|
| - // Update the tab_id of the node, so that it is considered a valid
|
| - // unassociated node otherwise it will be mistaken for a corrupted node and
|
| - // will be deleted before being added to the tab node pool.
|
| - sync_pb::EntitySpecifics entity(changes[0].sync_data().GetSpecifics());
|
| - entity.mutable_session()->mutable_tab()->set_tab_id(1);
|
| - SyncData d = CreateRemoteData(entity);
|
| - syncer::SyncDataList in(&d, &d + 1);
|
| - changes.clear();
|
| - SessionsSyncManager manager2(GetSyncSessionsClient(), sync_prefs(),
|
| - local_device(), NewDummyRouter(),
|
| - base::Closure(), base::Closure());
|
| - syncer::SyncMergeResult result = manager2.MergeDataAndStartSyncing(
|
| - syncer::SESSIONS, in, std::unique_ptr<syncer::SyncChangeProcessor>(
|
| - new TestSyncProcessorStub(&changes)),
|
| - std::unique_ptr<syncer::SyncErrorFactory>(
|
| - new syncer::SyncErrorFactoryMock()));
|
| - ASSERT_FALSE(result.error().IsSet());
|
| - EXPECT_TRUE(FilterOutLocalHeaderChanges(&changes)->empty());
|
| -}
|
| -
|
| -TEST_F(SessionsSyncManagerTest, MergeDeletesCorruptNode) {
|
| - syncer::SyncChangeList changes;
|
| - InitWithNoSyncData();
|
| -
|
| - std::string local_tag = manager()->current_machine_tag();
|
| - int tab_node_id = manager()->local_tab_pool_.GetFreeTabNode(&changes);
|
| - SyncData d = CreateRemoteData(changes[0].sync_data().GetSpecifics());
|
| - syncer::SyncDataList in(&d, &d + 1);
|
| - changes.clear();
|
| - TearDown();
|
| - SetUp();
|
| - InitWithSyncDataTakeOutput(in, &changes);
|
| - EXPECT_EQ(1U, FilterOutLocalHeaderChanges(&changes)->size());
|
| - EXPECT_EQ(SyncChange::ACTION_DELETE, changes[0].change_type());
|
| - EXPECT_EQ(TabNodePool::TabIdToTag(local_tag, tab_node_id),
|
| - syncer::SyncDataLocal(changes[0].sync_data()).GetTag());
|
| -}
|
| -
|
| -// Verifies that we drop both headers and tabs during merge if their stored tag
|
| -// hash doesn't match a computer tag hash. This mitigates potential failures
|
| -// while cleaning up bad foreign data, see crbug.com/604657.
|
| -TEST_F(SessionsSyncManagerTest, MergeDeletesBadHash) {
|
| - syncer::SyncDataList foreign_data;
|
| - std::vector<SessionID::id_type> empty_ids;
|
| - std::vector<sync_pb::SessionSpecifics> empty_tabs;
|
| - sync_pb::EntitySpecifics entity;
|
| -
|
| - const std::string good_header_tag = "good_header_tag";
|
| - sync_pb::SessionSpecifics good_header(
|
| - helper()->BuildForeignSession(good_header_tag, empty_ids, &empty_tabs));
|
| - foreign_data.push_back(CreateRemoteData(good_header));
|
| -
|
| - const std::string bad_header_tag = "bad_header_tag";
|
| - sync_pb::SessionSpecifics bad_header(
|
| - helper()->BuildForeignSession(bad_header_tag, empty_ids, &empty_tabs));
|
| - entity.mutable_session()->CopyFrom(bad_header);
|
| - foreign_data.push_back(SyncData::CreateRemoteData(
|
| - 1, entity, base::Time(), syncer::AttachmentIdList(),
|
| - syncer::AttachmentServiceProxyForTest::Create(), "bad_header_tag_hash"));
|
| -
|
| - const std::string good_tag_tab = "good_tag_tab";
|
| - sync_pb::SessionSpecifics good_tab;
|
| - helper()->BuildTabSpecifics(good_tag_tab, 0, 1, &good_tab);
|
| - foreign_data.push_back(CreateRemoteData(good_tab));
|
| -
|
| - const std::string bad_tab_tag = "bad_tab_tag";
|
| - sync_pb::SessionSpecifics bad_tab;
|
| - helper()->BuildTabSpecifics(bad_tab_tag, 0, 2, &bad_tab);
|
| - entity.mutable_session()->CopyFrom(bad_tab);
|
| - foreign_data.push_back(SyncData::CreateRemoteData(
|
| - 1, entity, base::Time(), syncer::AttachmentIdList(),
|
| - syncer::AttachmentServiceProxyForTest::Create(), "bad_tab_tag_hash"));
|
| -
|
| - syncer::SyncChangeList output;
|
| - InitWithSyncDataTakeOutput(foreign_data, &output);
|
| - ASSERT_EQ(2U, FilterOutLocalHeaderChanges(&output)->size());
|
| - ExpectAllOfChangesType(output, SyncChange::ACTION_DELETE);
|
| - EXPECT_EQ(1, CountIfTagMatches(output, bad_header_tag));
|
| - EXPECT_EQ(1, CountIfTagMatches(output, bad_tab_tag));
|
| -
|
| - std::vector<const SyncedSession*> sessions;
|
| - manager()->session_tracker_.LookupAllForeignSessions(
|
| - &sessions, SyncedSessionTracker::RAW);
|
| - ASSERT_EQ(2U, sessions.size());
|
| - EXPECT_EQ(1, CountIfTagMatches(sessions, good_header_tag));
|
| - EXPECT_EQ(1, CountIfTagMatches(sessions, good_tag_tab));
|
| -}
|
| -
|
| -// Test that things work if a tab is initially ignored.
|
| -TEST_F(SessionsSyncManagerTest, AssociateWindowsDontReloadTabs) {
|
| - syncer::SyncChangeList out;
|
| - // Go to a URL that is ignored by session syncing.
|
| - AddTab(browser(), GURL("chrome://preferences/"));
|
| - InitWithSyncDataTakeOutput(syncer::SyncDataList(), &out);
|
| - ASSERT_EQ(2U, out.size()); // Header add and update.
|
| - EXPECT_EQ(
|
| - 0,
|
| - out[1].sync_data().GetSpecifics().session().header().window_size());
|
| - out.clear();
|
| -
|
| - // Go to a sync-interesting URL.
|
| - NavigateAndCommitActiveTab(GURL("http://foo2"));
|
| -
|
| - EXPECT_EQ(3U, out.size()); // Tab add, update, and header update.
|
| -
|
| - EXPECT_TRUE(
|
| - base::StartsWith(syncer::SyncDataLocal(out[0].sync_data()).GetTag(),
|
| - manager()->current_machine_tag(),
|
| - base::CompareCase::SENSITIVE));
|
| - EXPECT_EQ(manager()->current_machine_tag(),
|
| - out[0].sync_data().GetSpecifics().session().session_tag());
|
| - EXPECT_EQ(SyncChange::ACTION_ADD, out[0].change_type());
|
| -
|
| - EXPECT_TRUE(
|
| - base::StartsWith(syncer::SyncDataLocal(out[1].sync_data()).GetTag(),
|
| - manager()->current_machine_tag(),
|
| - base::CompareCase::SENSITIVE));
|
| - EXPECT_EQ(manager()->current_machine_tag(),
|
| - out[1].sync_data().GetSpecifics().session().session_tag());
|
| - EXPECT_TRUE(out[1].sync_data().GetSpecifics().session().has_tab());
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[1].change_type());
|
| -
|
| - EXPECT_TRUE(out[2].IsValid());
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[2].change_type());
|
| - const SyncData data(out[2].sync_data());
|
| - EXPECT_EQ(manager()->current_machine_tag(),
|
| - syncer::SyncDataLocal(data).GetTag());
|
| - const sync_pb::SessionSpecifics& specifics(data.GetSpecifics().session());
|
| - EXPECT_EQ(manager()->current_machine_tag(), specifics.session_tag());
|
| - EXPECT_TRUE(specifics.has_header());
|
| - const sync_pb::SessionHeader& header_s = specifics.header();
|
| - EXPECT_EQ(1, header_s.window_size());
|
| - EXPECT_EQ(1, header_s.window(0).tab_size());
|
| -}
|
| -
|
| -// Tests that the SyncSessionManager responds to local tab events properly.
|
| -TEST_F(SessionsSyncManagerTest, OnLocalTabModified) {
|
| - syncer::SyncChangeList out;
|
| - // Init with no local data, relies on MergeLocalSessionNoTabs.
|
| - InitWithSyncDataTakeOutput(syncer::SyncDataList(), &out);
|
| - ASSERT_FALSE(manager()->current_machine_tag().empty());
|
| - ASSERT_EQ(2U, out.size());
|
| -
|
| - // Copy the original header.
|
| - sync_pb::EntitySpecifics header(out[0].sync_data().GetSpecifics());
|
| - out.clear();
|
| -
|
| - const GURL foo1("http://foo/1");
|
| - const GURL foo2("http://foo/2");
|
| - const GURL bar1("http://bar/1");
|
| - const GURL bar2("http://bar/2");
|
| - AddTab(browser(), foo1);
|
| - NavigateAndCommitActiveTab(foo2);
|
| - AddTab(browser(), bar1);
|
| - NavigateAndCommitActiveTab(bar2);
|
| -
|
| - // One add, one update for each AddTab.
|
| - // One update for each NavigateAndCommit.
|
| - // = 6 total tab updates.
|
| - // One header update corresponding to each of those.
|
| - // = 6 total header updates.
|
| - // 12 total updates.
|
| - ASSERT_EQ(12U, out.size());
|
| -
|
| - // Verify the tab node creations and updates to ensure the SyncProcessor
|
| - // sees the right operations.
|
| - for (int i = 0; i < 12; i++) {
|
| - SCOPED_TRACE(i);
|
| - EXPECT_TRUE(out[i].IsValid());
|
| - const SyncData data(out[i].sync_data());
|
| - EXPECT_TRUE(base::StartsWith(syncer::SyncDataLocal(data).GetTag(),
|
| - manager()->current_machine_tag(),
|
| - base::CompareCase::SENSITIVE));
|
| - const sync_pb::SessionSpecifics& specifics(data.GetSpecifics().session());
|
| - EXPECT_EQ(manager()->current_machine_tag(), specifics.session_tag());
|
| - if (i % 6 == 0) {
|
| - // First thing on an AddTab is a no-op header update for parented tab.
|
| - EXPECT_EQ(header.SerializeAsString(),
|
| - data.GetSpecifics().SerializeAsString());
|
| - EXPECT_EQ(manager()->current_machine_tag(),
|
| - syncer::SyncDataLocal(data).GetTag());
|
| - } else if (i % 6 == 1) {
|
| - // Next, the TabNodePool should create the tab node.
|
| - EXPECT_EQ(SyncChange::ACTION_ADD, out[i].change_type());
|
| - EXPECT_EQ(TabNodePool::TabIdToTag(
|
| - manager()->current_machine_tag(),
|
| - data.GetSpecifics().session().tab_node_id()),
|
| - syncer::SyncDataLocal(data).GetTag());
|
| - } else if (i % 6 == 2) {
|
| - // Then we see the tab update to the URL.
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[i].change_type());
|
| - EXPECT_EQ(TabNodePool::TabIdToTag(
|
| - manager()->current_machine_tag(),
|
| - data.GetSpecifics().session().tab_node_id()),
|
| - syncer::SyncDataLocal(data).GetTag());
|
| - ASSERT_TRUE(specifics.has_tab());
|
| - } else if (i % 6 == 3) {
|
| - // The header needs to be updated to reflect the new window state.
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[i].change_type());
|
| - EXPECT_TRUE(specifics.has_header());
|
| - } else if (i % 6 == 4) {
|
| - // Now we move on to NavigateAndCommit. Update the tab.
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[i].change_type());
|
| - EXPECT_EQ(TabNodePool::TabIdToTag(
|
| - manager()->current_machine_tag(),
|
| - data.GetSpecifics().session().tab_node_id()),
|
| - syncer::SyncDataLocal(data).GetTag());
|
| - ASSERT_TRUE(specifics.has_tab());
|
| - } else if (i % 6 == 5) {
|
| - // The header needs to be updated to reflect the new window state.
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[i].change_type());
|
| - ASSERT_TRUE(specifics.has_header());
|
| - header = data.GetSpecifics();
|
| - }
|
| - }
|
| -
|
| - // Verify the actual content to ensure sync sees the right data.
|
| - // When it's all said and done, the header should reflect two tabs.
|
| - const sync_pb::SessionHeader& session_header = header.session().header();
|
| - ASSERT_EQ(1, session_header.window_size());
|
| - EXPECT_EQ(2, session_header.window(0).tab_size());
|
| -
|
| - // ASSERT_TRUEs above allow us to dive in freely here.
|
| - // Verify first tab.
|
| - const sync_pb::SessionTab& tab1_1 =
|
| - out[2].sync_data().GetSpecifics().session().tab();
|
| - ASSERT_EQ(1, tab1_1.navigation_size());
|
| - EXPECT_EQ(foo1.spec(), tab1_1.navigation(0).virtual_url());
|
| - const sync_pb::SessionTab& tab1_2 =
|
| - out[4].sync_data().GetSpecifics().session().tab();
|
| - ASSERT_EQ(2, tab1_2.navigation_size());
|
| - EXPECT_EQ(foo1.spec(), tab1_2.navigation(0).virtual_url());
|
| - EXPECT_EQ(foo2.spec(), tab1_2.navigation(1).virtual_url());
|
| -
|
| - // Verify second tab.
|
| - const sync_pb::SessionTab& tab2_1 =
|
| - out[8].sync_data().GetSpecifics().session().tab();
|
| - ASSERT_EQ(1, tab2_1.navigation_size());
|
| - EXPECT_EQ(bar1.spec(), tab2_1.navigation(0).virtual_url());
|
| - const sync_pb::SessionTab& tab2_2 =
|
| - out[10].sync_data().GetSpecifics().session().tab();
|
| - ASSERT_EQ(2, tab2_2.navigation_size());
|
| - EXPECT_EQ(bar1.spec(), tab2_2.navigation(0).virtual_url());
|
| - EXPECT_EQ(bar2.spec(), tab2_2.navigation(1).virtual_url());
|
| -}
|
| -
|
| -// Check that if a tab becomes uninteresting (for example no syncable URLs),
|
| -// we correctly remove it from the header node.
|
| -TEST_F(SessionsSyncManagerTest, TabBecomesUninteresting) {
|
| - syncer::SyncChangeList out;
|
| - // Init with no local data, relies on MergeLocalSessionNoTabs.
|
| - InitWithSyncDataTakeOutput(syncer::SyncDataList(), &out);
|
| - ASSERT_FALSE(manager()->current_machine_tag().empty());
|
| - ASSERT_EQ(2U, out.size());
|
| - out.clear();
|
| -
|
| - const GURL kValidUrl("http://foo/1");
|
| - const GURL kInternalUrl("chrome://internal");
|
| -
|
| - // Add an interesting tab.
|
| - AddTab(browser(), kValidUrl);
|
| - // No-op header update, tab creation, tab update, header update.
|
| - ASSERT_EQ(4U, out.size());
|
| - // The last two are the interesting updates.
|
| - ASSERT_TRUE(out[2].sync_data().GetSpecifics().session().has_tab());
|
| - EXPECT_EQ(kValidUrl.spec(), out[2]
|
| - .sync_data()
|
| - .GetSpecifics()
|
| - .session()
|
| - .tab()
|
| - .navigation(0)
|
| - .virtual_url());
|
| - ASSERT_TRUE(out[3].sync_data().GetSpecifics().session().has_header());
|
| - ASSERT_EQ(1,
|
| - out[3].sync_data().GetSpecifics().session().header().window_size());
|
| - ASSERT_EQ(1, out[3]
|
| - .sync_data()
|
| - .GetSpecifics()
|
| - .session()
|
| - .header()
|
| - .window(0)
|
| - .tab_size());
|
| -
|
| - // Navigate five times to uninteresting urls to push the interesting one off
|
| - // the back of the stack.
|
| - NavigateAndCommitActiveTab(kInternalUrl);
|
| - NavigateAndCommitActiveTab(kInternalUrl);
|
| - NavigateAndCommitActiveTab(kInternalUrl);
|
| - NavigateAndCommitActiveTab(kInternalUrl);
|
| -
|
| - // Reset |out| so we only see the effects of the final navigation.
|
| - out.clear();
|
| - NavigateAndCommitActiveTab(kInternalUrl);
|
| -
|
| - // Only the header node should be updated, and it should no longer have any
|
| - // valid windows/tabs.
|
| - ASSERT_EQ(2U, out.size()); // Two header updates (first is a no-op).
|
| - ASSERT_TRUE(out[1].sync_data().GetSpecifics().session().has_header());
|
| - EXPECT_EQ(1,
|
| - out[1].sync_data().GetSpecifics().session().header().window_size());
|
| -}
|
| -
|
| -// Ensure model association associates the pre-existing tabs.
|
| -TEST_F(SessionsSyncManagerTest, MergeLocalSessionExistingTabs) {
|
| - AddTab(browser(), GURL("http://foo1"));
|
| - NavigateAndCommitActiveTab(GURL("http://foo2")); // Adds back entry.
|
| - AddTab(browser(), GURL("http://bar1"));
|
| - NavigateAndCommitActiveTab(GURL("http://bar2")); // Adds back entry.
|
| -
|
| - syncer::SyncChangeList out;
|
| - InitWithSyncDataTakeOutput(syncer::SyncDataList(), &out);
|
| - ASSERT_EQ(6U, out.size());
|
| -
|
| - // Check that this machine's data is not included in the foreign windows.
|
| - std::vector<const SyncedSession*> foreign_sessions;
|
| - ASSERT_FALSE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| -
|
| - // Verify the header.
|
| - EXPECT_TRUE(out[0].IsValid());
|
| - EXPECT_EQ(SyncChange::ACTION_ADD, out[0].change_type());
|
| - const SyncData data(out[0].sync_data());
|
| - EXPECT_EQ(manager()->current_machine_tag(),
|
| - syncer::SyncDataLocal(data).GetTag());
|
| - const sync_pb::SessionSpecifics& specifics(data.GetSpecifics().session());
|
| - EXPECT_EQ(manager()->current_machine_tag(), specifics.session_tag());
|
| - EXPECT_TRUE(specifics.has_header());
|
| - const sync_pb::SessionHeader& header_s = specifics.header();
|
| - EXPECT_TRUE(header_s.has_device_type());
|
| - EXPECT_EQ(GetLocalDeviceInfo()->client_name(), header_s.client_name());
|
| - EXPECT_EQ(0, header_s.window_size());
|
| -
|
| - // Verify the tab node creations and updates with content.
|
| - for (int i = 1; i < 5; i++) {
|
| - EXPECT_TRUE(out[i].IsValid());
|
| - const SyncData data(out[i].sync_data());
|
| - EXPECT_TRUE(base::StartsWith(syncer::SyncDataLocal(data).GetTag(),
|
| - manager()->current_machine_tag(),
|
| - base::CompareCase::SENSITIVE));
|
| - const sync_pb::SessionSpecifics& specifics(data.GetSpecifics().session());
|
| - EXPECT_EQ(manager()->current_machine_tag(), specifics.session_tag());
|
| - if (i % 2 == 1) {
|
| - EXPECT_EQ(SyncChange::ACTION_ADD, out[i].change_type());
|
| - } else {
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[i].change_type());
|
| - EXPECT_TRUE(specifics.has_tab());
|
| - }
|
| - }
|
| -
|
| - // Verify the header was updated to reflect new window state.
|
| - EXPECT_TRUE(out[5].IsValid());
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[5].change_type());
|
| - const SyncData data_2(out[5].sync_data());
|
| - EXPECT_EQ(manager()->current_machine_tag(),
|
| - syncer::SyncDataLocal(data_2).GetTag());
|
| - const sync_pb::SessionSpecifics& specifics2(data_2.GetSpecifics().session());
|
| - EXPECT_EQ(manager()->current_machine_tag(), specifics2.session_tag());
|
| - EXPECT_TRUE(specifics2.has_header());
|
| - const sync_pb::SessionHeader& header_s2 = specifics2.header();
|
| - EXPECT_EQ(1, header_s2.window_size());
|
| -
|
| - // Verify TabLinks.
|
| - SessionsSyncManager::TabLinksMap tab_map = manager()->local_tab_map_;
|
| - ASSERT_EQ(2U, tab_map.size());
|
| - // Tabs are ordered by sessionid in tab_map, so should be able to traverse
|
| - // the tree based on order of tabs created
|
| - SessionsSyncManager::TabLinksMap::iterator iter = tab_map.begin();
|
| - ASSERT_EQ(2, iter->second->tab()->GetEntryCount());
|
| - EXPECT_EQ(GURL("http://foo1"), iter->second->tab()->GetVirtualURLAtIndex(0));
|
| - EXPECT_EQ(GURL("http://foo2"), iter->second->tab()->GetVirtualURLAtIndex(1));
|
| - iter++;
|
| - ASSERT_EQ(2, iter->second->tab()->GetEntryCount());
|
| - EXPECT_EQ(GURL("http://bar1"), iter->second->tab()->GetVirtualURLAtIndex(0));
|
| - EXPECT_EQ(GURL("http://bar2"), iter->second->tab()->GetVirtualURLAtIndex(1));
|
| -}
|
| -
|
| -TEST_F(SessionsSyncManagerTest, ForeignSessionModifiedTime) {
|
| - syncer::SyncDataList foreign_data;
|
| - base::Time newest_time = base::Time::Now() - base::TimeDelta::FromDays(1);
|
| - base::Time middle_time = base::Time::Now() - base::TimeDelta::FromDays(2);
|
| - base::Time oldest_time = base::Time::Now() - base::TimeDelta::FromDays(3);
|
| -
|
| - {
|
| - std::string session_tag = "tag1";
|
| - SessionID::id_type n[] = {1, 2};
|
| - std::vector<SessionID::id_type> tab_list(n, n + arraysize(n));
|
| - std::vector<sync_pb::SessionSpecifics> tabs;
|
| - sync_pb::SessionSpecifics meta(
|
| - helper()->BuildForeignSession(session_tag, tab_list, &tabs));
|
| - AddToSyncDataList(tabs[0], &foreign_data, newest_time);
|
| - AddToSyncDataList(meta, &foreign_data, middle_time);
|
| - AddToSyncDataList(tabs[1], &foreign_data, oldest_time);
|
| - }
|
| -
|
| - {
|
| - std::string session_tag = "tag2";
|
| - SessionID::id_type n[] = {3, 4};
|
| - std::vector<SessionID::id_type> tab_list(n, n + arraysize(n));
|
| - std::vector<sync_pb::SessionSpecifics> tabs;
|
| - sync_pb::SessionSpecifics meta(
|
| - helper()->BuildForeignSession(session_tag, tab_list, &tabs));
|
| - AddToSyncDataList(tabs[0], &foreign_data, middle_time);
|
| - AddToSyncDataList(meta, &foreign_data, newest_time);
|
| - AddToSyncDataList(tabs[1], &foreign_data, oldest_time);
|
| - }
|
| -
|
| - {
|
| - std::string session_tag = "tag3";
|
| - SessionID::id_type n[] = {5, 6};
|
| - std::vector<SessionID::id_type> tab_list(n, n + arraysize(n));
|
| - std::vector<sync_pb::SessionSpecifics> tabs;
|
| - sync_pb::SessionSpecifics meta(
|
| - helper()->BuildForeignSession(session_tag, tab_list, &tabs));
|
| - AddToSyncDataList(tabs[0], &foreign_data, oldest_time);
|
| - AddToSyncDataList(meta, &foreign_data, middle_time);
|
| - AddToSyncDataList(tabs[1], &foreign_data, newest_time);
|
| - }
|
| -
|
| - syncer::SyncChangeList output;
|
| - InitWithSyncDataTakeOutput(foreign_data, &output);
|
| - ASSERT_EQ(2U, output.size());
|
| - output.clear();
|
| -
|
| - std::vector<const SyncedSession*> foreign_sessions;
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - ASSERT_EQ(3U, foreign_sessions.size());
|
| - EXPECT_EQ(newest_time, foreign_sessions[0]->modified_time);
|
| - EXPECT_EQ(newest_time, foreign_sessions[1]->modified_time);
|
| - EXPECT_EQ(newest_time, foreign_sessions[2]->modified_time);
|
| -}
|
| -
|
| -// Test garbage collection of stale foreign sessions.
|
| -TEST_F(SessionsSyncManagerTest, DoGarbageCollection) {
|
| - // Fill two instances of session specifics with a foreign session's data.
|
| - std::string tag1 = "tag1";
|
| - SessionID::id_type n1[] = {5, 10, 13, 17};
|
| - std::vector<SessionID::id_type> tab_list1(n1, n1 + arraysize(n1));
|
| - std::vector<sync_pb::SessionSpecifics> tabs1;
|
| - sync_pb::SessionSpecifics meta(helper()->BuildForeignSession(
|
| - tag1, tab_list1, &tabs1));
|
| - std::string tag2 = "tag2";
|
| - SessionID::id_type n2[] = {8, 15, 18, 20};
|
| - std::vector<SessionID::id_type> tab_list2(n2, n2 + arraysize(n2));
|
| - std::vector<sync_pb::SessionSpecifics> tabs2;
|
| - sync_pb::SessionSpecifics meta2(helper()->BuildForeignSession(
|
| - tag2, tab_list2, &tabs2));
|
| - // Set the modification time for tag1 to be 21 days ago, tag2 to 5 days ago.
|
| - base::Time tag1_time = base::Time::Now() - base::TimeDelta::FromDays(21);
|
| - base::Time tag2_time = base::Time::Now() - base::TimeDelta::FromDays(5);
|
| -
|
| - syncer::SyncDataList foreign_data;
|
| - foreign_data.push_back(CreateRemoteData(meta, tag1_time));
|
| - foreign_data.push_back(CreateRemoteData(meta2, tag2_time));
|
| - AddTabsToSyncDataList(tabs1, &foreign_data);
|
| - AddTabsToSyncDataList(tabs2, &foreign_data);
|
| -
|
| - syncer::SyncChangeList output;
|
| - InitWithSyncDataTakeOutput(foreign_data, &output);
|
| - ASSERT_EQ(2U, output.size());
|
| - output.clear();
|
| -
|
| - // Check that the foreign session was associated and retrieve the data.
|
| - std::vector<const SyncedSession*> foreign_sessions;
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - ASSERT_EQ(2U, foreign_sessions.size());
|
| - foreign_sessions.clear();
|
| -
|
| - // Now garbage collect and verify the non-stale session is still there.
|
| - manager()->DoGarbageCollection();
|
| - ASSERT_EQ(5U, output.size());
|
| - EXPECT_EQ(SyncChange::ACTION_DELETE, output[0].change_type());
|
| - const SyncData data(output[0].sync_data());
|
| - EXPECT_EQ(tag1, syncer::SyncDataLocal(data).GetTag());
|
| - for (int i = 1; i < 5; i++) {
|
| - EXPECT_EQ(SyncChange::ACTION_DELETE, output[i].change_type());
|
| - const SyncData data(output[i].sync_data());
|
| - EXPECT_EQ(TabNodePool::TabIdToTag(tag1, i),
|
| - syncer::SyncDataLocal(data).GetTag());
|
| - }
|
| -
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - ASSERT_EQ(1U, foreign_sessions.size());
|
| - std::vector<std::vector<SessionID::id_type> > session_reference;
|
| - session_reference.push_back(tab_list2);
|
| - helper()->VerifySyncedSession(tag2, session_reference,
|
| - *(foreign_sessions[0]));
|
| -}
|
| -
|
| -TEST_F(SessionsSyncManagerTest, DoGarbageCollectionOrphans) {
|
| - syncer::SyncDataList foreign_data;
|
| - base::Time stale_mtime = base::Time::Now() - base::TimeDelta::FromDays(15);
|
| -
|
| - {
|
| - // A stale session with empty header
|
| - std::string session_tag = "tag1";
|
| - std::vector<SessionID::id_type> tab_list;
|
| - std::vector<sync_pb::SessionSpecifics> tabs;
|
| - sync_pb::SessionSpecifics meta(
|
| - helper()->BuildForeignSession(session_tag, tab_list, &tabs));
|
| - AddToSyncDataList(meta, &foreign_data, stale_mtime);
|
| - }
|
| -
|
| - {
|
| - // A stale session with orphans w/o header
|
| - std::string session_tag = "tag2";
|
| - sync_pb::SessionSpecifics orphan;
|
| - helper()->BuildTabSpecifics(session_tag, 0, 1, &orphan);
|
| - AddToSyncDataList(orphan, &foreign_data, stale_mtime);
|
| - }
|
| -
|
| - {
|
| - // A stale session with valid header/tab and an orphaned tab.
|
| - std::string session_tag = "tag3";
|
| - SessionID::id_type n[] = {2};
|
| - std::vector<SessionID::id_type> tab_list(n, n + arraysize(n));
|
| - std::vector<sync_pb::SessionSpecifics> tabs;
|
| - sync_pb::SessionSpecifics meta(
|
| - helper()->BuildForeignSession(session_tag, tab_list, &tabs));
|
| -
|
| - // BuildForeignSession(...) will use a window id of 0, and we're also
|
| - // passing a window id of 0 to BuildTabSpecifics(...) here. It doesn't
|
| - // really matter what window id we use for the orphaned tab, in the real
|
| - // world orphans often reference real/still valid windows, but they're
|
| - // orphans because the window/header doesn't reference back to them.
|
| - sync_pb::SessionSpecifics orphan;
|
| - helper()->BuildTabSpecifics(session_tag, 0, 1, &orphan);
|
| - AddToSyncDataList(orphan, &foreign_data, stale_mtime);
|
| -
|
| - AddToSyncDataList(tabs[0], &foreign_data, stale_mtime);
|
| - AddToSyncDataList(orphan, &foreign_data, stale_mtime);
|
| - AddToSyncDataList(meta, &foreign_data, stale_mtime);
|
| - }
|
| -
|
| - syncer::SyncChangeList output;
|
| - InitWithSyncDataTakeOutput(foreign_data, &output);
|
| - ASSERT_EQ(2U, output.size());
|
| - output.clear();
|
| -
|
| - // Although we have 3 foreign sessions, only 1 is valid/clean enough.
|
| - std::vector<const SyncedSession*> foreign_sessions;
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - ASSERT_EQ(1U, foreign_sessions.size());
|
| - foreign_sessions.clear();
|
| -
|
| - // Everything should get removed here.
|
| - manager()->DoGarbageCollection();
|
| - // Expect 5 deletions. tag1 header only, tag2 tab only, tag3 header + 2x tabs.
|
| - ASSERT_EQ(5U, output.size());
|
| -}
|
| -
|
| -// Test that an update to a previously considered "stale" session,
|
| -// prior to garbage collection, will save the session from deletion.
|
| -TEST_F(SessionsSyncManagerTest, GarbageCollectionHonoursUpdate) {
|
| - std::string tag1 = "tag1";
|
| - SessionID::id_type n1[] = {5, 10, 13, 17};
|
| - std::vector<SessionID::id_type> tab_list1(n1, n1 + arraysize(n1));
|
| - std::vector<sync_pb::SessionSpecifics> tabs1;
|
| - sync_pb::SessionSpecifics meta(helper()->BuildForeignSession(
|
| - tag1, tab_list1, &tabs1));
|
| - syncer::SyncDataList foreign_data;
|
| - base::Time tag1_time = base::Time::Now() - base::TimeDelta::FromDays(21);
|
| - foreign_data.push_back(CreateRemoteData(meta, tag1_time));
|
| - AddTabsToSyncDataList(tabs1, &foreign_data);
|
| - syncer::SyncChangeList output;
|
| - InitWithSyncDataTakeOutput(foreign_data, &output);
|
| - ASSERT_EQ(2U, output.size());
|
| -
|
| - // Update to a non-stale time.
|
| - sync_pb::EntitySpecifics update_entity;
|
| - update_entity.mutable_session()->CopyFrom(tabs1[0]);
|
| - syncer::SyncChangeList changes;
|
| - changes.push_back(
|
| - syncer::SyncChange(FROM_HERE, SyncChange::ACTION_UPDATE,
|
| - CreateRemoteData(tabs1[0], base::Time::Now())));
|
| - manager()->ProcessSyncChanges(FROM_HERE, changes);
|
| -
|
| - // Check that the foreign session was associated and retrieve the data.
|
| - std::vector<const SyncedSession*> foreign_sessions;
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - ASSERT_EQ(1U, foreign_sessions.size());
|
| - foreign_sessions.clear();
|
| -
|
| - // Verify the now non-stale session does not get deleted.
|
| - manager()->DoGarbageCollection();
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - ASSERT_EQ(1U, foreign_sessions.size());
|
| - std::vector<std::vector<SessionID::id_type> > session_reference;
|
| - session_reference.push_back(tab_list1);
|
| - helper()->VerifySyncedSession(
|
| - tag1, session_reference, *(foreign_sessions[0]));
|
| -}
|
| -
|
| -// Test that swapping WebContents for a tab is properly observed and handled
|
| -// by the SessionsSyncManager.
|
| -TEST_F(SessionsSyncManagerTest, CheckPrerenderedWebContentsSwap) {
|
| - AddTab(browser(), GURL("http://foo1"));
|
| - NavigateAndCommitActiveTab(GURL("http://foo2"));
|
| -
|
| - syncer::SyncChangeList out;
|
| - InitWithSyncDataTakeOutput(syncer::SyncDataList(), &out);
|
| - ASSERT_EQ(4U, out.size()); // Header, tab ADD, tab UPDATE, header UPDATE.
|
| -
|
| - // To simulate WebContents swap during prerendering, create new WebContents
|
| - // and swap with old WebContents.
|
| - std::unique_ptr<content::WebContents> old_web_contents;
|
| - old_web_contents.reset(browser()->tab_strip_model()->GetActiveWebContents());
|
| -
|
| - // Create new WebContents, with the required tab helpers.
|
| - WebContents* new_web_contents = WebContents::CreateWithSessionStorage(
|
| - WebContents::CreateParams(profile()),
|
| - old_web_contents->GetController().GetSessionStorageNamespaceMap());
|
| - SessionTabHelper::CreateForWebContents(new_web_contents);
|
| - TabContentsSyncedTabDelegate::CreateForWebContents(new_web_contents);
|
| - new_web_contents->GetController()
|
| - .CopyStateFrom(old_web_contents->GetController());
|
| -
|
| - // Swap the WebContents.
|
| - int index = browser()->tab_strip_model()->GetIndexOfWebContents(
|
| - old_web_contents.get());
|
| - browser()->tab_strip_model()->ReplaceWebContentsAt(index, new_web_contents);
|
| -
|
| - ASSERT_EQ(9U, out.size());
|
| - EXPECT_EQ(SyncChange::ACTION_ADD, out[4].change_type());
|
| - EXPECT_EQ(SyncChange::ACTION_UPDATE, out[5].change_type());
|
| -
|
| - // Navigate away.
|
| - NavigateAndCommitActiveTab(GURL("http://bar2"));
|
| -
|
| - // Delete old WebContents. This should not crash.
|
| - old_web_contents.reset();
|
| -
|
| - // Try more navigations and verify output size. This can also reveal
|
| - // bugs (leaks) on memcheck bots if the SessionSyncManager
|
| - // didn't properly clean up the tab pool or session tracker.
|
| - NavigateAndCommitActiveTab(GURL("http://bar3"));
|
| -
|
| - AddTab(browser(), GURL("http://bar4"));
|
| - NavigateAndCommitActiveTab(GURL("http://bar5"));
|
| - ASSERT_EQ(19U, out.size());
|
| -}
|
| -
|
| -// Test that NOTIFICATION_FOREIGN_SESSION_UPDATED is sent when processing
|
| -// sync changes.
|
| -TEST_F(SessionsSyncManagerTest, NotifiedOfUpdates) {
|
| - ASSERT_FALSE(observer()->notified_of_update());
|
| - InitWithNoSyncData();
|
| -
|
| - SessionID::id_type n[] = {5};
|
| - std::vector<sync_pb::SessionSpecifics> tabs1;
|
| - std::vector<SessionID::id_type> tab_list(n, n + arraysize(n));
|
| - sync_pb::SessionSpecifics meta(helper()->BuildForeignSession(
|
| - "tag1", tab_list, &tabs1));
|
| -
|
| - syncer::SyncChangeList changes;
|
| - changes.push_back(MakeRemoteChange(meta, SyncChange::ACTION_ADD));
|
| - manager()->ProcessSyncChanges(FROM_HERE, changes);
|
| - EXPECT_TRUE(observer()->notified_of_update());
|
| -
|
| - changes.clear();
|
| - observer()->Reset();
|
| - AddTabsToChangeList(tabs1, SyncChange::ACTION_ADD, &changes);
|
| - manager()->ProcessSyncChanges(FROM_HERE, changes);
|
| - EXPECT_TRUE(observer()->notified_of_update());
|
| -
|
| - changes.clear();
|
| - observer()->Reset();
|
| - changes.push_back(MakeRemoteChange(meta, SyncChange::ACTION_DELETE));
|
| - manager()->ProcessSyncChanges(FROM_HERE, changes);
|
| - EXPECT_TRUE(observer()->notified_of_update());
|
| -}
|
| -
|
| -// Test that NOTIFICATION_FOREIGN_SESSION_UPDATED is sent when handling
|
| -// local hide/removal of foreign session.
|
| -TEST_F(SessionsSyncManagerTest, NotifiedOfLocalRemovalOfForeignSession) {
|
| - InitWithNoSyncData();
|
| - const std::string tag("tag1");
|
| - SessionID::id_type n[] = {5};
|
| - std::vector<sync_pb::SessionSpecifics> tabs1;
|
| - std::vector<SessionID::id_type> tab_list(n, n + arraysize(n));
|
| - sync_pb::SessionSpecifics meta(helper()->BuildForeignSession(
|
| - tag, tab_list, &tabs1));
|
| -
|
| - syncer::SyncChangeList changes;
|
| - changes.push_back(MakeRemoteChange(meta, SyncChange::ACTION_ADD));
|
| - manager()->ProcessSyncChanges(FROM_HERE, changes);
|
| -
|
| - observer()->Reset();
|
| - ASSERT_FALSE(observer()->notified_of_update());
|
| - manager()->DeleteForeignSession(tag);
|
| - ASSERT_TRUE(observer()->notified_of_update());
|
| -}
|
| -
|
| -#if defined(OS_ANDROID)
|
| -// Tests that opening the other devices page triggers a session sync refresh.
|
| -// This page only exists on mobile platforms today; desktop has a
|
| -// search-enhanced NTP without other devices.
|
| -TEST_F(SessionsSyncManagerTest, NotifiedOfRefresh) {
|
| - ASSERT_FALSE(observer()->notified_of_refresh());
|
| - InitWithNoSyncData();
|
| - AddTab(browser(), GURL("http://foo1"));
|
| - EXPECT_FALSE(observer()->notified_of_refresh());
|
| - NavigateAndCommitActiveTab(GURL("chrome://newtab/#open_tabs"));
|
| - EXPECT_TRUE(observer()->notified_of_refresh());
|
| -}
|
| -#endif // defined(OS_ANDROID)
|
| -
|
| -// Tests receipt of duplicate tab IDs in the same window. This should never
|
| -// happen, but we want to make sure the client won't do anything bad if it does
|
| -// receive such garbage input data.
|
| -TEST_F(SessionsSyncManagerTest, ReceiveDuplicateTabInSameWindow) {
|
| - std::string tag = "tag1";
|
| -
|
| - // Reuse tab ID 10 in an attempt to trigger bad behavior.
|
| - SessionID::id_type n1[] = {5, 10, 10, 17};
|
| - std::vector<SessionID::id_type> tab_list1(n1, n1 + arraysize(n1));
|
| - std::vector<sync_pb::SessionSpecifics> tabs1;
|
| - sync_pb::SessionSpecifics meta(
|
| - helper()->BuildForeignSession(tag, tab_list1, &tabs1));
|
| -
|
| - // Set up initial data.
|
| - syncer::SyncDataList initial_data;
|
| - sync_pb::EntitySpecifics entity;
|
| - entity.mutable_session()->CopyFrom(meta);
|
| - initial_data.push_back(CreateRemoteData(entity));
|
| - AddTabsToSyncDataList(tabs1, &initial_data);
|
| -
|
| - syncer::SyncChangeList output;
|
| - InitWithSyncDataTakeOutput(initial_data, &output);
|
| -}
|
| -
|
| -// Tests receipt of duplicate tab IDs for the same session. The duplicate tab
|
| -// ID is present in two different windows. A client can't be expected to do
|
| -// anything reasonable with this input, but we can expect that it doesn't
|
| -// crash.
|
| -TEST_F(SessionsSyncManagerTest, ReceiveDuplicateTabInOtherWindow) {
|
| - std::string tag = "tag1";
|
| -
|
| - SessionID::id_type n1[] = {5, 10, 17};
|
| - std::vector<SessionID::id_type> tab_list1(n1, n1 + arraysize(n1));
|
| - std::vector<sync_pb::SessionSpecifics> tabs1;
|
| - sync_pb::SessionSpecifics meta(
|
| - helper()->BuildForeignSession(tag, tab_list1, &tabs1));
|
| -
|
| - // Add a second window. Tab ID 10 is a duplicate.
|
| - SessionID::id_type n2[] = {10, 18, 20};
|
| - std::vector<SessionID::id_type> tab_list2(n2, n2 + arraysize(n2));
|
| - helper()->AddWindowSpecifics(1, tab_list2, &meta);
|
| -
|
| - // Set up initial data.
|
| - syncer::SyncDataList initial_data;
|
| - sync_pb::EntitySpecifics entity;
|
| - entity.mutable_session()->CopyFrom(meta);
|
| - initial_data.push_back(CreateRemoteData(entity));
|
| - AddTabsToSyncDataList(tabs1, &initial_data);
|
| -
|
| - for (size_t i = 0; i < tab_list2.size(); ++i) {
|
| - sync_pb::EntitySpecifics entity;
|
| - helper()->BuildTabSpecifics(tag, 0, tab_list2[i], entity.mutable_session());
|
| - initial_data.push_back(CreateRemoteData(entity));
|
| - }
|
| -
|
| - syncer::SyncChangeList output;
|
| - InitWithSyncDataTakeOutput(initial_data, &output);
|
| -}
|
| -
|
| -// Tests receipt of multiple unassociated tabs and makes sure that
|
| -// the ones with later timestamp win
|
| -TEST_F(SessionsSyncManagerTest, ReceiveDuplicateUnassociatedTabs) {
|
| - std::string tag = "tag1";
|
| -
|
| - SessionID::id_type n1[] = {5, 10, 17};
|
| - std::vector<SessionID::id_type> tab_list1(n1, n1 + arraysize(n1));
|
| - std::vector<sync_pb::SessionSpecifics> tabs1;
|
| - sync_pb::SessionSpecifics meta(
|
| - helper()->BuildForeignSession(tag, tab_list1, &tabs1));
|
| -
|
| - // Set up initial data.
|
| - syncer::SyncDataList initial_data;
|
| - initial_data.push_back(CreateRemoteData(meta));
|
| -
|
| - sync_pb::EntitySpecifics entity;
|
| -
|
| - for (size_t i = 0; i < tabs1.size(); i++) {
|
| - entity.mutable_session()->CopyFrom(tabs1[i]);
|
| - initial_data.push_back(
|
| - CreateRemoteData(entity, base::Time::FromDoubleT(2000)));
|
| - }
|
| -
|
| - // Add two more tabs with duplicating IDs but with different modification
|
| - // times, one before and one after the tabs above.
|
| - // These two tabs get a different visual indices to distinguish them from the
|
| - // tabs above that get visual index 1 by default.
|
| - sync_pb::SessionSpecifics duplicating_tab1;
|
| - helper()->BuildTabSpecifics(tag, 0, 10, &duplicating_tab1);
|
| - duplicating_tab1.mutable_tab()->set_tab_visual_index(2);
|
| - entity.mutable_session()->CopyFrom(duplicating_tab1);
|
| - initial_data.push_back(
|
| - CreateRemoteData(entity, base::Time::FromDoubleT(1000)));
|
| -
|
| - sync_pb::SessionSpecifics duplicating_tab2;
|
| - helper()->BuildTabSpecifics(tag, 0, 17, &duplicating_tab2);
|
| - duplicating_tab2.mutable_tab()->set_tab_visual_index(3);
|
| - entity.mutable_session()->CopyFrom(duplicating_tab2);
|
| - initial_data.push_back(
|
| - CreateRemoteData(entity, base::Time::FromDoubleT(3000)));
|
| -
|
| - syncer::SyncChangeList output;
|
| - InitWithSyncDataTakeOutput(initial_data, &output);
|
| -
|
| - std::vector<const SyncedSession*> foreign_sessions;
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| -
|
| - const std::vector<std::unique_ptr<sessions::SessionTab>>& window_tabs =
|
| - foreign_sessions[0]->windows.find(0)->second->tabs;
|
| - ASSERT_EQ(3U, window_tabs.size());
|
| - // The first one is from the original set of tabs.
|
| - ASSERT_EQ(1, window_tabs[0]->tab_visual_index);
|
| - // The one from the original set of tabs wins over duplicating_tab1.
|
| - ASSERT_EQ(1, window_tabs[1]->tab_visual_index);
|
| - // duplicating_tab2 wins due to the later timestamp.
|
| - ASSERT_EQ(3, window_tabs[2]->tab_visual_index);
|
| -}
|
| -
|
| -// Verify that GetAllForeignSessions returns all sessions sorted by recency.
|
| -TEST_F(SessionsSyncManagerTest, GetAllForeignSessions) {
|
| - SessionID::id_type ids[] = {5, 10, 13, 17};
|
| - std::vector<SessionID::id_type> tab_list(ids, ids + arraysize(ids));
|
| -
|
| - const std::string kTag = "tag1";
|
| - std::vector<sync_pb::SessionSpecifics> tabs1;
|
| - sync_pb::SessionSpecifics meta1(helper()->BuildForeignSession(
|
| - kTag, tab_list, &tabs1));
|
| -
|
| - const std::string kTag2 = "tag2";
|
| - std::vector<sync_pb::SessionSpecifics> tabs2;
|
| - sync_pb::SessionSpecifics meta2(helper()->BuildForeignSession(
|
| - kTag2, tab_list, &tabs2));
|
| -
|
| - syncer::SyncDataList initial_data;
|
| - initial_data.push_back(
|
| - CreateRemoteData(meta1, base::Time::FromInternalValue(10)));
|
| - AddTabsToSyncDataList(tabs1, &initial_data);
|
| - initial_data.push_back(
|
| - CreateRemoteData(meta2, base::Time::FromInternalValue(200)));
|
| - AddTabsToSyncDataList(tabs2, &initial_data);
|
| -
|
| - syncer::SyncChangeList output;
|
| - InitWithSyncDataTakeOutput(initial_data, &output);
|
| -
|
| - std::vector<const SyncedSession*> foreign_sessions;
|
| - ASSERT_TRUE(manager()->GetAllForeignSessions(&foreign_sessions));
|
| - ASSERT_EQ(2U, foreign_sessions.size());
|
| - ASSERT_GT(foreign_sessions[0]->modified_time,
|
| - foreign_sessions[1]->modified_time);
|
| -}
|
| -
|
| -// Verify that GetForeignSessionTabs returns all tabs for a session sorted
|
| -// by recency.
|
| -TEST_F(SessionsSyncManagerTest, GetForeignSessionTabs) {
|
| - const std::string kTag = "tag1";
|
| -
|
| - SessionID::id_type n1[] = {5, 10, 13, 17};
|
| - std::vector<SessionID::id_type> tab_list1(n1, n1 + arraysize(n1));
|
| - std::vector<sync_pb::SessionSpecifics> tabs1;
|
| - sync_pb::SessionSpecifics meta(helper()->BuildForeignSession(
|
| - kTag, tab_list1, &tabs1));
|
| - // Add a second window.
|
| - SessionID::id_type n2[] = {7, 15, 18, 20};
|
| - std::vector<SessionID::id_type> tab_list2(n2, n2 + arraysize(n2));
|
| - helper()->AddWindowSpecifics(1, tab_list2, &meta);
|
| -
|
| - // Set up initial data.
|
| - syncer::SyncDataList initial_data;
|
| - initial_data.push_back(CreateRemoteData(meta));
|
| -
|
| - // Add the first window's tabs.
|
| - AddTabsToSyncDataList(tabs1, &initial_data);
|
| -
|
| - // Add the second window's tabs.
|
| - for (size_t i = 0; i < tab_list2.size(); ++i) {
|
| - sync_pb::EntitySpecifics entity;
|
| - helper()->BuildTabSpecifics(kTag, 0, tab_list2[i],
|
| - entity.mutable_session());
|
| - // Order the tabs oldest to most ReceiveDuplicateUnassociatedTabs and
|
| - // left to right visually.
|
| - initial_data.push_back(
|
| - CreateRemoteData(entity, base::Time::FromInternalValue(i + 1)));
|
| - }
|
| -
|
| - syncer::SyncChangeList output;
|
| - InitWithSyncDataTakeOutput(initial_data, &output);
|
| -
|
| - std::vector<const sessions::SessionTab*> tabs;
|
| - ASSERT_TRUE(manager()->GetForeignSessionTabs(kTag, &tabs));
|
| - // Assert that the size matches the total number of tabs and that the order
|
| - // is from most recent to least.
|
| - ASSERT_EQ(tab_list1.size() + tab_list2.size(), tabs.size());
|
| - base::Time last_time;
|
| - for (size_t i = 0; i < tabs.size(); ++i) {
|
| - base::Time this_time = tabs[i]->timestamp;
|
| - if (i > 0)
|
| - ASSERT_GE(last_time, this_time);
|
| - last_time = tabs[i]->timestamp;
|
| - }
|
| -}
|
| -
|
| -} // namespace sync_sessions
|
|
|
Chromium Code Reviews
Issue 2706343004:
[Sync] Refactor SessionsSyncManager unit tests (Closed)
