Index: chrome/browser/chromeos/arc/arc_optin_uma.h |
diff --git a/chrome/browser/chromeos/arc/arc_optin_uma.h b/chrome/browser/chromeos/arc/arc_optin_uma.h |
index 3b5844f058e4efa9a4723e6087d9f5fe3c3fe038..bdbf7a9f772da77efc4d0a35a0fbea24111f123b 100644 |
--- a/chrome/browser/chromeos/arc/arc_optin_uma.h |
+++ b/chrome/browser/chromeos/arc/arc_optin_uma.h |
@@ -119,6 +119,26 @@ enum class ProvisioningResult : int { |
SIZE, |
}; |
+enum class AccountCheckStatus : int { |
Ilya Sherman
2017/03/24 23:22:08
Please document that this enum is used to back an
khmel
2017/03/24 23:51:53
Done.
|
+ // Checked account exists and is up to date. |
+ UP_TO_DATE = 0, |
+ |
+ // Checked account is new in the system. |
+ NEW = 1, |
+ |
+ // Checked account exists but need to be reauthorized. |
+ NEED_REAUTH = 2, |
+ |
+ // Checked account exists but it is in unknown state. |
+ UNKNOWN = 3, |
+ |
+ // Account check has failed. |
+ CHECK_FAILED = 4, |
+ |
+ // The size of this enum; keep last. |
+ SIZE, |
dcheng
2017/03/25 00:55:04
Another possibility is to use the mojo enum throug
khmel
2017/03/25 01:24:14
This good discussion. But we already use similar m
yzshen1
2017/03/27 16:53:05
(Sorry for late reply! I didn't notice this earlie
Ilya Sherman
2017/03/27 17:00:40
FWIW, you don't have to add a new value to the moj
yzshen1
2017/03/27 17:07:46
WRT the "max + 1" approach: if "max" is not named
khmel
2017/03/27 17:17:38
Thank you for you comments! What is about includin
|
+}; |
+ |
void UpdateOptInActionUMA(OptInActionType type); |
void UpdateOptInCancelUMA(OptInCancelReason reason); |
void UpdateEnabledStateUMA(bool enabled); |
@@ -129,6 +149,7 @@ void UpdateProvisioningTiming(const base::TimeDelta& elapsed_time, |
void UpdateSilentAuthCodeUMA(OptInSilentAuthCode state); |
void UpdateAuthTiming(const char* histogram_name, base::TimeDelta elapsed_time); |
void UpdateAuthCheckinAttempts(int32_t num_attempts); |
+void UpdateAuthAccountCheckStatus(AccountCheckStatus status); |
// Outputs the stringified |result| to |os|. This is only for logging purposes. |
std::ostream& operator<<(std::ostream& os, const ProvisioningResult& result); |