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

Unified Diff: net/http/http_stream_factory_impl_request.h

Issue 275953002: Remove HTTP pipelining support. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge Created 6 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
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 22c8ff2d485bedabb520b86e176e866229660de0..55f13b9be2cc1a65755ddb02448c4dc941f41191 100644
--- a/net/http/http_stream_factory_impl_request.h
+++ b/net/http/http_stream_factory_impl_request.h
@@ -38,12 +38,6 @@ class HttpStreamFactoryImpl::Request : public HttpStreamRequest {
// before knowing if SPDY is available.
void SetSpdySessionKey(const SpdySessionKey& 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.
- // Returns true if this is this key was new to the factory.
- bool SetHttpPipeliningKey(const HttpPipelinedHost::Key& 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);
@@ -59,10 +53,6 @@ class HttpStreamFactoryImpl::Request : public HttpStreamRequest {
// 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 OnNewSpdySessionReady(Job* job,
scoped_ptr<HttpStream> stream,
@@ -137,7 +127,6 @@ class HttpStreamFactoryImpl::Request : public HttpStreamRequest {
scoped_ptr<Job> bound_job_;
std::set<HttpStreamFactoryImpl::Job*> jobs_;
scoped_ptr<const SpdySessionKey> spdy_session_key_;
- scoped_ptr<const HttpPipelinedHost::Key> http_pipelining_key_;
bool completed_;
bool was_npn_negotiated_;

Powered by Google App Engine
This is Rietveld 408576698