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/cancelable_callback.h"
 #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 next policy refresh to run after |delay|.  (Deletes any previously
+  // scheduled refresh tasks.)
+  void ScheduleRefresh(base::TimeDelta delay);
+
+  // Schedule next automatic policy refresh based on initial fetch delay or
+  // refresh interval.  (Deletes any previously scheduled refresh tasks.)
+  void ScheduleAutomaticRefresh();
+
+  // Actually execute the scheduled policy refresh.
+  void RunScheduledRefresh();
+
   const AccountId account_id_;
   std::unique_ptr<CloudPolicyStore> store_;
 
+  base::TimeTicks startup_ = base::TimeTicks::Now();

[emaxx, 2017/01/26 20:54:02]

nit: const?

[Thiemo Nagel, 2017/01/27 10:55:35]

Done.

+  base::TimeTicks last_refresh_{};

[emaxx, 2017/01/26 20:54:02]

nit: Can the bracket initialization be omitted here?

[Thiemo Nagel, 2017/01/27 10:55:35]

Sure.

+  bool refresh_in_progress_ = false;
+  std::unique_ptr<base::CancelableClosure> refresh_task_;
+
   // 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_