Update AD status on chrome://policy page automatically

chrome/browser/chromeos/policy/device_active_directory_policy_manager.h

Index: chrome/browser/chromeos/policy/device_active_directory_policy_manager.h
diff --git a/chrome/browser/chromeos/policy/device_active_directory_policy_manager.h b/chrome/browser/chromeos/policy/device_active_directory_policy_manager.h
index ab3a41b9a5f67fe9ff12d58cbd7fe371b2f0ffbb..ccd373de00a05e31fb5c60d708bbb17e08bb2d19 100644
--- a/chrome/browser/chromeos/policy/device_active_directory_policy_manager.h
+++ b/chrome/browser/chromeos/policy/device_active_directory_policy_manager.h
@@ -35,6 +35,7 @@ class DeviceActiveDirectoryPolicyManager : public ConfigurationPolicyProvider,
   void OnStoreError(CloudPolicyStore* cloud_policy_store) override;
 
   const CloudPolicyStore* store() const { return store_.get(); }

[Bernhard Bauer, 2016/11/25 15:17:20]

Do you use this anywhere else? Otherwise I'd just make this return a mutable
CloudPolicyStore*. TBH, having these two methods feels a bit like overkill, but
your call, really.

[Thiemo Nagel, 2016/11/28 14:49:40]

Imho it would be better style to separate mutable and immutable accessors, but
it doesn't seem to be common in the codebase, thus I probably should prioritize
consistency...

+  CloudPolicyStore* mutable_store() const { return store_.get(); }
 
  private:
   // Publishes the policy that's currently cached in the store.