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

Unified Diff: chrome/browser/metrics/variations/resource_request_allowed_notifier.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/metrics/variations/resource_request_allowed_notifier.cc
diff --git a/chrome/browser/metrics/variations/resource_request_allowed_notifier.cc b/chrome/browser/metrics/variations/resource_request_allowed_notifier.cc
index 302df1072eb65bf8f6e4425e86057f7c023d88c6..c64023f92d1ca1bd94df70531f7ef77c740f23df 100644
--- a/chrome/browser/metrics/variations/resource_request_allowed_notifier.cc
+++ b/chrome/browser/metrics/variations/resource_request_allowed_notifier.cc
@@ -23,14 +23,14 @@ ResourceRequestAllowedNotifier::ResourceRequestAllowedNotifier()
ResourceRequestAllowedNotifier::~ResourceRequestAllowedNotifier() {
if (observer_)
- net::NetworkChangeNotifier::RemoveConnectionTypeObserver(this);
+ net::NetworkChangeNotifier::RemoveNetworkChangeObserver(this);
}
void ResourceRequestAllowedNotifier::Init(Observer* observer) {
DCHECK(!observer_ && observer);
observer_ = observer;
- net::NetworkChangeNotifier::AddConnectionTypeObserver(this);
+ net::NetworkChangeNotifier::AddNetworkChangeObserver(this);
// Check this state during initialization. It is not expected to change until
// the corresponding notification is received.
@@ -94,7 +94,7 @@ bool ResourceRequestAllowedNotifier::NeedsEulaAcceptance() {
}
#endif
-void ResourceRequestAllowedNotifier::OnConnectionTypeChanged(
+void ResourceRequestAllowedNotifier::OnNetworkChanged(
net::NetworkChangeNotifier::ConnectionType type) {
// Only attempt to notify observers if this was previously waiting for the
// network to reconnect, and new network state is actually available. This

Powered by Google App Engine
This is Rietveld 408576698