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

Unified Diff: chrome/browser/chromeos/login/enrollment/enrollment_screen.h

Issue 385983007: Enrollment recovery: Add UMA stats. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix UMA_HISTOGRAM_ENUMERATION() call. Created 6 years, 5 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 | « no previous file | chrome/browser/chromeos/login/enrollment/enrollment_screen.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/login/enrollment/enrollment_screen.h
diff --git a/chrome/browser/chromeos/login/enrollment/enrollment_screen.h b/chrome/browser/chromeos/login/enrollment/enrollment_screen.h
index f88384f9249ef057d20764ae574614cc77752b58..0002d88d161bd205a26bd3f671d4d027467bd6f0 100644
--- a/chrome/browser/chromeos/login/enrollment/enrollment_screen.h
+++ b/chrome/browser/chromeos/login/enrollment/enrollment_screen.h
@@ -14,6 +14,7 @@
#include "chrome/browser/chromeos/login/enrollment/enrollment_screen_actor.h"
#include "chrome/browser/chromeos/login/screens/wizard_screen.h"
#include "components/policy/core/common/cloud/cloud_policy_constants.h"
+#include "components/policy/core/common/cloud/enterprise_metrics.h"
namespace chromeos {
@@ -69,10 +70,15 @@ class EnrollmentScreen
// operations are completed.
void ShowEnrollmentStatusOnSuccess(const policy::EnrollmentStatus& status);
- // Logs a UMA event in the kMetricEnrollment histogram. If auto-enrollment is
- // on |sample| is ignored and a kMetricEnrollmentAutoFailed sample is logged
+ // Logs an UMA event in the kMetricEnrollment or the kMetricEnrollmentRecovery
+ // histogram, depending on |enrollment_mode_|.
+ void UMA(policy::MetricEnrollment sample);
+
+ // Logs an UMA event in the kMetricEnrollment or the kMetricEnrollmentRecovery
+ // histogram, depending on |enrollment_mode_|. If auto-enrollment is on,
+ // |sample| is ignored and a kMetricEnrollmentAutoFailed sample is logged
// instead.
- void UMAFailure(int sample);
+ void UMAFailure(policy::MetricEnrollment sample);
// Shows the signin screen. Used as a callback to run after auth reset.
void ShowSigninScreen();
« no previous file with comments | « no previous file | chrome/browser/chromeos/login/enrollment/enrollment_screen.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698