Update AD status on chrome://policy page automatically

chrome/browser/chromeos/policy/device_active_directory_policy_manager.h

 // Copyright 2016 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_POLICY_DEVICE_ACTIVE_DIRECTORY_POLICY_MANAGER_H_
 #define CHROME_BROWSER_CHROMEOS_POLICY_DEVICE_ACTIVE_DIRECTORY_POLICY_MANAGER_H_
 
 #include <memory>
 
 #include "base/macros.h"
@@ skipping 16 matching lines @@
   // ConfigurationPolicyProvider:
   void Init(SchemaRegistry* registry) override;
   void Shutdown() override;
   bool IsInitializationComplete(PolicyDomain domain) const override;
   void RefreshPolicies() override;
 
   // CloudPolicyStore::Observer:
   void OnStoreLoaded(CloudPolicyStore* cloud_policy_store) override;
   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.
   void PublishPolicy();
 
   // Callback from authpolicyd.
   void OnPolicyRefreshed(bool success);
 
   std::unique_ptr<CloudPolicyStore> store_;
 
   // Must be last member.
   base::WeakPtrFactory<DeviceActiveDirectoryPolicyManager> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(DeviceActiveDirectoryPolicyManager);
 };
 
 }  // namespace policy
 
 #endif  // CHROME_BROWSER_CHROMEOS_POLICY_DEVICE_ACTIVE_DIRECTORY_POLICY_MANAGER_H_