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 "chrome/browser/browser_process.h"
 #include "chrome/browser/chromeos/login/startup_utils.h"
 #include "chrome/browser/chromeos/login/wizard_controller.h"
@@ skipping 169 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 "

[emaxx, 2017/07/21 12:57:29]

nit: #include "base/logging.h"

[emaxx, 2017/07/21 12:57:29]

nit: Is this a TODO, or should a separate bug be filed for that?
Or will this be something that won't be realistically implemented?
(To clarify: I have no idea what "host-paired enrollment" is.)

[Denis Kuznetsov (DE-MUC), 2017/07/25 21:51:05]

This is an outdated branch. It was a plan that user could have keyboardless
device to enroll (e.g. chromebox) that can be controlled from other ChromeOS
device (like tablet) during enrollment. There are no  devices in production now
that have this feature.
There is a special protocol for interaction between two devices, and supporting
Mixed Licenses would require update of this protocol also. This is out of the
scope of this issue.

+             << "with Mixed Licenses Enrollment Flow";
+  enrollment_error_string_ = view_->GetErrorStringFromOtherError(
+      EnterpriseEnrollmentHelper::OTHER_ERROR_FATAL);
+  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