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

Unified Diff: net/http/http_network_session.cc

Issue 9764003: Increase number of max sockets per group for WebSocket connections. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sync. Created 8 years, 9 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_session_peer.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
diff --git a/net/http/http_network_session.cc b/net/http/http_network_session.cc
index e10c12f7f150e8e950685e5862c0b85dab9b0dc2..f5cc7b15b6b006504dbdf6c260f470b1dd5bfe03 100644
--- a/net/http/http_network_session.cc
+++ b/net/http/http_network_session.cc
@@ -23,6 +23,7 @@
namespace {
net::ClientSocketPoolManager* CreateSocketPoolManager(
+ net::HttpNetworkSession::SocketPoolType pool_type,
const net::HttpNetworkSession::Params& params) {
// TODO(yutak): Differentiate WebSocket pool manager and allow more
// simultaneous connections for WebSockets.
@@ -38,7 +39,8 @@ net::ClientSocketPoolManager* CreateSocketPoolManager(
params.ssl_host_info_factory,
params.ssl_session_cache_shard,
params.proxy_service,
- params.ssl_config_service);
+ params.ssl_config_service,
+ pool_type);
}
} // unnamed namespace
@@ -55,8 +57,10 @@ HttpNetworkSession::HttpNetworkSession(const Params& params)
force_http_pipelining_(params.force_http_pipelining),
proxy_service_(params.proxy_service),
ssl_config_service_(params.ssl_config_service),
- normal_socket_pool_manager_(CreateSocketPoolManager(params)),
- websocket_socket_pool_manager_(CreateSocketPoolManager(params)),
+ normal_socket_pool_manager_(
+ CreateSocketPoolManager(NORMAL_SOCKET_POOL, params)),
+ websocket_socket_pool_manager_(
+ CreateSocketPoolManager(WEBSOCKET_SOCKET_POOL, params)),
spdy_session_pool_(params.host_resolver,
params.ssl_config_service,
params.http_server_properties),
@@ -142,8 +146,10 @@ ClientSocketPoolManager* HttpNetworkSession::GetSocketPoolManager(
return normal_socket_pool_manager_.get();
case WEBSOCKET_SOCKET_POOL:
return websocket_socket_pool_manager_.get();
+ default:
+ NOTREACHED();
+ break;
}
- NOTREACHED();
return NULL;
}
« no previous file with comments | « net/http/http_network_session.h ('k') | net/http/http_network_session_peer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698