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

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: Added unit tests. Depends on CL 9251019 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..cb3430584489e79446d0e26f07180f2c3ee373fe 100644
--- a/net/dns/async_host_resolver.h
+++ b/net/dns/async_host_resolver.h
@@ -17,7 +17,7 @@
#include "net/base/host_resolver.h"
#include "net/base/ip_endpoint.h"
#include "net/base/net_log.h"
-#include "net/dns/dns_client.h"
+#include "net/dns/dns_transaction.h"
namespace net {
@@ -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_;
mmenke 2012/01/19 17:24:47 nit: max_dns_transactions
// List of current DNS requests.
- DnsRequestList dns_requests_;
+ DnsTransactionList dns_requests_;
mmenke 2012/01/19 17:24:47 nit: 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