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

Unified Diff: net/base/host_resolver_impl.h

Issue 10873018: [net] Move DnsConfigService to NetworkChangeNotifier. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: add comment Created 8 years, 4 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
« no previous file with comments | « net/base/dns_reloader.cc ('k') | net/base/host_resolver_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/base/host_resolver_impl.h
diff --git a/net/base/host_resolver_impl.h b/net/base/host_resolver_impl.h
index 7044be31f8675026cbb6ad4d930e2e918a38cfdc..9e89d21df96361bfd71d4d40dddfe797b77374d1 100644
--- a/net/base/host_resolver_impl.h
+++ b/net/base/host_resolver_impl.h
@@ -26,8 +26,6 @@ namespace net {
class BoundNetLog;
class DnsClient;
-struct DnsConfig;
-class DnsConfigService;
class NetLog;
// For each hostname that is requested, HostResolver creates a
@@ -62,6 +60,7 @@ class NET_EXPORT HostResolverImpl
: public HostResolver,
NON_EXPORTED_BASE(public base::NonThreadSafe),
public NetworkChangeNotifier::IPAddressObserver,
+ public NetworkChangeNotifier::DNSObserver,
public base::SupportsWeakPtr<HostResolverImpl> {
public:
// Parameters for ProcTask which resolves hostnames using HostResolveProc.
@@ -120,7 +119,6 @@ class NET_EXPORT HostResolverImpl
HostResolverImpl(HostCache* cache,
const PrioritizedDispatcher::Limits& job_limits,
const ProcTaskParams& proc_params,
- scoped_ptr<DnsConfigService> dns_config_service,
scoped_ptr<DnsClient> dns_client,
NetLog* net_log);
@@ -221,8 +219,8 @@ class NET_EXPORT HostResolverImpl
// NetworkChangeNotifier::IPAddressObserver:
virtual void OnIPAddressChanged() OVERRIDE;
- // DnsConfigService callback:
- void OnDnsConfigChanged(const DnsConfig& dns_config);
+ // NetworkChangeNotifier::DNSObserver:
+ virtual void OnDNSChanged() OVERRIDE;
// True if have a DnsClient with a valid DnsConfig.
bool HaveDnsConfig() const;
@@ -250,8 +248,6 @@ class NET_EXPORT HostResolverImpl
// Address family to use when the request doesn't specify one.
AddressFamily default_address_family_;
- scoped_ptr<DnsConfigService> dns_config_service_;
-
// If present, used by DnsTask and ServeFromHosts to resolve requests.
scoped_ptr<DnsClient> dns_client_;
« no previous file with comments | « net/base/dns_reloader.cc ('k') | net/base/host_resolver_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698