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

Unified Diff: chrome/browser/chromeos/arc/arc_optin_uma.cc

Issue 2771943003: arc: Skip GMS Sign-In in case Android is already signed-in. (Closed)
Patch Set: use sparse UMA Created 3 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
« no previous file with comments | « chrome/browser/chromeos/arc/arc_optin_uma.h ('k') | components/arc/common/auth.mojom » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/arc/arc_optin_uma.cc
diff --git a/chrome/browser/chromeos/arc/arc_optin_uma.cc b/chrome/browser/chromeos/arc/arc_optin_uma.cc
index b52991a391af47a11c5d99cc826273855cf612c3..cdd627cc1d305bdfcaa19e7f1238a6ca913ce539 100644
--- a/chrome/browser/chromeos/arc/arc_optin_uma.cc
+++ b/chrome/browser/chromeos/arc/arc_optin_uma.cc
@@ -64,9 +64,16 @@ void UpdateAuthCheckinAttempts(int32_t num_attempts) {
UMA_HISTOGRAM_SPARSE_SLOWLY("ArcAuth.CheckinAttempts", num_attempts);
}
+void UpdateAuthAccountCheckStatus(mojom::AccountCheckStatus status) {
+ DCHECK_LE(status, mojom::AccountCheckStatus::CHECK_FAILED);
+ UMA_HISTOGRAM_ENUMERATION(
+ "ArcAuth.AccountCheckStatus", static_cast<int>(status),
+ static_cast<int>(mojom::AccountCheckStatus::CHECK_FAILED) + 1);
+}
+
void UpdateSilentAuthCodeUMA(OptInSilentAuthCode state) {
- UMA_HISTOGRAM_ENUMERATION("Arc.OptInSilentAuthCode", static_cast<int>(state),
- static_cast<int>(OptInSilentAuthCode::SIZE));
+ UMA_HISTOGRAM_SPARSE_SLOWLY("Arc.OptInSilentAuthCode",
+ static_cast<int>(state));
}
std::ostream& operator<<(std::ostream& os, const ProvisioningResult& result) {
« no previous file with comments | « chrome/browser/chromeos/arc/arc_optin_uma.h ('k') | components/arc/common/auth.mojom » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698