Commit 44ea159c authored by Marc Treib's avatar Marc Treib Committed by Commit Bot

Clean up syncer::StopSource enum

- Deprecate the PROFILE_DESTRUCTION entry which was unused.
- Add explicit numeric values to the remaining entries (as is
  recommended for enums used in UMA histograms).

Bug: 939752
Change-Id: Iedd144b5a958a4bd55c78b1e25fba23d8d9f2ed4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2467956
Auto-Submit: Marc Treib <treib@chromium.org>
Reviewed-by: default avatarMaksim Moskvitin <mmoskvitin@google.com>
Commit-Queue: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#816578}
parent 02693335
...@@ -13,13 +13,13 @@ namespace syncer { ...@@ -13,13 +13,13 @@ namespace syncer {
// A Java counterpart will be generated for this enum. // A Java counterpart will be generated for this enum.
// GENERATED_JAVA_ENUM_PACKAGE: org.chromium.components.sync // GENERATED_JAVA_ENUM_PACKAGE: org.chromium.components.sync
enum StopSource { enum StopSource {
PROFILE_DESTRUCTION, // The user destroyed the profile. // Deprecated: PROFILE_DESTRUCTION = 0,
SIGN_OUT, // The user signed out of Chrome. SIGN_OUT = 1, // The user signed out of Chrome.
BIRTHDAY_ERROR, // A dashboard stop-and-clear on the server. BIRTHDAY_ERROR = 2, // A dashboard stop-and-clear on the server.
CHROME_SYNC_SETTINGS, // The on/off switch in settings for mobile Chrome. CHROME_SYNC_SETTINGS = 3, // The on/off switch in settings for mobile Chrome.
ANDROID_CHROME_SYNC, // Android's sync setting for Chrome. ANDROID_CHROME_SYNC = 4, // Android's sync setting for Chrome.
ANDROID_MASTER_SYNC, // Android's master sync setting. ANDROID_MASTER_SYNC = 5, // Android's master sync setting.
STOP_SOURCE_LIMIT, STOP_SOURCE_LIMIT = 6,
}; };
} // namespace syncer } // namespace syncer
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment