Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(600)

Side by Side Diff: chrome/browser/policy/cloud_policy_refresh_scheduler.cc

Issue 11299305: Break CloudPolicyRefreshScheduler's dependency on PrefService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address review feedback Created 8 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/policy/cloud_policy_refresh_scheduler.h" 5 #include "chrome/browser/policy/cloud_policy_refresh_scheduler.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 8
9 #include "base/task_runner.h" 9 #include "base/task_runner.h"
10 #include "chrome/browser/policy/cloud_policy_constants.h" 10 #include "chrome/browser/policy/cloud_policy_constants.h"
11 #include "chrome/browser/prefs/pref_service.h"
12 #include "chrome/common/chrome_notification_types.h" 11 #include "chrome/common/chrome_notification_types.h"
13 #include "content/public/browser/notification_details.h" 12 #include "content/public/browser/notification_details.h"
14 13
15 namespace policy { 14 namespace policy {
16 15
16 const int64 CloudPolicyRefreshScheduler::kDefaultRefreshDelayMs =
17 3 * 60 * 60 * 1000; // 3 hours.
17 const int64 CloudPolicyRefreshScheduler::kUnmanagedRefreshDelayMs = 18 const int64 CloudPolicyRefreshScheduler::kUnmanagedRefreshDelayMs =
18 24 * 60 * 60 * 1000; // 1 day. 19 24 * 60 * 60 * 1000; // 1 day.
19 const int64 CloudPolicyRefreshScheduler::kInitialErrorRetryDelayMs = 20 const int64 CloudPolicyRefreshScheduler::kInitialErrorRetryDelayMs =
20 5 * 60 * 1000; // 5 minutes. 21 5 * 60 * 1000; // 5 minutes.
21 const int64 CloudPolicyRefreshScheduler::kRefreshDelayMinMs = 22 const int64 CloudPolicyRefreshScheduler::kRefreshDelayMinMs =
22 30 * 60 * 1000; // 30 minutes. 23 30 * 60 * 1000; // 30 minutes.
23 const int64 CloudPolicyRefreshScheduler::kRefreshDelayMaxMs = 24 const int64 CloudPolicyRefreshScheduler::kRefreshDelayMaxMs =
24 24 * 60 * 60 * 1000; // 1 day. 25 24 * 60 * 60 * 1000; // 1 day.
25 26
26 CloudPolicyRefreshScheduler::CloudPolicyRefreshScheduler( 27 CloudPolicyRefreshScheduler::CloudPolicyRefreshScheduler(
27 CloudPolicyClient* client, 28 CloudPolicyClient* client,
28 CloudPolicyStore* store, 29 CloudPolicyStore* store,
29 PrefService* prefs,
30 const std::string& refresh_pref,
31 const scoped_refptr<base::TaskRunner>& task_runner) 30 const scoped_refptr<base::TaskRunner>& task_runner)
32 : client_(client), 31 : client_(client),
33 store_(store), 32 store_(store),
34 task_runner_(task_runner), 33 task_runner_(task_runner),
35 error_retry_delay_ms_(kInitialErrorRetryDelayMs) { 34 error_retry_delay_ms_(kInitialErrorRetryDelayMs),
35 refresh_delay_ms_(kDefaultRefreshDelayMs) {
36 client_->AddObserver(this); 36 client_->AddObserver(this);
37 store_->AddObserver(this); 37 store_->AddObserver(this);
38 net::NetworkChangeNotifier::AddIPAddressObserver(this); 38 net::NetworkChangeNotifier::AddIPAddressObserver(this);
39 39
40 refresh_delay_.Init(
41 refresh_pref.c_str(), prefs,
42 base::Bind(&CloudPolicyRefreshScheduler::ScheduleRefresh,
43 base::Unretained(this)));
44
45 UpdateLastRefreshFromPolicy(); 40 UpdateLastRefreshFromPolicy();
46 ScheduleRefresh(); 41 ScheduleRefresh();
47 } 42 }
48 43
49 CloudPolicyRefreshScheduler::~CloudPolicyRefreshScheduler() { 44 CloudPolicyRefreshScheduler::~CloudPolicyRefreshScheduler() {
50 store_->RemoveObserver(this); 45 store_->RemoveObserver(this);
51 client_->RemoveObserver(this); 46 client_->RemoveObserver(this);
52 net::NetworkChangeNotifier::RemoveIPAddressObserver(this); 47 net::NetworkChangeNotifier::RemoveIPAddressObserver(this);
53 } 48 }
54 49
50 void CloudPolicyRefreshScheduler::SetRefreshDelay(int64 refresh_delay) {
51 refresh_delay_ms_ = std::min(std::max(refresh_delay, kRefreshDelayMinMs),
52 kRefreshDelayMaxMs);
53 ScheduleRefresh();
54 }
55
55 void CloudPolicyRefreshScheduler::OnPolicyFetched(CloudPolicyClient* client) { 56 void CloudPolicyRefreshScheduler::OnPolicyFetched(CloudPolicyClient* client) {
56 error_retry_delay_ms_ = kInitialErrorRetryDelayMs; 57 error_retry_delay_ms_ = kInitialErrorRetryDelayMs;
57 58
58 // Schedule the next refresh. 59 // Schedule the next refresh.
59 last_refresh_ = base::Time::NowFromSystemTime(); 60 last_refresh_ = base::Time::NowFromSystemTime();
60 ScheduleRefresh(); 61 ScheduleRefresh();
61 } 62 }
62 63
63 void CloudPolicyRefreshScheduler::OnRegistrationStateChanged( 64 void CloudPolicyRefreshScheduler::OnRegistrationStateChanged(
64 CloudPolicyClient* client) { 65 CloudPolicyClient* client) {
(...skipping 10 matching lines...) Expand all
75 76
76 // Schedule an error retry if applicable. 77 // Schedule an error retry if applicable.
77 last_refresh_ = base::Time::NowFromSystemTime(); 78 last_refresh_ = base::Time::NowFromSystemTime();
78 ScheduleRefresh(); 79 ScheduleRefresh();
79 80
80 // Update the retry delay. 81 // Update the retry delay.
81 if (client->is_registered() && 82 if (client->is_registered() &&
82 (status == DM_STATUS_REQUEST_FAILED || 83 (status == DM_STATUS_REQUEST_FAILED ||
83 status == DM_STATUS_TEMPORARY_UNAVAILABLE)) { 84 status == DM_STATUS_TEMPORARY_UNAVAILABLE)) {
84 error_retry_delay_ms_ = std::min(error_retry_delay_ms_ * 2, 85 error_retry_delay_ms_ = std::min(error_retry_delay_ms_ * 2,
85 GetRefreshDelay()); 86 refresh_delay_ms_);
86 } else { 87 } else {
87 error_retry_delay_ms_ = kInitialErrorRetryDelayMs; 88 error_retry_delay_ms_ = kInitialErrorRetryDelayMs;
88 } 89 }
89 } 90 }
90 91
91 void CloudPolicyRefreshScheduler::OnStoreLoaded(CloudPolicyStore* store) { 92 void CloudPolicyRefreshScheduler::OnStoreLoaded(CloudPolicyStore* store) {
92 UpdateLastRefreshFromPolicy(); 93 UpdateLastRefreshFromPolicy();
93 94
94 // Re-schedule the next refresh in case the is_managed bit changed. 95 // Re-schedule the next refresh in case the is_managed bit changed.
95 ScheduleRefresh(); 96 ScheduleRefresh();
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
134 if (!client_->is_registered()) { 135 if (!client_->is_registered()) {
135 refresh_callback_.Cancel(); 136 refresh_callback_.Cancel();
136 return; 137 return;
137 } 138 }
138 139
139 // If there is a registration, go by the client's status. That will tell us 140 // If there is a registration, go by the client's status. That will tell us
140 // what the appropriate refresh delay should be. 141 // what the appropriate refresh delay should be.
141 switch (client_->status()) { 142 switch (client_->status()) {
142 case DM_STATUS_SUCCESS: 143 case DM_STATUS_SUCCESS:
143 if (store_->is_managed()) 144 if (store_->is_managed())
144 RefreshAfter(GetRefreshDelay()); 145 RefreshAfter(refresh_delay_ms_);
145 else 146 else
146 RefreshAfter(kUnmanagedRefreshDelayMs); 147 RefreshAfter(kUnmanagedRefreshDelayMs);
147 return; 148 return;
148 case DM_STATUS_SERVICE_ACTIVATION_PENDING: 149 case DM_STATUS_SERVICE_ACTIVATION_PENDING:
149 case DM_STATUS_SERVICE_POLICY_NOT_FOUND: 150 case DM_STATUS_SERVICE_POLICY_NOT_FOUND:
150 RefreshAfter(GetRefreshDelay()); 151 RefreshAfter(refresh_delay_ms_);
151 return; 152 return;
152 case DM_STATUS_REQUEST_FAILED: 153 case DM_STATUS_REQUEST_FAILED:
153 case DM_STATUS_TEMPORARY_UNAVAILABLE: 154 case DM_STATUS_TEMPORARY_UNAVAILABLE:
154 RefreshAfter(error_retry_delay_ms_); 155 RefreshAfter(error_retry_delay_ms_);
155 return; 156 return;
156 case DM_STATUS_REQUEST_INVALID: 157 case DM_STATUS_REQUEST_INVALID:
157 case DM_STATUS_HTTP_STATUS_ERROR: 158 case DM_STATUS_HTTP_STATUS_ERROR:
158 case DM_STATUS_RESPONSE_DECODING_ERROR: 159 case DM_STATUS_RESPONSE_DECODING_ERROR:
159 case DM_STATUS_SERVICE_MANAGEMENT_NOT_SUPPORTED: 160 case DM_STATUS_SERVICE_MANAGEMENT_NOT_SUPPORTED:
160 RefreshAfter(kUnmanagedRefreshDelayMs); 161 RefreshAfter(kUnmanagedRefreshDelayMs);
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
195 // Schedule the callback. 196 // Schedule the callback.
196 base::TimeDelta delay = 197 base::TimeDelta delay =
197 std::max((last_refresh_ + delta) - base::Time::NowFromSystemTime(), 198 std::max((last_refresh_ + delta) - base::Time::NowFromSystemTime(),
198 base::TimeDelta()); 199 base::TimeDelta());
199 refresh_callback_.Reset( 200 refresh_callback_.Reset(
200 base::Bind(&CloudPolicyRefreshScheduler::PerformRefresh, 201 base::Bind(&CloudPolicyRefreshScheduler::PerformRefresh,
201 base::Unretained(this))); 202 base::Unretained(this)));
202 task_runner_->PostDelayedTask(FROM_HERE, refresh_callback_.callback(), delay); 203 task_runner_->PostDelayedTask(FROM_HERE, refresh_callback_.callback(), delay);
203 } 204 }
204 205
205 int64 CloudPolicyRefreshScheduler::GetRefreshDelay() {
206 return std::min(std::max<int64>(refresh_delay_.GetValue(),
207 kRefreshDelayMinMs),
208 kRefreshDelayMaxMs);
209 }
210
211 } // namespace policy 206 } // namespace policy
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698