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

Unified Diff: chrome/browser/sync/profile_sync_components_factory_impl.cc

Issue 9729012: Reland 127510 - Enable tab sync on all platforms by default. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 9 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: chrome/browser/sync/profile_sync_components_factory_impl.cc
diff --git a/chrome/browser/sync/profile_sync_components_factory_impl.cc b/chrome/browser/sync/profile_sync_components_factory_impl.cc
index c5ef8d3acc38b0108fed9601bd00174e96855279..9dd3fc902d1f49730e04e3f6c040edac7764e82c 100644
--- a/chrome/browser/sync/profile_sync_components_factory_impl.cc
+++ b/chrome/browser/sync/profile_sync_components_factory_impl.cc
@@ -154,10 +154,10 @@ void ProfileSyncComponentsFactoryImpl::RegisterDataTypes(
new SearchEngineDataTypeController(this, profile_, pss));
}
- // Session sync is disabled by default, but has some logic to automatically
- // enable. Register only if explicitly enabled and not explicitly disabled.
- if (command_line_->HasSwitch(switches::kEnableSyncTabs) &&
- !command_line_->HasSwitch(switches::kDisableSyncTabs)) {
+ // Session sync is enabled by default. Register unless explicitly disabled.
+ // TODO(dubroy): Once this change is permanent, clean up the logic for
+ // explicitly enabling tab sync (http://crbug.com/118570).
+ if (!command_line_->HasSwitch(switches::kDisableSyncTabs)) {
pss->RegisterDataTypeController(
new SessionDataTypeController(this, profile_, pss));
}
« no previous file with comments | « chrome/browser/sync/internal_api/sync_manager.cc ('k') | chrome/browser/sync/profile_sync_components_factory_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698