UMA for pin unlock success/failure.

chrome/browser/chromeos/login/lock/screen_locker.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 "chrome/browser/chromeos/login/lock/screen_locker.h"
 
 #include <string>
 #include <vector>
 
 #include "ash/common/ash_switches.h"
@@ skipping 171 matching lines @@
 void ScreenLocker::OnAuthFailure(const AuthFailure& error) {
   content::RecordAction(UserMetricsAction("ScreenLocker_OnLoginFailure"));
   if (authentication_start_time_.is_null()) {
     LOG(ERROR) << "Start time is not set at authentication failure";
   } else {
     base::TimeDelta delta = base::Time::Now() - authentication_start_time_;
     VLOG(1) << "Authentication failure: " << delta.InSecondsF() << " second(s)";
     UMA_HISTOGRAM_TIMES("ScreenLocker.AuthenticationFailureTime", delta);
   }
 
+  if (error.IsPinAttempt()) {
+    UMA_HISTOGRAM_ENUMERATION("ScreenLocker.AuthenticationFailure",
+                              UnlockFailure::AUTH_FAILURE_PIN,
+                              UnlockFailure::AUTH_FAILURE_COUNT);
+  } else {
+    UMA_HISTOGRAM_ENUMERATION("ScreenLocker.AuthenticationFailure",
+                              UnlockFailure::AUTH_FAILURE_PASSWORD,
+                              UnlockFailure::AUTH_FAILURE_COUNT);
+  }
+
   EnableInput();
   // Don't enable signout button here as we're showing
   // MessageBubble.
 
   delegate_->ShowErrorMessage(incorrect_passwords_count_++ ?
                                   IDS_LOGIN_ERROR_AUTHENTICATING_2ND_TIME :
                                   IDS_LOGIN_ERROR_AUTHENTICATING,
                               HelpAppLauncher::HELP_CANT_ACCESS_ACCOUNT);
 
   if (auth_status_consumer_)
     auth_status_consumer_->OnAuthFailure(error);
 }
 
 void ScreenLocker::OnAuthSuccess(const UserContext& user_context) {
   incorrect_passwords_count_ = 0;
   if (authentication_start_time_.is_null()) {
     if (user_context.GetAccountId().is_valid())
       LOG(ERROR) << "Start time is not set at authentication success";
   } else {
     base::TimeDelta delta = base::Time::Now() - authentication_start_time_;
     VLOG(1) << "Authentication success: " << delta.InSecondsF() << " second(s)";
     UMA_HISTOGRAM_TIMES("ScreenLocker.AuthenticationSuccessTime", delta);
   }
 
+  if (user_context.IsUsingPin()) {
+    UMA_HISTOGRAM_ENUMERATION("ScreenLocker.AuthenticationSuccess",
+                              int{UnlockSuccess::AUTH_SUCCESS_PIN},

[jdufault, 2016/08/08 18:54:26]

Do you need these casts? You don't have them above.

[sammiequon, 2016/08/09 00:31:25]

Done.

+                              int{UnlockSuccess::AUTH_SUCCESS_COUNT});
+  } else {
+    UMA_HISTOGRAM_ENUMERATION("ScreenLocker.AuthenticationSuccess",
+                              int{UnlockSuccess::AUTH_SUCCESS_PASSWORD},
+                              int{UnlockSuccess::AUTH_SUCCESS_COUNT});
+  }
+
   const user_manager::User* user =
       user_manager::UserManager::Get()->FindUser(user_context.GetAccountId());
   if (user) {
     if (!user->is_active()) {
       saved_ime_state_ = NULL;
       user_manager::UserManager::Get()->SwitchActiveUser(
           user_context.GetAccountId());
     }
 
     // Reset the number of PIN attempts available to the user. We always do this
@@ skipping 60 matching lines @@
 
   const user_manager::User* user = FindUnlockUser(user_context.GetAccountId());
   if (user) {
     // Check to see if the user submitted a PIN and it is valid.
     const std::string pin = user_context.GetKey()->GetSecret();
 
     // We only want to try authenticating the pin if it is a number,
     // otherwise we will timeout PIN if the user enters their account password
     // incorrectly more than a few times.
     int dummy_value;
     if (base::StringToInt(pin, &dummy_value)) {

[jdufault, 2016/08/08 18:54:26]

Also check if user_context.IsUsingPin() is set to true before attempting PIN.

[sammiequon, 2016/08/09 00:31:25]

Done.

       chromeos::PinStorage* pin_storage =
           chromeos::PinStorageFactory::GetForUser(user);
       if (pin_storage && pin_storage->TryAuthenticatePin(pin)) {
         OnAuthSuccess(user_context);
         return;
       }
     }
 
     // Special case: supervised users. Use special authenticator.
     if (user->GetType() == user_manager::USER_TYPE_SUPERVISED) {
@@ skipping 232 matching lines @@
 
 bool ScreenLocker::IsUserLoggedIn(const AccountId& account_id) const {
   for (user_manager::User* user : users_) {
     if (user->GetAccountId() == account_id)
       return true;
   }
   return false;
 }
 
 }  // namespace chromeos