Return management state without retrieving the management domain if the

chrome/browser/policy/profile_policy_connector.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/policy/profile_policy_connector.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/logging.h"
@@ skipping 136 matching lines @@
   if (special_user_policy_provider_)
     special_user_policy_provider_->Shutdown();
 #endif
   if (wrapped_platform_policy_provider_)
     wrapped_platform_policy_provider_->Shutdown();
 }
 
 bool ProfilePolicyConnector::IsManaged() const {
   if (is_managed_override_)
     return *is_managed_override_;
-  return !GetManagementDomain().empty();
+  if (policy_store_)
+    return policy_store_->is_managed();
+#if defined(OS_CHROMEOS)

[Thiemo Nagel, 2017/02/15 16:23:03]

Nit: Instead of duplicating that code, I'd suggest to create a private method
which returns a const ptr to the store that's being used.  (Eg.
GetEffectiveStore() or GetActualStore() or maybe also just GetStore().)

[Marton Hunyady, 2017/02/15 20:13:54]

Done.

+  if (special_user_policy_provider_) {
+    // If |special_user_policy_provider_| is non-null the profile is managed iff
+    // the device itself is managed.
+    const DeviceCloudPolicyManagerChromeOS* const device_cloud_policy_manager =
+        g_browser_process->platform_part()
+            ->browser_policy_connector_chromeos()
+            ->GetDeviceCloudPolicyManager();
+    // The device_cloud_policy_manager can be a nullptr in unit tests.
+    if (device_cloud_policy_manager)
+      return device_cloud_policy_manager->core()->store()->is_managed();
+  }
+#endif
+  return false;
 }
 
 std::string ProfilePolicyConnector::GetManagementDomain() const {
   if (policy_store_)
     return GetStoreManagementDomain(policy_store_);
 #if defined(OS_CHROMEOS)
   if (special_user_policy_provider_) {
     // |special_user_policy_provider_| is non-null for device-local accounts and
     // for the login profile.
     // They receive policy iff the device itself is managed.
@@ skipping 21 matching lines @@
     const char* policy_key) const {
   const PolicyNamespace chrome_ns(POLICY_DOMAIN_CHROME, "");
   for (const ConfigurationPolicyProvider* provider : policy_providers_) {
     if (provider->policies().Get(chrome_ns).Get(policy_key))
       return provider;
   }
   return nullptr;
 }
 
 }  // namespace policy