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

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: Add NUM_SOCKET_POOL_TYPES. 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
Index: net/http/http_network_session.cc
diff --git a/net/http/http_network_session.cc b/net/http/http_network_session.cc
index 6cacaf36d84e174a08d50590c24c237e3946da62..d79077b9653b822b52430c6cb8e6c90ec42bddfd 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;
Yuta Kitamura 2012/03/21 00:47:02 This change was necessary to avoid a compiler warn
}
- NOTREACHED();
return NULL;
}

Powered by Google App Engine
This is Rietveld 408576698