Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3465)

Unified Diff: chrome/browser/chromeos/login/screens/host_pairing_screen.cc

Issue 2771623002: Bootstrapping: Display meaningful enrollment error message on Slave device. (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/login/screens/host_pairing_screen.cc
diff --git a/chrome/browser/chromeos/login/screens/host_pairing_screen.cc b/chrome/browser/chromeos/login/screens/host_pairing_screen.cc
index 6244428208efff5d15c23e35096cb6804829f491..454a12352227911ef69a31e6010895188b732bb4 100644
--- a/chrome/browser/chromeos/login/screens/host_pairing_screen.cc
+++ b/chrome/browser/chromeos/login/screens/host_pairing_screen.cc
@@ -106,6 +106,7 @@ void HostPairingScreen::PairingStageChanged(Stage new_stage) {
// TODO(xdai): Maybe return to the Network Setup page?
remora_controller_->RemoveObserver(this);
desired_page = kPageEnrollmentError;
+ context_.SetString(kContextKeyEnrollmentError, enrollment_error_string_);
break;
}
default:
@@ -157,15 +158,18 @@ void HostPairingScreen::OnViewDestroyed(HostPairingScreenView* view) {
}
void HostPairingScreen::OnAuthError(const GoogleServiceAuthError& error) {
+ enrollment_error_string_ = view_->GetErrorStringFromAuthError(error);
OnAnyEnrollmentError();
}
void HostPairingScreen::OnEnrollmentError(policy::EnrollmentStatus status) {
+ enrollment_error_string_ = view_->GetErrorStringFromEnrollmentError(status);
OnAnyEnrollmentError();
}
void HostPairingScreen::OnOtherError(
EnterpriseEnrollmentHelper::OtherError error) {
+ enrollment_error_string_ = view_->GetErrorStringFromOtherError(error);
OnAnyEnrollmentError();
}

Powered by Google App Engine
This is Rietveld 408576698