diff options
author | simonjam@chromium.org <simonjam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-01 21:31:31 +0000 |
---|---|---|
committer | simonjam@chromium.org <simonjam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-01 21:31:31 +0000 |
commit | 5477d890353c732bbd0c16d8a3c5ce56a52ee2fe (patch) | |
tree | b225cdf9c0dba2d2c77cc76b094551a8be9ba5cf /net/http/http_network_session.cc | |
parent | 271398aa3ff859d3deace9ff3ca575f14d263964 (diff) | |
download | chromium_src-5477d890353c732bbd0c16d8a3c5ce56a52ee2fe.zip chromium_src-5477d890353c732bbd0c16d8a3c5ce56a52ee2fe.tar.gz chromium_src-5477d890353c732bbd0c16d8a3c5ce56a52ee2fe.tar.bz2 |
Add a force pipelining option to load flags.
Details:
- Add a HttpPipelinedHostForced class for connections with forced requests.
+ Forced requests get their own pipeline and there's only one per host.
+ They always try to pipeline and won't retry if evicted.
+ Only one HttpStreamFactoryImpl::Job runs for all requests to the same
origin with forced pipelining. All requests will fail if that Job fails.
- Track HttpPipelinedHosts with a Key. Right now that's origin and
force-pipelining, but it might be expanded to include content type.
- Add a BufferedWriteStreamSocket that wraps a normal socket. It buffers Write()
calls until a task fires to dispatch the buffer to the underlying socket.
BUG=110794
TEST=net_unittests and unit_tests
Review URL: http://codereview.chromium.org/9433015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124487 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/http/http_network_session.cc')
-rw-r--r-- | net/http/http_network_session.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/net/http/http_network_session.cc b/net/http/http_network_session.cc index 1a59c9f..6c10c1a 100644 --- a/net/http/http_network_session.cc +++ b/net/http/http_network_session.cc @@ -1,4 +1,4 @@ -// Copyright (c) 2011 The Chromium Authors. All rights reserved. +// Copyright (c) 2012 The Chromium Authors. All rights reserved. // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. @@ -29,6 +29,7 @@ HttpNetworkSession::HttpNetworkSession(const Params& params) http_server_properties_(params.http_server_properties), cert_verifier_(params.cert_verifier), http_auth_handler_factory_(params.http_auth_handler_factory), + force_http_pipelining_(params.force_http_pipelining), proxy_service_(params.proxy_service), ssl_config_service_(params.ssl_config_service), socket_pool_manager_( @@ -49,7 +50,8 @@ HttpNetworkSession::HttpNetworkSession(const Params& params) params.ssl_config_service, params.http_server_properties), ALLOW_THIS_IN_INITIALIZER_LIST(http_stream_factory_( - new HttpStreamFactoryImpl(this))) { + new HttpStreamFactoryImpl(this))), + params_(params) { DCHECK(proxy_service_); DCHECK(ssl_config_service_); CHECK(http_server_properties_); |