Break CloudPolicyRefreshScheduler's dependency on PrefService.

chrome/browser/policy/cloud_policy_refresh_scheduler.h

 // Copyright (c) 2012 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_POLICY_CLOUD_POLICY_REFRESH_SCHEDULER_H_
 #define CHROME_BROWSER_POLICY_CLOUD_POLICY_REFRESH_SCHEDULER_H_
 
 #include <string>

[bartfab (slow), 2012/12/04 12:25:18]

Nit: No longer needed.

[Mattias Nissler (ping if slow), 2012/12/04 13:20:40]

Done.

 
 #include "base/basictypes.h"
 #include "base/cancelable_callback.h"
 #include "base/memory/ref_counted.h"
 #include "base/time.h"
-#include "chrome/browser/api/prefs/pref_member.h"
 #include "chrome/browser/policy/cloud_policy_client.h"
 #include "chrome/browser/policy/cloud_policy_store.h"
 #include "net/base/network_change_notifier.h"
 
 class PrefService;

[bartfab (slow), 2012/12/04 12:25:18]

Nit: No longer needed.

[Mattias Nissler (ping if slow), 2012/12/04 13:20:40]

Done.

 
 namespace base {
 class TaskRunner;
 }
 
 namespace policy {
 
 // Observes CloudPolicyClient and CloudPolicyStore to trigger periodic policy
 // fetches and issue retries on error conditions.
 class CloudPolicyRefreshScheduler
     : public CloudPolicyClient::Observer,
       public CloudPolicyStore::Observer,
       public net::NetworkChangeNotifier::IPAddressObserver {
  public:
   // Refresh constants.
+  static const int64 kDefaultRefreshDelayMs;
   static const int64 kUnmanagedRefreshDelayMs;
   static const int64 kInitialErrorRetryDelayMs;
 
   // Refresh delay bounds.
   static const int64 kRefreshDelayMinMs;
   static const int64 kRefreshDelayMaxMs;
 
   // |client|, |store| and |prefs| pointers must stay valid throughout the
   // lifetime of CloudPolicyRefreshScheduler.
   CloudPolicyRefreshScheduler(
       CloudPolicyClient* client,
       CloudPolicyStore* store,
-      PrefService* prefs,
-      const std::string& refresh_pref,
       const scoped_refptr<base::TaskRunner>& task_runner);
   virtual ~CloudPolicyRefreshScheduler();
 
+  // Sets the refresh delay to |refresh_delay| (subject to min/max clamping).
+  void SetRefreshDelay(int64 refresh_delay);
+
   // CloudPolicyClient::Observer:
   virtual void OnPolicyFetched(CloudPolicyClient* client) OVERRIDE;
   virtual void OnRegistrationStateChanged(CloudPolicyClient* client) OVERRIDE;
   virtual void OnClientError(CloudPolicyClient* client) OVERRIDE;
 
   // CloudPolicyStore::Observer:
   virtual void OnStoreLoaded(CloudPolicyStore* store) OVERRIDE;
   virtual void OnStoreError(CloudPolicyStore* store) OVERRIDE;
 
   // net::NetworkChangeNotifier::IPAddressObserver:
@@ skipping 29 matching lines @@
 
   // The delayed refresh callback.
   base::CancelableClosure refresh_callback_;
 
   // The last time a refresh callback completed.
   base::Time last_refresh_;
 
   // Error retry delay in milliseconds.
   int64 error_retry_delay_ms_;
 
-  IntegerPrefMember refresh_delay_;
+  // The refresh delay.
+  int64 refresh_delay_ms_;
 
   DISALLOW_COPY_AND_ASSIGN(CloudPolicyRefreshScheduler);
 };
 
 }  // namespace policy
 
 #endif  // CHROME_BROWSER_POLICY_CLOUD_POLICY_REFRESH_SCHEDULER_H_