| Index: components/browser_sync/profile_sync_service_startup_unittest.cc
|
| diff --git a/components/browser_sync/profile_sync_service_startup_unittest.cc b/components/browser_sync/profile_sync_service_startup_unittest.cc
|
| index 68944ca021a6b789b33bff9cb6bc2b1c3ff7cd81..7e9d8b9a59fdf6fb9e39c29d84ff7619611b6197 100644
|
| --- a/components/browser_sync/profile_sync_service_startup_unittest.cc
|
| +++ b/components/browser_sync/profile_sync_service_startup_unittest.cc
|
| @@ -33,8 +33,9 @@
|
| #include "testing/gmock/include/gmock/gmock.h"
|
| #include "testing/gtest/include/gtest/gtest.h"
|
|
|
| -using sync_driver::DataTypeManager;
|
| -using sync_driver::DataTypeManagerMock;
|
| +using syncer::DataTypeManager;
|
| +using syncer::DataTypeManagerMock;
|
| +using syncer::SyncBackendHostMock;
|
| using testing::_;
|
| using testing::AnyNumber;
|
| using testing::DoAll;
|
| @@ -49,7 +50,7 @@ const char kGaiaId[] = "12345";
|
| const char kEmail[] = "test_user@gmail.com";
|
| const char kDummyPassword[] = "";
|
|
|
| -class SyncServiceObserverMock : public sync_driver::SyncServiceObserver {
|
| +class SyncServiceObserverMock : public syncer::SyncServiceObserver {
|
| public:
|
| SyncServiceObserverMock();
|
| virtual ~SyncServiceObserverMock();
|
| @@ -70,7 +71,7 @@ ACTION_P(InvokeOnConfigureStart, sync_service) {
|
| ACTION_P3(InvokeOnConfigureDone, sync_service, error_callback, result) {
|
| DataTypeManager::ConfigureResult configure_result =
|
| static_cast<DataTypeManager::ConfigureResult>(result);
|
| - if (result.status == sync_driver::DataTypeManager::ABORTED)
|
| + if (result.status == syncer::DataTypeManager::ABORTED)
|
| error_callback.Run(&configure_result);
|
| sync_service->OnConfigureDone(configure_result);
|
| }
|
| @@ -97,8 +98,7 @@ class ProfileSyncServiceStartupTest : public testing::Test {
|
|
|
| sync_service_.reset(new ProfileSyncService(std::move(init_params)));
|
| sync_service_->RegisterDataTypeController(
|
| - base::MakeUnique<sync_driver::FakeDataTypeController>(
|
| - syncer::BOOKMARKS));
|
| + base::MakeUnique<syncer::FakeDataTypeController>(syncer::BOOKMARKS));
|
| sync_service_->AddObserver(&observer_);
|
| }
|
|
|
| @@ -108,7 +108,7 @@ class ProfileSyncServiceStartupTest : public testing::Test {
|
| }
|
|
|
| void SetError(DataTypeManager::ConfigureResult* result) {
|
| - sync_driver::DataTypeStatusTable::TypeErrorMap errors;
|
| + syncer::DataTypeStatusTable::TypeErrorMap errors;
|
| errors[syncer::BOOKMARKS] =
|
| syncer::SyncError(FROM_HERE, syncer::SyncError::UNRECOVERABLE_ERROR,
|
| "Error", syncer::BOOKMARKS);
|
| @@ -155,8 +155,8 @@ class ProfileSyncServiceStartupTest : public testing::Test {
|
| ProfileSyncServiceBundle profile_sync_service_bundle_;
|
| std::unique_ptr<ProfileSyncService> sync_service_;
|
| SyncServiceObserverMock observer_;
|
| - sync_driver::DataTypeStatusTable data_type_status_table_;
|
| - SyncApiComponentFactoryMock* component_factory_ = nullptr;
|
| + syncer::DataTypeStatusTable data_type_status_table_;
|
| + syncer::SyncApiComponentFactoryMock* component_factory_ = nullptr;
|
| };
|
|
|
| class ProfileSyncServiceStartupCrosTest : public ProfileSyncServiceStartupTest {
|
| @@ -171,7 +171,7 @@ class ProfileSyncServiceStartupCrosTest : public ProfileSyncServiceStartupTest {
|
|
|
| TEST_F(ProfileSyncServiceStartupTest, StartFirstTime) {
|
| // We've never completed startup.
|
| - pref_service()->ClearPref(sync_driver::prefs::kSyncFirstSetupComplete);
|
| + pref_service()->ClearPref(syncer::prefs::kSyncFirstSetupComplete);
|
| CreateSyncService(ProfileSyncService::MANUAL_START);
|
| SetUpSyncBackendHost();
|
| DataTypeManagerMock* data_type_manager = SetUpDataTypeManager();
|
| @@ -183,10 +183,9 @@ TEST_F(ProfileSyncServiceStartupTest, StartFirstTime) {
|
| sync_service_->Initialize();
|
|
|
| // Preferences should be back to defaults.
|
| - EXPECT_EQ(0,
|
| - pref_service()->GetInt64(sync_driver::prefs::kSyncLastSyncedTime));
|
| + EXPECT_EQ(0, pref_service()->GetInt64(syncer::prefs::kSyncLastSyncedTime));
|
| EXPECT_FALSE(
|
| - pref_service()->GetBoolean(sync_driver::prefs::kSyncFirstSetupComplete));
|
| + pref_service()->GetBoolean(syncer::prefs::kSyncFirstSetupComplete));
|
| Mock::VerifyAndClearExpectations(data_type_manager);
|
|
|
| // Then start things up.
|
| @@ -213,7 +212,7 @@ TEST_F(ProfileSyncServiceStartupTest, StartFirstTime) {
|
| // TODO(pavely): Reenable test once android is switched to oauth2.
|
| TEST_F(ProfileSyncServiceStartupTest, DISABLED_StartNoCredentials) {
|
| // We've never completed startup.
|
| - pref_service()->ClearPref(sync_driver::prefs::kSyncFirstSetupComplete);
|
| + pref_service()->ClearPref(syncer::prefs::kSyncFirstSetupComplete);
|
| CreateSyncService(ProfileSyncService::MANUAL_START);
|
|
|
| // Should not actually start, rather just clean things up and wait
|
| @@ -224,10 +223,9 @@ TEST_F(ProfileSyncServiceStartupTest, DISABLED_StartNoCredentials) {
|
| sync_service_->Initialize();
|
|
|
| // Preferences should be back to defaults.
|
| - EXPECT_EQ(0,
|
| - pref_service()->GetInt64(sync_driver::prefs::kSyncLastSyncedTime));
|
| + EXPECT_EQ(0, pref_service()->GetInt64(syncer::prefs::kSyncLastSyncedTime));
|
| EXPECT_FALSE(
|
| - pref_service()->GetBoolean(sync_driver::prefs::kSyncFirstSetupComplete));
|
| + pref_service()->GetBoolean(syncer::prefs::kSyncFirstSetupComplete));
|
|
|
| // Then start things up.
|
| auto sync_blocker = sync_service_->GetSetupInProgressHandle();
|
| @@ -284,7 +282,7 @@ TEST_F(ProfileSyncServiceStartupCrosTest, StartCrosNoCredentials) {
|
| EXPECT_CALL(*component_factory_, CreateDataTypeManager(_, _, _, _, _))
|
| .Times(0);
|
| EXPECT_CALL(*component_factory_, CreateSyncBackendHost(_, _, _, _)).Times(0);
|
| - pref_service()->ClearPref(sync_driver::prefs::kSyncFirstSetupComplete);
|
| + pref_service()->ClearPref(syncer::prefs::kSyncFirstSetupComplete);
|
| EXPECT_CALL(observer_, OnStateChanged()).Times(AnyNumber());
|
|
|
| sync_service_->Initialize();
|
| @@ -299,7 +297,7 @@ TEST_F(ProfileSyncServiceStartupCrosTest, StartCrosNoCredentials) {
|
| TEST_F(ProfileSyncServiceStartupCrosTest, StartFirstTime) {
|
| SetUpSyncBackendHost();
|
| DataTypeManagerMock* data_type_manager = SetUpDataTypeManager();
|
| - pref_service()->ClearPref(sync_driver::prefs::kSyncFirstSetupComplete);
|
| + pref_service()->ClearPref(syncer::prefs::kSyncFirstSetupComplete);
|
| EXPECT_CALL(*data_type_manager, Configure(_, _));
|
| EXPECT_CALL(*data_type_manager, state())
|
| .WillRepeatedly(Return(DataTypeManager::CONFIGURED));
|
| @@ -336,12 +334,12 @@ TEST_F(ProfileSyncServiceStartupTest, StartNormal) {
|
| // therefore being left unset.
|
| TEST_F(ProfileSyncServiceStartupTest, StartRecoverDatatypePrefs) {
|
| // Clear the datatype preference fields (simulating bug 154940).
|
| - pref_service()->ClearPref(sync_driver::prefs::kSyncKeepEverythingSynced);
|
| + pref_service()->ClearPref(syncer::prefs::kSyncKeepEverythingSynced);
|
| syncer::ModelTypeSet user_types = syncer::UserTypes();
|
| for (syncer::ModelTypeSet::Iterator iter = user_types.First(); iter.Good();
|
| iter.Inc()) {
|
| pref_service()->ClearPref(
|
| - sync_driver::SyncPrefs::GetPrefNameForDataType(iter.Get()));
|
| + syncer::SyncPrefs::GetPrefNameForDataType(iter.Get()));
|
| }
|
|
|
| // Pre load the tokens
|
| @@ -359,8 +357,8 @@ TEST_F(ProfileSyncServiceStartupTest, StartRecoverDatatypePrefs) {
|
| IssueTestTokens(account_id);
|
| sync_service_->Initialize();
|
|
|
| - EXPECT_TRUE(pref_service()->GetBoolean(
|
| - sync_driver::prefs::kSyncKeepEverythingSynced));
|
| + EXPECT_TRUE(
|
| + pref_service()->GetBoolean(syncer::prefs::kSyncKeepEverythingSynced));
|
| }
|
|
|
| // Verify that the recovery of datatype preferences doesn't overwrite a valid
|
| @@ -368,8 +366,7 @@ TEST_F(ProfileSyncServiceStartupTest, StartRecoverDatatypePrefs) {
|
| TEST_F(ProfileSyncServiceStartupTest, StartDontRecoverDatatypePrefs) {
|
| // Explicitly set Keep Everything Synced to false and have only bookmarks
|
| // enabled.
|
| - pref_service()->SetBoolean(sync_driver::prefs::kSyncKeepEverythingSynced,
|
| - false);
|
| + pref_service()->SetBoolean(syncer::prefs::kSyncKeepEverythingSynced, false);
|
|
|
| // Pre load the tokens
|
| CreateSyncService(ProfileSyncService::MANUAL_START);
|
| @@ -385,8 +382,8 @@ TEST_F(ProfileSyncServiceStartupTest, StartDontRecoverDatatypePrefs) {
|
| IssueTestTokens(account_id);
|
| sync_service_->Initialize();
|
|
|
| - EXPECT_FALSE(pref_service()->GetBoolean(
|
| - sync_driver::prefs::kSyncKeepEverythingSynced));
|
| + EXPECT_FALSE(
|
| + pref_service()->GetBoolean(syncer::prefs::kSyncKeepEverythingSynced));
|
| }
|
|
|
| TEST_F(ProfileSyncServiceStartupTest, ManagedStartup) {
|
| @@ -395,7 +392,7 @@ TEST_F(ProfileSyncServiceStartupTest, ManagedStartup) {
|
| CreateSyncService(ProfileSyncService::MANUAL_START);
|
|
|
| // Disable sync through policy.
|
| - pref_service()->SetBoolean(sync_driver::prefs::kSyncManaged, true);
|
| + pref_service()->SetBoolean(syncer::prefs::kSyncManaged, true);
|
| EXPECT_CALL(*component_factory_, CreateDataTypeManager(_, _, _, _, _))
|
| .Times(0);
|
| EXPECT_CALL(observer_, OnStateChanged()).Times(AnyNumber());
|
| @@ -423,7 +420,7 @@ TEST_F(ProfileSyncServiceStartupTest, SwitchManaged) {
|
| EXPECT_CALL(*data_type_manager, state())
|
| .WillOnce(Return(DataTypeManager::CONFIGURED));
|
| EXPECT_CALL(*data_type_manager, Stop()).Times(1);
|
| - pref_service()->SetBoolean(sync_driver::prefs::kSyncManaged, true);
|
| + pref_service()->SetBoolean(syncer::prefs::kSyncManaged, true);
|
| EXPECT_FALSE(sync_service_->IsBackendInitialized());
|
| // Note that PSS no longer references |data_type_manager| after stopping.
|
|
|
| @@ -433,7 +430,7 @@ TEST_F(ProfileSyncServiceStartupTest, SwitchManaged) {
|
| Mock::VerifyAndClearExpectations(data_type_manager);
|
| EXPECT_CALL(*component_factory_, CreateDataTypeManager(_, _, _, _, _))
|
| .Times(0);
|
| - pref_service()->ClearPref(sync_driver::prefs::kSyncManaged);
|
| + pref_service()->ClearPref(syncer::prefs::kSyncManaged);
|
| EXPECT_FALSE(sync_service_->IsBackendInitialized());
|
| EXPECT_FALSE(sync_service_->IsSyncActive());
|
| }
|
| @@ -469,7 +466,7 @@ TEST_F(ProfileSyncServiceStartupTest, StartDownloadFailed) {
|
| SyncBackendHostMock* mock_sbh = SetUpSyncBackendHost();
|
| mock_sbh->set_fail_initial_download(true);
|
|
|
| - pref_service()->ClearPref(sync_driver::prefs::kSyncFirstSetupComplete);
|
| + pref_service()->ClearPref(syncer::prefs::kSyncFirstSetupComplete);
|
|
|
| EXPECT_CALL(observer_, OnStateChanged()).Times(AnyNumber());
|
| sync_service_->Initialize();
|
|
|