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

Unified Diff: components/google/core/browser/google_url_tracker.h

Issue 512843002: Listen to OnNetworkChanged instead of OnIPAddressChanged (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update tests Created 6 years, 3 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: components/google/core/browser/google_url_tracker.h
diff --git a/components/google/core/browser/google_url_tracker.h b/components/google/core/browser/google_url_tracker.h
index 644555d9069900f4a66b921e66f7266b1a805512..d54dc9b657ebe43bc1a1be6f87bf5594fc761fc3 100644
--- a/components/google/core/browser/google_url_tracker.h
+++ b/components/google/core/browser/google_url_tracker.h
@@ -40,9 +40,10 @@ class InfoBar;
// To protect users' privacy and reduce server load, no updates will be
// performed (ever) unless at least one consumer registers interest by calling
// RequestServerCheck().
-class GoogleURLTracker : public net::URLFetcherDelegate,
- public net::NetworkChangeNotifier::IPAddressObserver,
- public KeyedService {
+class GoogleURLTracker
+ : public net::URLFetcherDelegate,
+ public net::NetworkChangeNotifier::NetworkChangeObserver,
+ public KeyedService {
public:
// Callback that is called when the Google URL is updated. The arguments are
// the old and new URLs.
@@ -130,7 +131,8 @@ class GoogleURLTracker : public net::URLFetcherDelegate,
virtual void OnURLFetchComplete(const net::URLFetcher* source) OVERRIDE;
// NetworkChangeNotifier::IPAddressObserver:
- virtual void OnIPAddressChanged() OVERRIDE;
+ virtual void OnNetworkChanged(
+ net::NetworkChangeNotifier::ConnectionType type) OVERRIDE;
// KeyedService:
virtual void Shutdown() OVERRIDE;
« no previous file with comments | « chrome/browser/google/google_url_tracker_unittest.cc ('k') | components/google/core/browser/google_url_tracker.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698