diff options
author | ericroman@google.com <ericroman@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-12 03:23:54 +0000 |
---|---|---|
committer | ericroman@google.com <ericroman@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-12 03:23:54 +0000 |
commit | d66688b2a167882dad56fcb6d963d918ed413052 (patch) | |
tree | 7b12c4d606533caa828accc14716c4fb06cb31f2 /net | |
parent | a1257b12bd288161e1eadeff9f2378de9b6e029a (diff) | |
download | chromium_src-d66688b2a167882dad56fcb6d963d918ed413052.zip chromium_src-d66688b2a167882dad56fcb6d963d918ed413052.tar.gz chromium_src-d66688b2a167882dad56fcb6d963d918ed413052.tar.bz2 |
Experiment: turn off host caching to see if it relates to client socket pool leak in URLRequestTest.Cancel[24].
Review URL: http://codereview.chromium.org/126035
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18249 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net')
-rw-r--r-- | net/url_request/url_request_unittest.cc | 4 | ||||
-rw-r--r-- | net/url_request/url_request_unittest.h | 8 |
2 files changed, 9 insertions, 3 deletions
diff --git a/net/url_request/url_request_unittest.cc b/net/url_request/url_request_unittest.cc index 18ff111..e4301b5 100644 --- a/net/url_request/url_request_unittest.cc +++ b/net/url_request/url_request_unittest.cc @@ -45,7 +45,9 @@ namespace { class URLRequestHttpCacheContext : public URLRequestContext { public: URLRequestHttpCacheContext() { - host_resolver_ = new net::HostResolver; + // TODO(eroman): we turn off host caching to see if synchronous + // host resolving interacts poorly with client socket pool. [experiment] + host_resolver_ = new net::HostResolver(0, 0); 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 1c880fd..3210656 100644 --- a/net/url_request/url_request_unittest.h +++ b/net/url_request/url_request_unittest.h @@ -43,7 +43,9 @@ using base::TimeDelta; class TestURLRequestContext : public URLRequestContext { public: TestURLRequestContext() { - host_resolver_ = new net::HostResolver; + // TODO(eroman): we turn off host caching to see if synchronous + // host resolving interacts poorly with client socket pool. [experiment] + host_resolver_ = new net::HostResolver(0, 0); proxy_service_ = net::ProxyService::CreateNull(); http_transaction_factory_ = net::HttpNetworkLayer::CreateFactory(host_resolver_, @@ -51,7 +53,9 @@ class TestURLRequestContext : public URLRequestContext { } explicit TestURLRequestContext(const std::string& proxy) { - host_resolver_ = new net::HostResolver; + // TODO(eroman): we turn off host caching to see if synchronous + // host resolving interacts poorly with client socket pool. [experiment] + host_resolver_ = new net::HostResolver(0, 0); net::ProxyConfig proxy_config; proxy_config.proxy_rules.ParseFromString(proxy); proxy_service_ = net::ProxyService::CreateFixed(proxy_config); |