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

Unified Diff: net/base/network_change_notifier_mac.h

Issue 10377092: [net/dns] Isolate DnsConfigWatcher from DnsConfigService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Comments. Braces. Created 8 years, 7 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: net/base/network_change_notifier_mac.h
diff --git a/net/base/network_change_notifier_mac.h b/net/base/network_change_notifier_mac.h
index ec2bb1fed50e97c42be5bf424f01e8e9117f9546..c95e952b873c2ef7a17cc24099e2168972784ac4 100644
--- a/net/base/network_change_notifier_mac.h
+++ b/net/base/network_change_notifier_mac.h
@@ -24,7 +24,7 @@ class NetworkChangeNotifierMac: public NetworkChangeNotifier {
NetworkChangeNotifierMac();
virtual ~NetworkChangeNotifierMac();
- // NetworkChangeNotifier implementation:
+ // NetworkChangeNotifier:
virtual bool IsCurrentlyOffline() const OVERRIDE;
private:
@@ -34,6 +34,8 @@ class NetworkChangeNotifierMac: public NetworkChangeNotifier {
ONLINE = 1
};
+ class DnsWatcherThread;
+
// Forwarder just exists to keep the NetworkConfigWatcherMac API out of
// NetworkChangeNotifierMac's public API.
class Forwarder : public NetworkConfigWatcherMac::Delegate {
@@ -83,6 +85,9 @@ class NetworkChangeNotifierMac: public NetworkChangeNotifier {
Forwarder forwarder_;
scoped_ptr<const NetworkConfigWatcherMac> config_watcher_;
+ // Thread on which we can run DnsConfigWatcher, which requires TYPE_IO.
+ scoped_ptr<DnsWatcherThread> dns_watcher_thread_;
+
DISALLOW_COPY_AND_ASSIGN(NetworkChangeNotifierMac);
};

Powered by Google App Engine
This is Rietveld 408576698