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

Unified Diff: chrome/browser/chromeos/login/enrollment/enterprise_enrollment_screen.cc

Issue 10443125: Allow re-enrollment to the same domain in case of policy data loss. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed unit tests. Created 8 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/enrollment/enterprise_enrollment_screen.cc
diff --git a/chrome/browser/chromeos/login/enrollment/enterprise_enrollment_screen.cc b/chrome/browser/chromeos/login/enrollment/enterprise_enrollment_screen.cc
index e9f8da6b2086f73023af91247ecba1206a835999..5d2ec9c9083c8b60ade56a0fb709d5a1ed6f0a3a 100644
--- a/chrome/browser/chromeos/login/enrollment/enterprise_enrollment_screen.cc
+++ b/chrome/browser/chromeos/login/enrollment/enterprise_enrollment_screen.cc
@@ -213,12 +213,16 @@ void EnterpriseEnrollmentScreen::RegisterForDevicePolicy(
g_browser_process->browser_policy_connector();
if (!connector->device_cloud_policy_subsystem()) {
LOG(ERROR) << "Cloud policy subsystem not initialized.";
- } else if (connector->IsEnterpriseManaged()) {
- LOG(ERROR) << "Device is already enterprise managed!";
} else if (connector->device_cloud_policy_subsystem()->state() ==
policy::CloudPolicySubsystem::SUCCESS) {
LOG(ERROR) << "A previous enrollment already succeeded!";
} else {
+ if (connector->IsEnterpriseManaged() &&
+ connector->GetEnterpriseDomain() !=
+ policy::EnterpriseInstallAttributes::ExtractDomainName(user_)) {
+ LOG(ERROR) << "Trying to re-enroll to a different domain than"
+ << connector->GetEnterpriseDomain();
+ }
// Make sure the device policy subsystem is in a clean slate.
connector->ResetDevicePolicy();
connector->ScheduleServiceInitialization(0);

Powered by Google App Engine
This is Rietveld 408576698