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

Unified Diff: components/browser_sync/profile_sync_service_unittest.cc

Issue 2388673002: Revert of [Sync] Move //components/sync to the syncer namespace. (patchset #5 id:40001 of https://co (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/browser_sync/profile_sync_service_unittest.cc
diff --git a/components/browser_sync/profile_sync_service_unittest.cc b/components/browser_sync/profile_sync_service_unittest.cc
index 53a47799328080f7f53157c68344abd0c2fdd9bb..b0e6a8969313c68a26512ff6809a091f535e80a7 100644
--- a/components/browser_sync/profile_sync_service_unittest.cc
+++ b/components/browser_sync/profile_sync_service_unittest.cc
@@ -47,7 +47,6 @@
#include "testing/gtest/include/gtest/gtest.h"
#include "ui/base/l10n/l10n_util.h"
-using syncer::SyncBackendHostMock;
using testing::Return;
namespace browser_sync {
@@ -57,7 +56,7 @@ namespace {
const char kGaiaId[] = "12345";
const char kEmail[] = "test_user@gmail.com";
-class FakeDataTypeManager : public syncer::DataTypeManager {
+class FakeDataTypeManager : public sync_driver::DataTypeManager {
public:
typedef base::Callback<void(syncer::ConfigureReason)> ConfigureCalled;
@@ -77,7 +76,9 @@ class FakeDataTypeManager : public syncer::DataTypeManager {
void PurgeForMigration(syncer::ModelTypeSet undesired_types,
syncer::ConfigureReason reason) override {}
void Stop() override{};
- State state() const override { return syncer::DataTypeManager::CONFIGURED; };
+ State state() const override {
+ return sync_driver::DataTypeManager::CONFIGURED;
+ };
private:
ConfigureCalled configure_called_;
@@ -91,7 +92,7 @@ using testing::Return;
using testing::StrictMock;
using testing::_;
-class TestSyncServiceObserver : public syncer::SyncServiceObserver {
+class TestSyncServiceObserver : public sync_driver::SyncServiceObserver {
public:
explicit TestSyncServiceObserver(ProfileSyncService* service)
: service_(service), setup_in_progress_(false) {}
@@ -110,7 +111,7 @@ class TestSyncServiceObserver : public syncer::SyncServiceObserver {
// that could happen while backend init is in progress.
class SyncBackendHostNoReturn : public SyncBackendHostMock {
void Initialize(
- syncer::SyncFrontend* frontend,
+ sync_driver::SyncFrontend* frontend,
std::unique_ptr<base::Thread> sync_thread,
const scoped_refptr<base::SingleThreadTaskRunner>& db_thread,
const scoped_refptr<base::SingleThreadTaskRunner>& file_thread,
@@ -135,7 +136,7 @@ class SyncBackendHostMockCollectDeleteDirParam : public SyncBackendHostMock {
: delete_dir_param_(delete_dir_param) {}
void Initialize(
- syncer::SyncFrontend* frontend,
+ sync_driver::SyncFrontend* frontend,
std::unique_ptr<base::Thread> sync_thread,
const scoped_refptr<base::SingleThreadTaskRunner>& db_thread,
const scoped_refptr<base::SingleThreadTaskRunner>& file_thread,
@@ -247,7 +248,8 @@ class ProfileSyncServiceTest : public ::testing::Test {
service_.reset(new ProfileSyncService(std::move(init_params)));
service_->RegisterDataTypeController(
- base::MakeUnique<syncer::FakeDataTypeController>(syncer::BOOKMARKS));
+ base::MakeUnique<sync_driver::FakeDataTypeController>(
+ syncer::BOOKMARKS));
}
#if defined(OS_WIN) || defined(OS_MACOSX) || \
@@ -267,7 +269,7 @@ class ProfileSyncServiceTest : public ::testing::Test {
void InitializeForNthSync() {
// Set first sync time before initialize to simulate a complete sync setup.
- syncer::SyncPrefs sync_prefs(prefs());
+ sync_driver::SyncPrefs sync_prefs(prefs());
sync_prefs.SetFirstSyncTime(base::Time::Now());
sync_prefs.SetFirstSetupComplete();
sync_prefs.SetKeepEverythingSynced(true);
@@ -286,8 +288,8 @@ class ProfileSyncServiceTest : public ::testing::Test {
}
void OnConfigureCalled(syncer::ConfigureReason configure_reason) {
- syncer::DataTypeManager::ConfigureResult result;
- result.status = syncer::DataTypeManager::OK;
+ sync_driver::DataTypeManager::ConfigureResult result;
+ result.status = sync_driver::DataTypeManager::OK;
service()->OnConfigureDone(result);
}
@@ -367,7 +369,7 @@ class ProfileSyncServiceTest : public ::testing::Test {
return profile_sync_service_bundle_.pref_service();
}
- syncer::SyncApiComponentFactoryMock* component_factory() {
+ SyncApiComponentFactoryMock* component_factory() {
return component_factory_;
}
@@ -386,7 +388,7 @@ class ProfileSyncServiceTest : public ::testing::Test {
// The current component factory used by sync. May be null if the server
// hasn't been created yet.
- syncer::SyncApiComponentFactoryMock* component_factory_;
+ SyncApiComponentFactoryMock* component_factory_;
};
// Verify that the server URLs are sane.
@@ -394,13 +396,13 @@ TEST_F(ProfileSyncServiceTest, InitialState) {
CreateService(ProfileSyncService::AUTO_START);
InitializeForNthSync();
const std::string& url = service()->sync_service_url().spec();
- EXPECT_TRUE(url == syncer::internal::kSyncServerUrl ||
- url == syncer::internal::kSyncDevServerUrl);
+ EXPECT_TRUE(url == internal::kSyncServerUrl ||
+ url == internal::kSyncDevServerUrl);
}
// Verify a successful initialization.
TEST_F(ProfileSyncServiceTest, SuccessfulInitialization) {
- prefs()->SetManagedPref(syncer::prefs::kSyncManaged,
+ prefs()->SetManagedPref(sync_driver::prefs::kSyncManaged,
new base::FundamentalValue(false));
IssueTestTokens();
CreateService(ProfileSyncService::AUTO_START);
@@ -414,11 +416,11 @@ TEST_F(ProfileSyncServiceTest, SuccessfulInitialization) {
// Verify that an initialization where first setup is not complete does not
// start up the backend.
TEST_F(ProfileSyncServiceTest, NeedsConfirmation) {
- prefs()->SetManagedPref(syncer::prefs::kSyncManaged,
+ prefs()->SetManagedPref(sync_driver::prefs::kSyncManaged,
new base::FundamentalValue(false));
IssueTestTokens();
CreateService(ProfileSyncService::MANUAL_START);
- syncer::SyncPrefs sync_prefs(prefs());
+ sync_driver::SyncPrefs sync_prefs(prefs());
base::Time now = base::Time::Now();
sync_prefs.SetLastSyncedTime(now);
sync_prefs.SetKeepEverythingSynced(true);
@@ -450,7 +452,7 @@ TEST_F(ProfileSyncServiceTest, SetupInProgress) {
// Verify that disable by enterprise policy works.
TEST_F(ProfileSyncServiceTest, DisabledByPolicyBeforeInit) {
- prefs()->SetManagedPref(syncer::prefs::kSyncManaged,
+ prefs()->SetManagedPref(sync_driver::prefs::kSyncManaged,
new base::FundamentalValue(true));
IssueTestTokens();
CreateService(ProfileSyncService::AUTO_START);
@@ -471,7 +473,7 @@ TEST_F(ProfileSyncServiceTest, DisabledByPolicyAfterInit) {
EXPECT_FALSE(service()->IsManaged());
EXPECT_TRUE(service()->IsSyncActive());
- prefs()->SetManagedPref(syncer::prefs::kSyncManaged,
+ prefs()->SetManagedPref(sync_driver::prefs::kSyncManaged,
new base::FundamentalValue(true));
EXPECT_TRUE(service()->IsManaged());
@@ -521,20 +523,20 @@ TEST_F(ProfileSyncServiceTest, DisableAndEnableSyncTemporarily) {
InitializeForNthSync();
EXPECT_TRUE(service()->IsSyncActive());
- EXPECT_FALSE(prefs()->GetBoolean(syncer::prefs::kSyncSuppressStart));
+ EXPECT_FALSE(prefs()->GetBoolean(sync_driver::prefs::kSyncSuppressStart));
testing::Mock::VerifyAndClearExpectations(component_factory());
service()->RequestStop(ProfileSyncService::KEEP_DATA);
EXPECT_FALSE(service()->IsSyncActive());
- EXPECT_TRUE(prefs()->GetBoolean(syncer::prefs::kSyncSuppressStart));
+ EXPECT_TRUE(prefs()->GetBoolean(sync_driver::prefs::kSyncSuppressStart));
ExpectDataTypeManagerCreation(1, GetDefaultConfigureCalledCallback());
ExpectSyncBackendHostCreation(1);
service()->RequestStart();
EXPECT_TRUE(service()->IsSyncActive());
- EXPECT_FALSE(prefs()->GetBoolean(syncer::prefs::kSyncSuppressStart));
+ EXPECT_FALSE(prefs()->GetBoolean(sync_driver::prefs::kSyncSuppressStart));
}
// Certain ProfileSyncService tests don't apply to Chrome OS, for example
@@ -548,7 +550,7 @@ TEST_F(ProfileSyncServiceTest, EnableSyncAndSignOut) {
InitializeForNthSync();
EXPECT_TRUE(service()->IsSyncActive());
- EXPECT_FALSE(prefs()->GetBoolean(syncer::prefs::kSyncSuppressStart));
+ EXPECT_FALSE(prefs()->GetBoolean(sync_driver::prefs::kSyncSuppressStart));
signin_manager()->SignOut(signin_metrics::SIGNOUT_TEST,
signin_metrics::SignoutDelete::IGNORE_METRIC);
@@ -683,14 +685,16 @@ TEST_F(ProfileSyncServiceTest, MemoryPressureRecording) {
InitializeForNthSync();
EXPECT_TRUE(service()->IsSyncActive());
- EXPECT_FALSE(prefs()->GetBoolean(syncer::prefs::kSyncSuppressStart));
+ EXPECT_FALSE(prefs()->GetBoolean(sync_driver::prefs::kSyncSuppressStart));
testing::Mock::VerifyAndClearExpectations(component_factory());
- syncer::SyncPrefs sync_prefs(service()->GetSyncClient()->GetPrefService());
+ sync_driver::SyncPrefs sync_prefs(
+ service()->GetSyncClient()->GetPrefService());
- EXPECT_EQ(prefs()->GetInteger(syncer::prefs::kSyncMemoryPressureWarningCount),
- 0);
+ EXPECT_EQ(
+ prefs()->GetInteger(sync_driver::prefs::kSyncMemoryPressureWarningCount),
+ 0);
EXPECT_FALSE(sync_prefs.DidSyncShutdownCleanly());
// Simulate memory pressure notification.
@@ -699,8 +703,9 @@ TEST_F(ProfileSyncServiceTest, MemoryPressureRecording) {
base::RunLoop().RunUntilIdle();
// Verify memory pressure recorded.
- EXPECT_EQ(prefs()->GetInteger(syncer::prefs::kSyncMemoryPressureWarningCount),
- 1);
+ EXPECT_EQ(
+ prefs()->GetInteger(sync_driver::prefs::kSyncMemoryPressureWarningCount),
+ 1);
EXPECT_FALSE(sync_prefs.DidSyncShutdownCleanly());
// Simulate memory pressure notification.
@@ -710,8 +715,9 @@ TEST_F(ProfileSyncServiceTest, MemoryPressureRecording) {
ShutdownAndDeleteService();
// Verify memory pressure and shutdown recorded.
- EXPECT_EQ(prefs()->GetInteger(syncer::prefs::kSyncMemoryPressureWarningCount),
- 2);
+ EXPECT_EQ(
+ prefs()->GetInteger(sync_driver::prefs::kSyncMemoryPressureWarningCount),
+ 2);
EXPECT_TRUE(sync_prefs.DidSyncShutdownCleanly());
}
@@ -737,8 +743,8 @@ TEST_F(ProfileSyncServiceTest, OnLocalSetPassphraseEncryption) {
EXPECT_TRUE(service()->IsSyncActive());
testing::Mock::VerifyAndClearExpectations(component_factory());
EXPECT_EQ(syncer::CONFIGURE_REASON_NEWLY_ENABLED_DATA_TYPE, configure_reason);
- syncer::DataTypeManager::ConfigureResult result;
- result.status = syncer::DataTypeManager::OK;
+ sync_driver::DataTypeManager::ConfigureResult result;
+ result.status = sync_driver::DataTypeManager::OK;
service()->OnConfigureDone(result);
// Simulate user entering encryption passphrase. Ensure that catch up
@@ -781,8 +787,8 @@ TEST_F(ProfileSyncServiceTest,
InitializeForNthSync();
testing::Mock::VerifyAndClearExpectations(component_factory());
EXPECT_EQ(syncer::CONFIGURE_REASON_NEWLY_ENABLED_DATA_TYPE, configure_reason);
- syncer::DataTypeManager::ConfigureResult result;
- result.status = syncer::DataTypeManager::OK;
+ sync_driver::DataTypeManager::ConfigureResult result;
+ result.status = sync_driver::DataTypeManager::OK;
service()->OnConfigureDone(result);
// Simulate user entering encryption passphrase. Ensure Configure was called
@@ -853,8 +859,8 @@ TEST_F(ProfileSyncServiceTest,
EXPECT_TRUE(captured_callback.is_null());
// Simulate configure successful. This time it should be catch up.
- syncer::DataTypeManager::ConfigureResult result;
- result.status = syncer::DataTypeManager::OK;
+ sync_driver::DataTypeManager::ConfigureResult result;
+ result.status = sync_driver::DataTypeManager::OK;
service()->OnConfigureDone(result);
EXPECT_EQ(syncer::CONFIGURE_REASON_CATCH_UP, configure_reason);
EXPECT_TRUE(captured_callback.is_null());
@@ -881,7 +887,8 @@ TEST_F(ProfileSyncServiceTest, PassphrasePromptDueToVersion) {
ExpectSyncBackendHostCreation(1);
InitializeForNthSync();
- syncer::SyncPrefs sync_prefs(service()->GetSyncClient()->GetPrefService());
+ sync_driver::SyncPrefs sync_prefs(
+ service()->GetSyncClient()->GetPrefService());
EXPECT_EQ(PRODUCT_VERSION, sync_prefs.GetLastRunVersion());
sync_prefs.SetPassphrasePrompted(true);
« no previous file with comments | « components/browser_sync/profile_sync_service_typed_url_unittest.cc ('k') | components/browser_sync/profile_sync_test_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698