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

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

Issue 289133002: Add a enterprise enrollment check screen to OOBE in ChromeOS. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed comments. Created 6 years, 7 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 0df78c6470529b11088a766c6254ad5f675d315d..eae4dd4ee47f51070e19b1a856ef6e1ab2946548 100644
--- a/chrome/browser/chromeos/login/wizard_controller.cc
+++ b/chrome/browser/chromeos/login/wizard_controller.cc
@@ -25,7 +25,7 @@
#include "chrome/browser/chromeos/app_mode/kiosk_app_manager.h"
#include "chrome/browser/chromeos/customization_document.h"
#include "chrome/browser/chromeos/geolocation/simple_geolocation_provider.h"
-#include "chrome/browser/chromeos/login/enrollment/auto_enrollment_check_step.h"
+#include "chrome/browser/chromeos/login/enrollment/auto_enrollment_check_screen.h"
#include "chrome/browser/chromeos/login/enrollment/enrollment_screen.h"
#include "chrome/browser/chromeos/login/existing_user_controller.h"
#include "chrome/browser/chromeos/login/helper.h"
@@ -124,6 +124,8 @@ const char WizardController::kKioskEnableScreenName[] = "kiosk-enable";
const char WizardController::kKioskAutolaunchScreenName[] = "autolaunch";
const char WizardController::kErrorScreenName[] = "error-message";
const char WizardController::kTermsOfServiceScreenName[] = "tos";
+const char WizardController::kAutoEnrollmentCheckScreenName[] =
+ "auto-enrollment-check";
const char WizardController::kWrongHWIDScreenName[] = "wrong-hwid";
const char WizardController::kLocallyManagedUserCreationScreenName[] =
"locally-managed-user-creation-flow";
@@ -325,6 +327,18 @@ chromeos::WrongHWIDScreen* WizardController::GetWrongHWIDScreen() {
return wrong_hwid_screen_.get();
}
+chromeos::AutoEnrollmentCheckScreen*
+ WizardController::GetAutoEnrollmentCheckScreen() {
+ if (!auto_enrollment_check_screen_.get()) {
+ auto_enrollment_check_screen_.reset(
+ new chromeos::AutoEnrollmentCheckScreen(
+ this,
+ oobe_display_->GetAutoEnrollmentCheckScreenActor(),
+ host_->GetAutoEnrollmentController()));
+ }
+ return auto_enrollment_check_screen_.get();
+}
+
chromeos::LocallyManagedUserCreationScreen*
WizardController::GetLocallyManagedUserCreationScreen() {
if (!locally_managed_user_creation_screen_.get()) {
@@ -480,6 +494,12 @@ void WizardController::ShowWrongHWIDScreen() {
SetCurrentScreen(GetWrongHWIDScreen());
}
+void WizardController::ShowAutoEnrollmentCheckScreen() {
+ VLOG(1) << "Showing Auto-enrollment check screen.";
+ SetStatusAreaVisible(true);
+ SetCurrentScreen(GetAutoEnrollmentCheckScreen());
+}
+
void WizardController::ShowLocallyManagedUserCreationScreen() {
VLOG(1) << "Showing Locally managed user creation screen screen.";
SetStatusAreaVisible(true);
@@ -557,7 +577,7 @@ void WizardController::OnConnectionFailed() {
}
void WizardController::OnUpdateCompleted() {
- StartAutoEnrollmentCheck();
+ ShowAutoEnrollmentCheckScreen();
}
void WizardController::OnEulaAccepted() {
@@ -677,7 +697,6 @@ void WizardController::OnAutoEnrollmentDone() {
}
void WizardController::OnOOBECompleted() {
- auto_enrollment_check_step_.reset();
if (ShouldAutoStartEnrollment()) {
ShowEnrollmentScreen();
} else {
@@ -811,6 +830,8 @@ void WizardController::AdvanceToScreen(const std::string& screen_name) {
ShowTermsOfServiceScreen();
} else if (screen_name == kWrongHWIDScreenName) {
ShowWrongHWIDScreen();
+ } else if (screen_name == kAutoEnrollmentCheckScreenName) {
+ ShowAutoEnrollmentCheckScreen();
} else if (screen_name == kLocallyManagedUserCreationScreenName) {
ShowLocallyManagedUserCreationScreen();
} else if (screen_name == kAppLaunchSplashScreenName) {
@@ -1007,12 +1028,6 @@ void WizardController::OnLocalStateInitialized(bool /* succeeded */) {
ShowErrorScreen();
}
-void WizardController::StartAutoEnrollmentCheck() {
- auto_enrollment_check_step_.reset(
- new AutoEnrollmentCheckStep(this, host_->GetAutoEnrollmentController()));
- auto_enrollment_check_step_->Start();
-}
-
PrefService* WizardController::GetLocalState() {
if (local_state_for_testing_)
return local_state_for_testing_;

Powered by Google App Engine
This is Rietveld 408576698