Mop up incorrect uses of IsEnterpriseManaged()

chrome/browser/chromeos/login/enrollment/enterprise_enrollment_helper_impl.cc

Index: chrome/browser/chromeos/login/enrollment/enterprise_enrollment_helper_impl.cc
diff --git a/chrome/browser/chromeos/login/enrollment/enterprise_enrollment_helper_impl.cc b/chrome/browser/chromeos/login/enrollment/enterprise_enrollment_helper_impl.cc
index b4b67ae7b8d17291cf65d3a56ead070ff0053a7d..9e9d683d6742cec3907bec2fe63481ecda71fb01 100644
--- a/chrome/browser/chromeos/login/enrollment/enterprise_enrollment_helper_impl.cc
+++ b/chrome/browser/chromeos/login/enrollment/enterprise_enrollment_helper_impl.cc
@@ -146,7 +146,8 @@ void EnterpriseEnrollmentHelperImpl::DoEnroll(const std::string& token) {
   oauth_token_ = token;
   policy::BrowserPolicyConnectorChromeOS* connector =
       g_browser_process->platform_part()->browser_policy_connector_chromeos();
-  if (connector->IsEnterpriseManaged() &&
+  // Re-enrollment not (yet?) supported for Active Directory.

[achuithb, 2016/11/25 10:10:48]

Is this intended to be a TODO? If so, we should add a bug. Otherwise, is it
useful?

[Thiemo Nagel, 2016/11/25 14:26:10]

Good point.  It's not intended as a TODO.  I've changed the text to "not
implemented" to clarify that re-enrollment for AD is not disabled for
fundamental reasons but only due to low priority. ;)

+  if (connector->IsCloudManaged() &&
       connector->GetEnterpriseDomain() != enrolling_user_domain_) {
     LOG(ERROR) << "Trying to re-enroll to a different domain than "
                << connector->GetEnterpriseDomain();