From b59ff376c5d5b950774fcbe65727611d51832b75 Mon Sep 17 00:00:00 2001 From: "ericroman@google.com" Date: Wed, 15 Jul 2009 22:04:32 +0000 Subject: Refactorings surrounding HostResolver: (1) Extract HostResolver to an interface. The existing concrete implementation is now named HostResolverImpl. This makes it possible to create mocks with more complex behavior (i.e. choose via rules if response will be sync vs async). (2) Transform HostMapper into HostResolverProc. Conceptually HostResolverProc maps a hostname to a socket address, whereas HostMapper mapped a hostname to another hostname (so you were still at the mercy of the system's host resolver). With HostResolverProc you can specify the exact AddressList, making it possible to run tests requiring IPv6 socketaddrs on systems (like WinXP) that don't actually support it. (3) Add a MockHostResolver implementation of HostResolver. This replaces the [ScopedHostMapper + RuleBasedHostMapper + HostResolver] combo. It is less clunky and a bit more expressive. BUG=http://crbug.com/16452 R=willchan TEST=existing Review URL: http://codereview.chromium.org/149511 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20795 0039d316-1c4b-4281-b951-d872f2087c98 --- net/url_request/url_request_unittest.cc | 2 +- net/url_request/url_request_unittest.h | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) (limited to 'net/url_request') diff --git a/net/url_request/url_request_unittest.cc b/net/url_request/url_request_unittest.cc index 220d7e0..2aaac0a 100644 --- a/net/url_request/url_request_unittest.cc +++ b/net/url_request/url_request_unittest.cc @@ -45,7 +45,7 @@ namespace { class URLRequestHttpCacheContext : public URLRequestContext { public: URLRequestHttpCacheContext() { - host_resolver_ = new net::HostResolver; + host_resolver_ = net::CreateSystemHostResolver(); proxy_service_ = net::ProxyService::CreateNull(); http_transaction_factory_ = new net::HttpCache( diff --git a/net/url_request/url_request_unittest.h b/net/url_request/url_request_unittest.h index a08d776..61a1712 100644 --- a/net/url_request/url_request_unittest.h +++ b/net/url_request/url_request_unittest.h @@ -43,7 +43,7 @@ using base::TimeDelta; class TestURLRequestContext : public URLRequestContext { public: TestURLRequestContext() { - host_resolver_ = new net::HostResolver; + host_resolver_ = net::CreateSystemHostResolver(); proxy_service_ = net::ProxyService::CreateNull(); http_transaction_factory_ = net::HttpNetworkLayer::CreateFactory(host_resolver_, @@ -51,7 +51,7 @@ class TestURLRequestContext : public URLRequestContext { } explicit TestURLRequestContext(const std::string& proxy) { - host_resolver_ = new net::HostResolver; + host_resolver_ = net::CreateSystemHostResolver(); net::ProxyConfig proxy_config; proxy_config.proxy_rules.ParseFromString(proxy); proxy_service_ = net::ProxyService::CreateFixed(proxy_config); -- cgit v1.1