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

Unified Diff: net/http/http_network_session.cc

Issue 2363003: Rework the logging for sockets/connectjobs.... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 7 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
« no previous file with comments | « net/http/http_network_session.h ('k') | net/http/http_network_transaction_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_network_session.cc
===================================================================
--- net/http/http_network_session.cc (revision 48758)
+++ net/http/http_network_session.cc (working copy)
@@ -42,7 +42,8 @@
ClientSocketFactory* client_socket_factory,
SSLConfigService* ssl_config_service,
SpdySessionPool* spdy_session_pool,
- HttpAuthHandlerFactory* http_auth_handler_factory)
+ HttpAuthHandlerFactory* http_auth_handler_factory,
+ NetLog* net_log)
: network_change_notifier_(network_change_notifier),
// TODO(vandebo) when we've completely converted to pools, the base TCP
// pool name should get changed to TCP instead of Transport.
@@ -53,13 +54,15 @@
socks_pool_histograms_(new ClientSocketPoolHistograms("SOCK")),
tcp_socket_pool_(new TCPClientSocketPool(
g_max_sockets, g_max_sockets_per_group, tcp_pool_histograms_,
- host_resolver, client_socket_factory, network_change_notifier_)),
+ host_resolver, client_socket_factory, network_change_notifier_,
+ net_log)),
socket_factory_(client_socket_factory),
host_resolver_(host_resolver),
proxy_service_(proxy_service),
ssl_config_service_(ssl_config_service),
spdy_session_pool_(spdy_session_pool),
- http_auth_handler_factory_(http_auth_handler_factory) {
+ http_auth_handler_factory_(http_auth_handler_factory),
+ net_log_(net_log) {
DCHECK(proxy_service);
DCHECK(ssl_config_service);
}
@@ -81,7 +84,8 @@
new TCPClientSocketPool(
g_max_sockets_per_proxy_server, g_max_sockets_per_group,
http_proxy_pool_histograms_, host_resolver_, socket_factory_,
- network_change_notifier_)));
+ network_change_notifier_,
+ net_log_)));
return ret.first->second;
}
@@ -104,8 +108,9 @@
g_max_sockets_per_group,
tcp_for_socks_pool_histograms_,
host_resolver_, socket_factory_,
- network_change_notifier_),
- network_change_notifier_)));
+ network_change_notifier_,
+ net_log_),
+ network_change_notifier_, net_log_)));
return ret.first->second;
}
@@ -144,7 +149,8 @@
tcp_pool_histograms_,
host_resolver_,
socket_factory_,
- network_change_notifier_);
+ network_change_notifier_,
+ net_log_);
}
} // namespace net
« no previous file with comments | « net/http/http_network_session.h ('k') | net/http/http_network_transaction_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698