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

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

Issue 8334030: Merge search engines sync data type with Preferences. Sync the default search provider. Add some ... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Merge to TOT and fixed additional conflicts from rsimha. Created 9 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: chrome/browser/sync/profile_sync_factory_impl.cc
===================================================================
--- chrome/browser/sync/profile_sync_factory_impl.cc (revision 109292)
+++ chrome/browser/sync/profile_sync_factory_impl.cc (working copy)
@@ -153,6 +153,13 @@
new TypedUrlDataTypeController(this, profile_));
}
+ // Search Engine sync is enabled by default. Register only if explicitly
+ // disabled.
+ if (!command_line_->HasSwitch(switches::kDisableSyncSearchEngines)) {
+ pss->RegisterDataTypeController(
+ new SearchEngineDataTypeController(this, profile_, pss));
+ }
+
// Session sync is disabled by default. Register only if explicitly
// enabled.
if (command_line_->HasSwitch(switches::kEnableSyncTabs)) {
@@ -176,13 +183,6 @@
new AutofillProfileDataTypeController(this, profile_));
}
- // Search Engine sync is disabled by default. Register only if explicitly
- // enabled.
- if (command_line_->HasSwitch(switches::kEnableSyncSearchEngines)) {
- pss->RegisterDataTypeController(
- new SearchEngineDataTypeController(this, profile_, pss));
- }
-
// App notifications sync is disabled by default. Register only if
// explicitly enabled.
if (command_line_->HasSwitch(switches::kEnableSyncAppNotifications)) {

Powered by Google App Engine
This is Rietveld 408576698