Chromad: Refresh policy every 90 minutes

chrome/browser/chromeos/policy/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_ACTIVE_DIRECTORY_POLICY_MANAGER_H_
 #define CHROME_BROWSER_CHROMEOS_POLICY_ACTIVE_DIRECTORY_POLICY_MANAGER_H_
 
 #include <memory>
 
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
+#include "base/time/time.h"
 #include "components/policy/core/common/cloud/cloud_policy_store.h"
 #include "components/policy/core/common/configuration_policy_provider.h"
 #include "components/signin/core/account_id/account_id.h"
 
 namespace policy {
 
 // ConfigurationPolicyProvider for device or user policy from Active Directory.
 // The choice of constructor determines whether device or user policy is
 // provided.  The policy is fetched from the Domain Controller by authpolicyd
 // which stores it in session manager and from where it is loaded by
@@ skipping 23 matching lines @@
   void OnStoreError(CloudPolicyStore* cloud_policy_store) override;
 
   CloudPolicyStore* store() const { return store_.get(); }
 
  private:
   // |account_id| specifies the user to manage policy for.  If |account_id| is
   // empty, device policy is managed.
   ActiveDirectoryPolicyManager(const AccountId& account_id,
                                std::unique_ptr<CloudPolicyStore> store);
 
-  // Publishes the policy that's currently cached in the store.
+  // Publish the policy that's currently cached in the store.
   void PublishPolicy();
 
   // Callback from authpolicyd.
   void OnPolicyRefreshed(bool success);
 
+  // Schedule the next policy refresh.
+  void ScheduleRefresh();
+
+  // Actually execute the scheduled policy refresh.
+  void RunScheduledRefresh(int task_number);
+
   const AccountId account_id_;
   std::unique_ptr<CloudPolicyStore> store_;
 
+  base::TimeDelta refresh_interval_;
+  base::TimeDelta min_scheduling_delay_;
+  base::TimeTicks last_refresh_;
+  bool refresh_in_progress_ = false;
+  int task_number_ = 0;

[emaxx, 2017/01/24 15:54:26]

Worth adding a comment for this member.
Also, maybe, adding "refresh_" into its title.

[Thiemo Nagel, 2017/01/26 20:14:18]

Obsolete.

+
   // Must be last member.
   base::WeakPtrFactory<ActiveDirectoryPolicyManager> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ActiveDirectoryPolicyManager);
 };
 
 }  // namespace policy
 
 #endif  // CHROME_BROWSER_CHROMEOS_POLICY_ACTIVE_DIRECTORY_POLICY_MANAGER_H_