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

Unified Diff: components/signin/core/browser/signin_metrics.h

Issue 1087933002: Cross Device Promo - Main Eligibility Flow (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: pre-review Created 5 years, 7 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/signin/core/browser/signin_metrics.h
diff --git a/components/signin/core/browser/signin_metrics.h b/components/signin/core/browser/signin_metrics.h
index aa043c25c3e2b3beafb071880f71282bd75ad6e3..c3eafa630db529246a6419ccd44f11a26f133171 100644
--- a/components/signin/core/browser/signin_metrics.h
+++ b/components/signin/core/browser/signin_metrics.h
@@ -126,6 +126,31 @@ enum AccountReauth {
HISTOGRAM_REAUTH_MAX
};
+// Enum values used for "Signin.XDevicePromo.Eligible" histogram, which tracks
+// the reasons for which a profile is or is not eligible for the promo.
+enum CrossDevicePromoEligibility {
+ // The profile has previously opted out of the promo.
+ OPTED_OUT,
+ // The profile is already signed in.
+ SIGNED_IN,
+ // The profile does not have a single, peristent GAIA cookie.
+ NOT_SINGLE_GAIA_ACCOUNT,
+ // Yet to determine how many devices the user has.
+ UNKNOWN_COUNT_DEVICES,
+ // An error was returned trying to determine the account's devices.
+ ERROR_FETCHING_DEVICE_ACTIVITY,
+ // The call to get device activity was throttled, and never executed.
+ THROTTLED_FETCHING_DEVICE_ACTIVITY,
+ // The user has no devices.
+ ZERO_DEVICES,
+ // The user has no device that was recently active.
+ NO_ACTIVE_DEVICES,
+ // The user is eligible for the promo.
+ ELIGIBLE,
+ // Always last enumerated type.
+ NUM_CROSS_DEVICE_PROMO_ELIGIBILITY_METRICS
+};
+
// Log to UMA histograms and UserCounts stats about a single execution of the
// AccountReconciler.
// |total_number_accounts| - How many accounts are in the browser for this
@@ -171,6 +196,10 @@ void LogAuthError(GoogleServiceAuthError::State auth_error);
void LogSigninConfirmHistogramValue(int action);
+void LogXDevicePromoEligible(CrossDevicePromoEligibility metric);
+
+void LogBrowsingSessionDuration(const base::Time& previous_activity);
+
} // namespace signin_metrics
#endif // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_

Powered by Google App Engine
This is Rietveld 408576698