Mixed Licenses Enrollment

chrome/browser/chromeos/login/screens/host_pairing_screen.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/screens/host_pairing_screen.h"
 
 #include "base/command_line.h"
+#include "base/logging.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chromeos/login/startup_utils.h"
 #include "chrome/browser/chromeos/login/wizard_controller.h"
 #include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h"
 #include "chrome/browser/chromeos/policy/enrollment_status_chromeos.h"
 #include "components/pairing/host_pairing_controller.h"
 #include "google_apis/gaia/google_service_auth_error.h"
 
 namespace chromeos {
 
@@ skipping 162 matching lines @@
 }
 
 void HostPairingScreen::OnAuthError(const GoogleServiceAuthError& error) {
   enrollment_error_string_ = view_->GetErrorStringFromAuthError(error);
   enrollment_error_code_ =
       GetEnrollmentErrorCode(HostPairingController::ErrorCode::AUTH_ERROR,
                              static_cast<int>(error.state()));
   OnAnyEnrollmentError();
 }
 
+void HostPairingScreen::OnMultipleLicensesAvailable(
+    const EnrollmentLicenseMap& licenses) {
+  LOG(ERROR) << "Host-paired enrollment is not yet compatible "
+             << "with Mixed Licenses Enrollment Flow";
+  enrollment_error_string_ = view_->GetErrorStringFromOtherError(
+      EnterpriseEnrollmentHelper::OTHER_ERROR_FATAL);

[achuithb, 2017/08/03 20:53:51]

This error is incorrect - it is meant for internal code errors, which I don't
think this is. We should introduce a new error type like
MULITPLE_LICENSES_NOT_SUPPORTED or something.

If this requires a new i18n string and you'd like to do it in a follow-up change
in another CL, that's fine. 

[Denis Kuznetsov (DE-MUC), 2017/08/03 23:07:51]

Ok, will do.

+  enrollment_error_code_ = GetEnrollmentErrorCode(
+      HostPairingController::ErrorCode::OTHER_ERROR,
+      static_cast<int>(EnterpriseEnrollmentHelper::OTHER_ERROR_FATAL));
+  OnAnyEnrollmentError();
+}
+
 void HostPairingScreen::OnEnrollmentError(policy::EnrollmentStatus status) {
   enrollment_error_string_ = view_->GetErrorStringFromEnrollmentError(status);
   enrollment_error_code_ =
       GetEnrollmentErrorCode(HostPairingController::ErrorCode::ENROLL_ERROR,
                              static_cast<int>(status.status()));
   OnAnyEnrollmentError();
 }
 
 void HostPairingScreen::OnOtherError(
     EnterpriseEnrollmentHelper::OtherError error) {
@@ skipping 29 matching lines @@
 void HostPairingScreen::OnAnyEnrollmentError() {
   enrollment_helper_->ClearAuth(base::Bind(&HostPairingScreen::OnAuthCleared,
                                            weak_ptr_factory_.GetWeakPtr()));
   remora_controller_->SetErrorCodeAndMessage(enrollment_error_code_,
                                              enrollment_error_string_);
   remora_controller_->OnEnrollmentStatusChanged(
       HostPairingController::ENROLLMENT_STATUS_FAILURE);
 }
 
 }  // namespace chromeos