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

Unified Diff: chrome/browser/policy/cloud_policy_refresh_scheduler.cc

Issue 11620007: Switch from OnIPAddressChanged and OnConnectionTypeChange to OnNetworkChanged Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 11 months 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/policy/cloud_policy_refresh_scheduler.cc
diff --git a/chrome/browser/policy/cloud_policy_refresh_scheduler.cc b/chrome/browser/policy/cloud_policy_refresh_scheduler.cc
index ff891281088d258cc9ee8594febc0b4a62ec74fb..b1f4639f4ea4ddac2ef047e28165edc8d9abd543 100644
--- a/chrome/browser/policy/cloud_policy_refresh_scheduler.cc
+++ b/chrome/browser/policy/cloud_policy_refresh_scheduler.cc
@@ -35,7 +35,7 @@ CloudPolicyRefreshScheduler::CloudPolicyRefreshScheduler(
refresh_delay_ms_(kDefaultRefreshDelayMs) {
client_->AddObserver(this);
store_->AddObserver(this);
- net::NetworkChangeNotifier::AddIPAddressObserver(this);
+ net::NetworkChangeNotifier::AddNetworkChangeObserver(this);
UpdateLastRefreshFromPolicy();
ScheduleRefresh();
@@ -44,7 +44,7 @@ CloudPolicyRefreshScheduler::CloudPolicyRefreshScheduler(
CloudPolicyRefreshScheduler::~CloudPolicyRefreshScheduler() {
store_->RemoveObserver(this);
client_->RemoveObserver(this);
- net::NetworkChangeNotifier::RemoveIPAddressObserver(this);
+ net::NetworkChangeNotifier::RemoveNetworkChangeObserver(this);
}
void CloudPolicyRefreshScheduler::SetRefreshDelay(int64 refresh_delay) {
@@ -103,8 +103,10 @@ void CloudPolicyRefreshScheduler::OnStoreError(CloudPolicyStore* store) {
// error is required. NB: Changes to is_managed fire OnStoreLoaded().
}
-void CloudPolicyRefreshScheduler::OnIPAddressChanged() {
- if (client_->status() == DM_STATUS_REQUEST_FAILED)
+void CloudPolicyRefreshScheduler::OnNetworkChanged(
+ net::NetworkChangeNotifier::ConnectionType type) {
+ if (type != net::NetworkChangeNotifier::CONNECTION_NONE &&
+ client_->status() == DM_STATUS_REQUEST_FAILED)
RefreshAfter(0);
}

Powered by Google App Engine
This is Rietveld 408576698