Cross Device Promo - Main Eligibility Flow

components/signin/core/browser/signin_metrics.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/signin/core/browser/signin_metrics.h"
 
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/metrics/user_metrics.h"
 #include "base/time/time.h"
@@ skipping 89 matching lines @@
 void LogAuthError(GoogleServiceAuthError::State auth_error) {
   UMA_HISTOGRAM_ENUMERATION("Signin.AuthError", auth_error,
       GoogleServiceAuthError::State::NUM_STATES);
 }
 
 void LogSigninConfirmHistogramValue(int action) {
   UMA_HISTOGRAM_ENUMERATION("Signin.OneClickConfirmation", 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