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

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: build.gn 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..ce1b3fce08b7e3c948dd6c1ef59cd7cd8277b778 100644
--- a/components/signin/core/browser/signin_metrics.h
+++ b/components/signin/core/browser/signin_metrics.h
@@ -126,6 +126,43 @@ 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 user is eligible for the promo.
+ ELIGIBLE,
+ // 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,
+ // Always last enumerated type.
+ NUM_CROSS_DEVICE_PROMO_ELIGIBILITY_METRICS
+};
+
+// Enum reasons the CrossDevicePromo couldn't initialize, or that it succeeded.
+enum CrossDevicePromoInitialized {
+ // The promo was initialized successfully.
+ INITIALIZED,
+ // The profile is opted out, so the promo didn't initialize.
+ UNINITIALIZED_OPTED_OUT,
+ // Unable to read the variations configuration.
+ NO_VARIATIONS_CONFIG,
+ // Always the last enumerated type.
+ NUM_CROSS_DEVICE_PROMO_INITIALIZED_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 +208,12 @@ void LogAuthError(GoogleServiceAuthError::State auth_error);
void LogSigninConfirmHistogramValue(int action);
+void LogXDevicePromoEligible(CrossDevicePromoEligibility metric);
+
+void LogXDevicePromoInitialized(CrossDevicePromoInitialized metric);
+
+void LogBrowsingSessionDuration(const base::Time& previous_activity_time);
+
} // namespace signin_metrics
#endif // COMPONENTS_SIGNIN_CORE_BROWSER_SIGNIN_METRICS_H_
« no previous file with comments | « components/signin/core/browser/device_activity_fetcher.cc ('k') | components/signin/core/browser/signin_metrics.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698