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

Unified Diff: net/http/http_stream_factory_impl_request.h

Issue 7289006: Basic HTTP pipelining support (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge Created 9 years, 2 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_stream_factory_impl_job.cc ('k') | net/http/http_stream_factory_impl_request.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_stream_factory_impl_request.h
diff --git a/net/http/http_stream_factory_impl_request.h b/net/http/http_stream_factory_impl_request.h
index 4a7ace49884fb44d362bf286fd98eb9447d5922e..da81b73cf43541a3f9640c7b07e2042df24a19e6 100644
--- a/net/http/http_stream_factory_impl_request.h
+++ b/net/http/http_stream_factory_impl_request.h
@@ -30,6 +30,11 @@ class HttpStreamFactoryImpl::Request : public HttpStreamRequest {
// before knowing if SPDY is available.
void SetSpdySessionKey(const HostPortProxyPair& spdy_session_key);
+ // Called when the Job determines the appropriate |http_pipelining_key| for
+ // the Request. Registers this Request with the factory, so that if an
+ // existing pipeline becomes available, this Request can be late bound to it.
+ void SetHttpPipeliningKey(const HostPortPair& http_pipelining_key);
+
// Attaches |job| to this request. Does not mean that Request will use |job|,
// but Request will own |job|.
void AttachJob(HttpStreamFactoryImpl::Job* job);
@@ -41,10 +46,14 @@ class HttpStreamFactoryImpl::Request : public HttpStreamRequest {
bool using_spdy,
const NetLog::Source& source);
- // If this Request has a spdy_session_key, remove this session from the
+ // If this Request has a |spdy_session_key_|, remove this session from the
// SpdySessionRequestMap.
void RemoveRequestFromSpdySessionRequestMap();
+ // If this Request has a |http_pipelining_key_|, remove this session from the
+ // HttpPipeliningRequestMap.
+ void RemoveRequestFromHttpPipeliningRequestMap();
+
// Called by an attached Job if it sets up a SpdySession.
void OnSpdySessionReady(Job* job,
scoped_refptr<SpdySession> spdy_session,
@@ -102,6 +111,7 @@ class HttpStreamFactoryImpl::Request : public HttpStreamRequest {
scoped_ptr<Job> bound_job_;
std::set<HttpStreamFactoryImpl::Job*> jobs_;
scoped_ptr<const HostPortProxyPair> spdy_session_key_;
+ scoped_ptr<const HostPortPair> http_pipelining_key_;
bool completed_;
bool was_npn_negotiated_;
« no previous file with comments | « net/http/http_stream_factory_impl_job.cc ('k') | net/http/http_stream_factory_impl_request.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698