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

Unified Diff: chrome/browser/chromeos/login/wizard_controller.cc

Issue 676773002: Add device disabling to OOBE flow (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update browser tests. Created 6 years, 2 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/wizard_controller.cc
diff --git a/chrome/browser/chromeos/login/wizard_controller.cc b/chrome/browser/chromeos/login/wizard_controller.cc
index 92c779015d571c79b989c5846301004f0917c5c9..b79c69718d60730a3b8cc6bd28693cbe56a4b111 100644
--- a/chrome/browser/chromeos/login/wizard_controller.cc
+++ b/chrome/browser/chromeos/login/wizard_controller.cc
@@ -534,7 +534,7 @@ void WizardController::SkipToLoginForTesting(
VLOG(1) << "SkipToLoginForTesting.";
StartupUtils::MarkEulaAccepted();
PerformPostEulaActions();
- OnAutoEnrollmentCheckCompleted();
+ OnDeviceNotDisabled();
}
void WizardController::AddObserver(Observer* observer) {
@@ -731,15 +731,6 @@ void WizardController::OnAutoEnrollmentDone() {
ResumeLoginScreen();
}
-void WizardController::OnAutoEnrollmentCheckCompleted() {
- if (ShouldAutoStartEnrollment() || enrollment_recovery_) {
- ShowEnrollmentScreen();
- } else {
- PerformOOBECompletedActions();
- ShowLoginScreen(LoginScreenContext());
- }
-}
-
void WizardController::OnTermsOfServiceDeclined() {
// If the user declines the Terms of Service, end the session and return to
// the login screen.
@@ -759,6 +750,17 @@ void WizardController::OnHostPairingFinished() {
InitiateOOBEUpdate();
}
+void WizardController::OnDeviceNotDisabled() {
+ if (skip_update_enroll_after_eula_ ||
+ ShouldAutoStartEnrollment() ||
+ enrollment_recovery_) {
+ ShowEnrollmentScreen();
+ } else {
+ PerformOOBECompletedActions();
+ ShowLoginScreen(LoginScreenContext());
+ }
+}
+
void WizardController::InitiateOOBEUpdate() {
VLOG(1) << "InitiateOOBEUpdate";
PerformPostEulaActions();
@@ -968,10 +970,7 @@ void WizardController::OnExit(ExitCodes exit_code) {
ShowNetworkScreen();
break;
case ENTERPRISE_AUTO_ENROLLMENT_CHECK_COMPLETED:
- if (skip_update_enroll_after_eula_)
- ShowEnrollmentScreen();
- else
- OnAutoEnrollmentCheckCompleted();
+ ShowDeviceDisabledScreen();
break;
case ENTERPRISE_ENROLLMENT_COMPLETED:
OnEnrollmentDone();
@@ -1009,6 +1008,9 @@ void WizardController::OnExit(ExitCodes exit_code) {
case HOST_PAIRING_FINISHED:
OnHostPairingFinished();
break;
+ case DEVICE_NOT_DISABLED:
+ OnDeviceNotDisabled();
+ break;
default:
NOTREACHED();
}
« no previous file with comments | « chrome/browser/chromeos/login/wizard_controller.h ('k') | chrome/browser/chromeos/login/wizard_controller_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698