Extract AD policy scheduler into separate class

chrome/browser/chromeos/policy/active_directory_policy_manager.h

Index: chrome/browser/chromeos/policy/active_directory_policy_manager.h
diff --git a/chrome/browser/chromeos/policy/active_directory_policy_manager.h b/chrome/browser/chromeos/policy/active_directory_policy_manager.h
index 9bf7935e70547556da3387e177da68710a16d6dd..598c545cf6225fad0ce45a7f84d26dab5e098ce9 100644
--- a/chrome/browser/chromeos/policy/active_directory_policy_manager.h
+++ b/chrome/browser/chromeos/policy/active_directory_policy_manager.h
@@ -7,12 +7,11 @@
 
 #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/policy/core/common/policy_scheduler.h"
 #include "components/signin/core/account_id/account_id.h"
 
 namespace policy {
@@ -57,35 +56,20 @@ class ActiveDirectoryPolicyManager : public ConfigurationPolicyProvider,
   // 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);
+  // Calls into authpolicyd to fetch policy. Reports success or failure via
+  // |callback|.
+  void DoRefresh(PolicyScheduler::TaskCallback callback);

[pmarko, 2017/06/19 10:14:37]

Do you think it'd be worth mentioning in a comment that this is going to be
executed by the scheduler?

[Thiemo Nagel, 2017/06/19 22:29:51]

I'm leaning towards not mentioning this because comments like this tend to get
out of sync quickly and also with codesearch we have a quick and easy way to
find the caller.

[pmarko, 2017/06/20 20:01:05]

Ack, I was just asking about this for consistency with the OnPolicyRefreshed
comment.

 
-  // 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();
+  // Called by scheduler with result of policy fetch.
+  void OnPolicyRefreshed(bool success);
 
   const AccountId account_id_;
   std::unique_ptr<CloudPolicyStore> store_;
 
-  // Whether a policy refresh is in progress (and thus any further requests need
-  // to be blocked).
-  bool refresh_in_progress_ = false;
-  // Whether a refresh had been blocked and thus the next refresh needs to be
-  // scheduled at the shorter "retry" interval.
-  bool retry_refresh_ = false;
-  base::TimeTicks last_refresh_;
-  const base::TimeTicks startup_ = base::TimeTicks::Now();
-  std::unique_ptr<base::CancelableClosure> refresh_task_;
+  std::unique_ptr<PolicyScheduler> scheduler_;
 
   // Must be last member.
-  base::WeakPtrFactory<ActiveDirectoryPolicyManager> weak_ptr_factory_;
+  base::WeakPtrFactory<ActiveDirectoryPolicyManager> weak_ptr_factory_{this};
 
   DISALLOW_COPY_AND_ASSIGN(ActiveDirectoryPolicyManager);
 };