Chromium Code Reviews

Unified Diff: chrome/browser/net/connection_tester_unittest.cc

Issue 6012008: Don't leak proxy script fetcher's http transaction factories. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 10 years ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
« no previous file with comments | « chrome/browser/io_thread.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/net/connection_tester_unittest.cc
diff --git a/chrome/browser/net/connection_tester_unittest.cc b/chrome/browser/net/connection_tester_unittest.cc
index 0ee05d20875427286471b16bfeebe95635dfc924..6735599b087117d85df0c053156f6c086ffde0b0 100644
--- a/chrome/browser/net/connection_tester_unittest.cc
+++ b/chrome/browser/net/connection_tester_unittest.cc
@@ -117,7 +117,7 @@ class ConnectionTesterTest : public PlatformTest {
proxy_service_ = net::ProxyService::CreateDirect();
proxy_script_fetcher_context_->set_proxy_service(proxy_service_);
ssl_config_service_ = net::SSLConfigService::CreateSystemSSLConfigService();
- proxy_script_fetcher_context_->set_http_transaction_factory(
+ http_transaction_factory_.reset(
new net::HttpNetworkLayer(
client_socket_factory_,
&host_resolver_,
@@ -131,6 +131,8 @@ class ConnectionTesterTest : public PlatformTest {
&http_auth_handler_factory_,
NULL /* NetworkDelegate */,
NULL /* NetLog */));
+ proxy_script_fetcher_context_->set_http_transaction_factory(
+ http_transaction_factory_.get());
// In-memory cookie store.
proxy_script_fetcher_context_->set_cookie_store(
new net::CookieMonster(NULL, NULL));
« no previous file with comments | « chrome/browser/io_thread.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine