Chrome Version: HEAD
Chrome OS Version: HEAD
Chrome OS Platform: ALL
Network info: N/A

See also: crbug.com/736416

A couple histograms are being constructed with bad parameters.  Though these are "corrected" at run-time, they may not be correct.  Here's the diff:

diff --git a/login_manager/login_metrics.cc b/login_manager/login_metrics.cc
index 8421aa810..ae4d988a2 100644
--- a/login_manager/login_metrics.cc
+++ b/login_manager/login_metrics.cc
@@ -66,13 +66,13 @@ LoginMetrics::~LoginMetrics() {}
 
 void LoginMetrics::SendConsumerAllowsNewUsers(bool allowed) {
   int uma_code = allowed ? ANY_USER_ALLOWED : ONLY_WHITELISTED_ALLOWED;
-  metrics_lib_.SendEnumToUMA(kLoginConsumerAllowsNewUsersMetric, uma_code, 1);
+  metrics_lib_.SendEnumToUMA(kLoginConsumerAllowsNewUsersMetric, uma_code, 2);
 }
 
 void LoginMetrics::SendLoginUserType(bool dev_mode, bool incognito,
                                      bool owner) {
   int uma_code = LoginUserTypeCode(dev_mode, incognito, owner);
-  metrics_lib_.SendEnumToUMA(kLoginUserTypeMetric, uma_code, NUM_TYPES - 1);
+  metrics_lib_.SendEnumToUMA(kLoginUserTypeMetric, uma_code, NUM_TYPES);
 }
 
 bool LoginMetrics::SendPolicyFilesStatus(const PolicyFilesStatus& status) {


I tried to create a patch for review but "repo upload" didn't know where to send it.