Minor cleanup of EnrollmentHandlerChromeOS.

chrome/browser/chromeos/login/enrollment/enrollment_screen.cc

 // Copyright (c) 2012 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/enrollment/enrollment_screen.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 92 matching lines @@
   else if (enrollment_failed_once_)
     UMA(policy::kMetricEnrollmentRetried);
   else
     UMA(policy::kMetricEnrollmentStarted);
 
   actor_->ShowEnrollmentSpinnerScreen();
   actor_->FetchOAuthToken();
 }
 
 void EnrollmentScreen::OnAuthError(const GoogleServiceAuthError& error) {
-  enrollment_failed_once_ = true;
-  actor_->ShowAuthError(error);
-
   switch (error.state()) {
     case GoogleServiceAuthError::NONE:
     case GoogleServiceAuthError::CAPTCHA_REQUIRED:
     case GoogleServiceAuthError::TWO_FACTOR:
     case GoogleServiceAuthError::HOSTED_NOT_ALLOWED:
     case GoogleServiceAuthError::INVALID_GAIA_CREDENTIALS:
     case GoogleServiceAuthError::REQUEST_CANCELED:
     case GoogleServiceAuthError::UNEXPECTED_SERVICE_RESPONSE:
     case GoogleServiceAuthError::SERVICE_ERROR:
       UMAFailure(policy::kMetricEnrollmentLoginFailed);
       LOG(ERROR) << "Auth error " << error.state();
-      return;
+      break;
     case GoogleServiceAuthError::USER_NOT_SIGNED_UP:
     case GoogleServiceAuthError::ACCOUNT_DELETED:
     case GoogleServiceAuthError::ACCOUNT_DISABLED:
       UMAFailure(policy::kMetricEnrollmentNotSupported);
       LOG(ERROR) << "Account error " << error.state();
-      return;
+      break;
     case GoogleServiceAuthError::CONNECTION_FAILED:
     case GoogleServiceAuthError::SERVICE_UNAVAILABLE:
       UMAFailure(policy::kMetricEnrollmentNetworkFailed);
       LOG(WARNING) << "Network error " << error.state();
-      return;
+      break;
     case GoogleServiceAuthError::NUM_STATES:
+      NOTREACHED();
       break;
   }
 
-  NOTREACHED();
-  UMAFailure(policy::kMetricEnrollmentOtherFailed);
+  enrollment_failed_once_ = true;
+  actor_->ShowAuthError(error);
 }
 
 void EnrollmentScreen::OnOAuthTokenAvailable(const std::string& token) {
   VLOG(1) << "OnOAuthTokenAvailable " << token;
   const bool is_shark =
       g_browser_process->platform_part()->browser_policy_connector_chromeos()->
           GetDeviceCloudPolicyManager()->IsSharkRequisition();
 
   if (is_shark && !remora_token_sent_) {
     // Fetch a second token for shark devices.
@@ skipping 89 matching lines @@
   // TODO(achuith, zork): Send token via Bluetooth to remote device.
 }
 
 void EnrollmentScreen::ShowEnrollmentStatusOnSuccess(
     const policy::EnrollmentStatus& status) {
   actor_->ShowEnrollmentStatus(status);
   StartupUtils::MarkOobeCompleted();
 }
 
 void EnrollmentScreen::ReportEnrollmentStatus(policy::EnrollmentStatus status) {
-  if (status.status() == policy::EnrollmentStatus::STATUS_SUCCESS) {
-    StartupUtils::MarkDeviceRegistered(
-        base::Bind(&EnrollmentScreen::ShowEnrollmentStatusOnSuccess,
-                   weak_ptr_factory_.GetWeakPtr(),
-                   status));
-    UMA(is_auto_enrollment() ? policy::kMetricEnrollmentAutoOK
-                             : policy::kMetricEnrollmentOK);
-    return;
-  } else {
-    enrollment_failed_once_ = true;
-  }
-  actor_->ShowEnrollmentStatus(status);
-
   switch (status.status()) {
+    case policy::EnrollmentStatus::STATUS_SUCCESS:
+      StartupUtils::MarkDeviceRegistered(
+          base::Bind(&EnrollmentScreen::ShowEnrollmentStatusOnSuccess,
+                     weak_ptr_factory_.GetWeakPtr(),
+                     status));
+      UMA(is_auto_enrollment() ? policy::kMetricEnrollmentAutoOK
+                               : policy::kMetricEnrollmentOK);
+      return;
     case policy::EnrollmentStatus::STATUS_REGISTRATION_FAILED:
     case policy::EnrollmentStatus::STATUS_POLICY_FETCH_FAILED:
       switch (status.client_status()) {
         case policy::DM_STATUS_SUCCESS:
         case policy::DM_STATUS_REQUEST_INVALID:
         case policy::DM_STATUS_SERVICE_DEVICE_NOT_FOUND:
         case policy::DM_STATUS_SERVICE_MANAGEMENT_TOKEN_INVALID:
         case policy::DM_STATUS_SERVICE_ACTIVATION_PENDING:
         case policy::DM_STATUS_SERVICE_DEVICE_ID_CONFLICT:
         case policy::DM_STATUS_SERVICE_POLICY_NOT_FOUND:
           UMAFailure(policy::kMetricEnrollmentOtherFailed);
-          return;
+          break;
         case policy::DM_STATUS_REQUEST_FAILED:
         case policy::DM_STATUS_TEMPORARY_UNAVAILABLE:
         case policy::DM_STATUS_HTTP_STATUS_ERROR:
         case policy::DM_STATUS_RESPONSE_DECODING_ERROR:
           UMAFailure(policy::kMetricEnrollmentNetworkFailed);
-          return;
+          break;
         case policy::DM_STATUS_SERVICE_MANAGEMENT_NOT_SUPPORTED:
           UMAFailure(policy::kMetricEnrollmentNotSupported);
-          return;
+          break;
         case policy::DM_STATUS_SERVICE_INVALID_SERIAL_NUMBER:
           UMAFailure(policy::kMetricEnrollmentInvalidSerialNumber);
-          return;
+          break;
         case policy::DM_STATUS_SERVICE_MISSING_LICENSES:
           UMAFailure(policy::kMetricMissingLicensesError);
-          return;
+          break;
         case policy::DM_STATUS_SERVICE_DEPROVISIONED:
           UMAFailure(policy::kMetricEnrollmentDeprovisioned);
-          return;
+          break;
         case policy::DM_STATUS_SERVICE_DOMAIN_MISMATCH:
           UMAFailure(policy::kMetricEnrollmentDomainMismatch);
-          return;
+          break;
       }
       break;
     case policy::EnrollmentStatus::STATUS_REGISTRATION_BAD_MODE:
       UMAFailure(policy::kMetricEnrollmentInvalidEnrollmentMode);
-      return;
+      break;
     case policy::EnrollmentStatus::STATUS_LOCK_TIMEOUT:
       UMAFailure(policy::kMetricLockboxTimeoutError);
-      return;
+      break;
     case policy::EnrollmentStatus::STATUS_LOCK_WRONG_USER:
       UMAFailure(policy::kMetricEnrollmentWrongUserError);
-      return;
+      break;
     case policy::EnrollmentStatus::STATUS_NO_STATE_KEYS:
       UMAFailure(policy::kMetricEnrollmentNoStateKeys);
-      return;
+      break;
     case policy::EnrollmentStatus::STATUS_VALIDATION_FAILED:
       UMAFailure(policy::kMetricEnrollmentPolicyValidationFailed);
-      return;
+      break;
     case policy::EnrollmentStatus::STATUS_STORE_ERROR:
       UMAFailure(policy::kMetricEnrollmentCloudPolicyStoreError);
-      return;
+      break;
     case policy::EnrollmentStatus::STATUS_LOCK_ERROR:
       UMAFailure(policy::kMetricEnrollmentLockBackendError);
-      return;
+      break;
     case policy::EnrollmentStatus::STATUS_ROBOT_AUTH_FETCH_FAILED:
       UMAFailure(policy::kMetricEnrollmentRobotAuthCodeFetchFailed);
-      return;
+      break;
     case policy::EnrollmentStatus::STATUS_ROBOT_REFRESH_FETCH_FAILED:
       UMAFailure(policy::kMetricEnrollmentRobotRefreshTokenFetchFailed);
-      return;
+      break;
     case policy::EnrollmentStatus::STATUS_ROBOT_REFRESH_STORE_FAILED:
       UMAFailure(policy::kMetricEnrollmentRobotRefreshTokenStoreFailed);
-      return;
+      break;
     case policy::EnrollmentStatus::STATUS_STORE_TOKEN_AND_ID_FAILED:
-      // This error should not happen for enterprise enrollment.
+      // This error should not happen for enterprise enrollment, it only affects
+      // consumer enrollment.
       UMAFailure(policy::kMetricEnrollmentStoreTokenAndIdFailed);
       NOTREACHED();
-      return;
-    case policy::EnrollmentStatus::STATUS_SUCCESS:
-      NOTREACHED();
-      return;
+      break;
   }
 
-  NOTREACHED();
-  UMAFailure(policy::kMetricEnrollmentOtherFailed);
+  enrollment_failed_once_ = true;
+  actor_->ShowEnrollmentStatus(status);
 }
 
 void EnrollmentScreen::UMA(policy::MetricEnrollment sample) {
   if (enrollment_mode_ == EnrollmentScreenActor::ENROLLMENT_MODE_RECOVERY) {
     UMA_HISTOGRAM_ENUMERATION(policy::kMetricEnrollmentRecovery, sample,
                               policy::kMetricEnrollmentSize);
   } else {
     UMA_HISTOGRAM_ENUMERATION(policy::kMetricEnrollment, sample,
                               policy::kMetricEnrollmentSize);
   }
 }
 
 void EnrollmentScreen::UMAFailure(policy::MetricEnrollment sample) {
   if (is_auto_enrollment())
     sample = policy::kMetricEnrollmentAutoFailed;
   UMA(sample);
 }
 
 void EnrollmentScreen::ShowSigninScreen() {
   actor_->Show();
   actor_->ShowSigninScreen();
 }
 
 }  // namespace chromeos