Mop up incorrect uses of IsEnterpriseManaged()

chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h

Index: chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h
diff --git a/chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h b/chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h
index c708998051606478e508da9515372eb6b5c7b08f..5ce7feb33d039c52d016ba36ad0b68947e6cf6ff 100644
--- a/chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h
+++ b/chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h
@@ -73,6 +73,13 @@ class BrowserPolicyConnectorChromeOS
   // a local owner).
   bool IsEnterpriseManaged() const;

[Roman Sorokin (ftl), 2016/11/25 09:40:16]

What's up with that method? Should we delete it? Or rewrite comment?

[Thiemo Nagel, 2016/11/25 09:58:10]

We need to keep it, there's many legit uses of it.  Still I've synchronized the
comments with those in InstallAttributes for maximal clarity.

 
+  // Returns true if this device is enterprise-managed via Google's cloud (DM
+  // server).
+  bool IsCloudManaged() const;
+
+  // Returns true if this device is enterprise-managed via Active Directory.
+  bool IsActiveDirectoryManaged() const;
+
   // Returns the enterprise domain if device is managed.
   std::string GetEnterpriseDomain() const;