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

Unified Diff: chrome/browser/chromeos/login/ui/login_display_host_impl.cc

Issue 374573006: Force enterprise enrollment flow upon detection of inconsistent state. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 5 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/ui/login_display_host_impl.cc
diff --git a/chrome/browser/chromeos/login/ui/login_display_host_impl.cc b/chrome/browser/chromeos/login/ui/login_display_host_impl.cc
index 4df0338cb5fc0f7092d489db7db1dc79bd333354..fee0a6ccf039cc02388f5eea9b65a72ebf4d1282 100644
--- a/chrome/browser/chromeos/login/ui/login_display_host_impl.cc
+++ b/chrome/browser/chromeos/login/ui/login_display_host_impl.cc
@@ -1193,9 +1193,10 @@ void ShowLoginWizard(const std::string& first_screen_name) {
policy::BrowserPolicyConnectorChromeOS* connector =
g_browser_process->platform_part()->browser_policy_connector_chromeos();
bool should_show_enrollment_screen =
- first_screen_name.empty() && oobe_complete &&
- chromeos::WizardController::ShouldAutoStartEnrollment() &&
- !connector->IsEnterpriseManaged();
+ chromeos::WizardController::ShouldRecoverEnrollment() ||
+ ( first_screen_name.empty() && oobe_complete &&
pastarmovj 2014/07/08 12:36:31 nit: no spaces in the braces.
Thiemo Nagel 2014/07/08 14:02:04 Done.
+ chromeos::WizardController::ShouldAutoStartEnrollment() &&
+ !connector->IsEnterpriseManaged() );
pastarmovj 2014/07/08 12:36:31 I think you should still respect the first_Screen_
Thiemo Nagel 2014/07/08 14:02:04 Ok, thanks.
if (should_show_enrollment_screen) {
// Shows networks screen instead of enrollment screen to resume the
// interrupted auto start enrollment flow because enrollment screen does

Powered by Google App Engine
This is Rietveld 408576698