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

Unified Diff: components/browser_sync/profile_sync_service.cc

Issue 2473613003: [Sync] Adds a new switch for enabling the new local server backend. (Closed)
Patch Set: Ifdef protect the variable. Created 4 years, 1 month 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.cc
diff --git a/components/browser_sync/profile_sync_service.cc b/components/browser_sync/profile_sync_service.cc
index f013dbc8d840baee3f730dd4501d34a646be487b..3ddedd855e477f37332bdb45c62946454e650eff 100644
--- a/components/browser_sync/profile_sync_service.cc
+++ b/components/browser_sync/profile_sync_service.cc
@@ -20,6 +20,7 @@
#include "base/memory/ptr_util.h"
#include "base/memory/ref_counted.h"
#include "base/metrics/histogram.h"
+#include "base/path_service.h"
#include "base/profiler/scoped_tracker.h"
#include "base/single_thread_task_runner.h"
#include "base/strings/stringprintf.h"
@@ -151,6 +152,11 @@ static const base::FilePath::CharType kSyncDataFolderName[] =
static const base::FilePath::CharType kLevelDBFolderName[] =
FILE_PATH_LITERAL("LevelDB");
+#if defined(OS_WIN)
+static const base::FilePath::CharType kLoopbackServerBackendFilename[] =
+ FILE_PATH_LITERAL("profile.pb");
+#endif
+
namespace {
// Perform the actual sync data folder deletion.
@@ -502,6 +508,36 @@ void ProfileSyncService::InitializeBackend(bool delete_stale_data) {
if (delete_stale_data)
ClearStaleErrors();
+ bool enable_local_sync_backend = false;
+ base::FilePath local_sync_backend_folder;
+#if defined(OS_WIN)
+ enable_local_sync_backend = base::CommandLine::ForCurrentProcess()->HasSwitch(
+ switches::kEnableLocalSyncBackend);
+ if (base::CommandLine::ForCurrentProcess()->HasSwitch(
+ switches::kLocalSyncBackendDir)) {
+ local_sync_backend_folder =
+ base::CommandLine::ForCurrentProcess()->GetSwitchValuePath(
+ switches::kLocalSyncBackendDir);
+ } else {
+ // TODO(pastarmovj): Add DIR_ROAMING_USER_DATA to PathService to simplify
+ // this code and move the logic in its right place. See crbug/657810.
+ CHECK(
+ base::PathService::Get(base::DIR_APP_DATA, &local_sync_backend_folder));
+ local_sync_backend_folder =
+ local_sync_backend_folder.Append(FILE_PATH_LITERAL("Chrome/User Data"));
+ }
+ // This code as it is now will assume the same profile order is present on all
+ // machines, which is not a given. It is to be defined if only the Default
+ // profile should get this treatment or all profile as is the case now. The
+ // solution for now will be to assume profiles are created in the same order
+ // on all machines and in the future decide if only the Default one should be
+ // considered roamed.
+ local_sync_backend_folder =
+ local_sync_backend_folder.Append(base_directory_.BaseName());
+ local_sync_backend_folder =
+ local_sync_backend_folder.Append(kLoopbackServerBackendFilename);
+#endif // defined(OS_WIN)
+
SyncBackendHost::HttpPostProviderFactoryGetter
http_post_provider_factory_getter =
base::Bind(&syncer::NetworkResources::GetHttpPostProviderFactory,
@@ -511,9 +547,9 @@ void ProfileSyncService::InitializeBackend(bool delete_stale_data) {
backend_->Initialize(
this, std::move(sync_thread_), db_thread_, file_thread_,
GetJsEventHandler(), sync_service_url_, local_device_->GetSyncUserAgent(),
- credentials, delete_stale_data,
- std::unique_ptr<syncer::SyncManagerFactory>(
- new syncer::SyncManagerFactory()),
+ credentials, delete_stale_data, enable_local_sync_backend,
+ local_sync_backend_folder, std::unique_ptr<syncer::SyncManagerFactory>(
+ new syncer::SyncManagerFactory()),
MakeWeakHandle(sync_enabled_weak_factory_.GetWeakPtr()),
base::Bind(syncer::ReportUnrecoverableError, channel_),
http_post_provider_factory_getter, std::move(saved_nigori_state_));
« no previous file with comments | « components/browser_sync/browser_sync_switches.cc ('k') | components/browser_sync/profile_sync_service_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698