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

Unified Diff: components/sync/driver/sync_driver_switches.cc

Issue 2956333002: [Sync] Enable UserEvents by default. (Closed)
Patch Set: Maybe iOS will work this time? Created 3 years, 5 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/sync/driver/sync_driver_switches.cc
diff --git a/components/sync/driver/sync_driver_switches.cc b/components/sync/driver/sync_driver_switches.cc
index 1b346e21c5b4ab1228035273a574b1c530a0ea83..c93cb0a36e685fb2456fff7aef634ef1c2186e00 100644
--- a/components/sync/driver/sync_driver_switches.cc
+++ b/components/sync/driver/sync_driver_switches.cc
@@ -36,9 +36,10 @@ const char kSyncShortNudgeDelayForTest[] = "sync-short-nudge-delay-for-test";
const base::Feature kSyncClearDataOnPassphraseEncryption{
"ClearSyncDataOnPassphraseEncryption", base::FEATURE_DISABLED_BY_DEFAULT};
-// Gates registration and construction of user events machinery.
+// Gates registration and construction of user events machinery. Enabled by
+// default as each use case should have their own gating feature as well.
const base::Feature kSyncUserEvents{"SyncUserEvents",
- base::FEATURE_DISABLED_BY_DEFAULT};
+ base::FEATURE_ENABLED_BY_DEFAULT};
// Gates registration for user language detection events.
const base::Feature kSyncUserLanguageDetectionEvents{

Powered by Google App Engine
This is Rietveld 408576698