Cross Device Promo - Main Eligibility Flow

components/signin/core/browser/signin_metrics.cc

Index: components/signin/core/browser/signin_metrics.cc
diff --git a/components/signin/core/browser/signin_metrics.cc b/components/signin/core/browser/signin_metrics.cc
index 4a2ea6bf809967496a52c673f63b1b87507bbf76..e338ca27bf188748a0370f69de73c155aee10d55 100644
--- a/components/signin/core/browser/signin_metrics.cc
+++ b/components/signin/core/browser/signin_metrics.cc
@@ -107,4 +107,23 @@ void LogSigninConfirmHistogramValue(int action) {
                             signin_metrics::HISTOGRAM_CONFIRM_MAX);
 }
 
+void LogXDevicePromoEligible(CrossDevicePromoEligibility metric) {
+  UMA_HISTOGRAM_ENUMERATION(
+      "Signin.XDevicePromo.Eligibility", metric,
+      CrossDevicePromoEligibility::NUM_CROSS_DEVICE_PROMO_ELIGIBILITY_METRICS);

[Alexei Svitkine (slow), 2015/05/19 17:10:38]

Nit: No need for the enum name prefix. Here and below.

[Mike Lerman, 2015/05/19 18:17:35]

Done.

+}
+
+void LogXDevicePromoInitialized(CrossDevicePromoInitialized metric) {
+  UMA_HISTOGRAM_ENUMERATION(
+      "Signin.XDevicePromo.Initialized", metric,
+      CrossDevicePromoInitialized::NUM_CROSS_DEVICE_PROMO_INITIALIZED_METRICS);
+}
+
+void LogBrowsingSessionDuration(const base::Time& previous_activity) {
+  UMA_HISTOGRAM_CUSTOM_COUNTS(
+      "Signin.XDevicePromo.BrowsingSessionDuration",
+      (base::Time::Now() - previous_activity).InMinutes(), 1,
+      base::TimeDelta::FromDays(30).InMinutes(), 50);
+}
+
 }  // namespace signin_metrics