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

Unified Diff: net/dns/async_host_resolver.h

Issue 9190031: DnsClient refactoring + features (timeout, suffix search, server rotation). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Responded to review. Renamed DnsClient -> DnsTransactionFactory. Completed logging. Created 8 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: net/dns/async_host_resolver.h
diff --git a/net/dns/async_host_resolver.h b/net/dns/async_host_resolver.h
index e8aeb8b90583685dd1a86bb9fb87c4d58d2f63ec..1b9713cd13a660b43d9e04cdca9313c38a27bcfd 100644
--- a/net/dns/async_host_resolver.h
+++ b/net/dns/async_host_resolver.h
@@ -28,7 +28,7 @@ class NET_EXPORT AsyncHostResolver
AsyncHostResolver(size_t max_dns_requests,
size_t max_pending_requests,
HostCache* cache,
- DnsClient* client,
+ scoped_ptr<DnsTransactionFactory> client,
NetLog* net_log);
virtual ~AsyncHostResolver();
@@ -46,9 +46,9 @@ class NET_EXPORT AsyncHostResolver
virtual AddressFamily GetDefaultAddressFamily() const OVERRIDE;
virtual HostCache* GetHostCache() OVERRIDE;
- void OnDnsRequestComplete(DnsClient::Request* request,
- int result,
- const DnsResponse* transaction);
+ void OnDnsTransactionComplete(DnsTransaction* transaction,
+ int result,
+ const DnsResponse* response);
private:
FRIEND_TEST_ALL_PREFIXES(AsyncHostResolverTest, QueuedLookup);
@@ -64,7 +64,7 @@ class NET_EXPORT AsyncHostResolver
typedef std::pair<std::string, uint16> Key;
typedef std::list<Request*> RequestList;
- typedef std::list<const DnsClient::Request*> DnsRequestList;
+ typedef std::list<const DnsTransaction*> DnsTransactionList;
typedef std::map<Key, RequestList> KeyRequestListMap;
// Create a new request for the incoming Resolve() call.
@@ -112,7 +112,7 @@ class NET_EXPORT AsyncHostResolver
size_t max_dns_requests_;
cbentzel 2012/01/13 22:17:25 Perhaps max_dns_transactions_.
// List of current DNS requests.
- DnsRequestList dns_requests_;
+ DnsTransactionList dns_requests_;
cbentzel 2012/01/13 22:17:25 dns_transactions_
// A map from Key to a list of requests waiting for the Key to resolve.
KeyRequestListMap requestlist_map_;
@@ -126,7 +126,7 @@ class NET_EXPORT AsyncHostResolver
// Cache of host resolution results.
scoped_ptr<HostCache> cache_;
- DnsClient* client_;
+ scoped_ptr<DnsTransactionFactory> client_;
NetLog* net_log_;

Powered by Google App Engine
This is Rietveld 408576698