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

Unified Diff: net/http/http_stream_factory_impl_job.h

Issue 7289006: Basic HTTP pipelining support (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Better Close() handling and tests Created 9 years, 4 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_job.h
diff --git a/net/http/http_stream_factory_impl_job.h b/net/http/http_stream_factory_impl_job.h
index ef0893d3c15a9bb49bd34d843a1d8cb8db188bf1..5e9634184892d2da7844262a0415c33d8e84af4e 100644
--- a/net/http/http_stream_factory_impl_job.h
+++ b/net/http/http_stream_factory_impl_job.h
@@ -24,6 +24,7 @@ namespace net {
class ClientSocketHandle;
class HttpAuthController;
class HttpNetworkSession;
+class HttpPipelinedConnection;
class HttpProxySocketParams;
class HttpStream;
class SOCKSSocketParams;
@@ -196,6 +197,8 @@ class HttpStreamFactoryImpl::Job {
// Should we force SPDY to run without SSL for this stream request.
bool ShouldForceSpdyWithoutSSL() const;
+ bool IsRequestEligibleForPipelining() const;
+
// Record histograms of latency until Connect() completes.
static void LogHttpConnectedMetrics(const ClientSocketHandle& handle);
@@ -274,6 +277,9 @@ class HttpStreamFactoryImpl::Job {
// Only used if |new_spdy_session_| is non-NULL.
bool spdy_session_direct_;
+ // Initialized when an existing pipeline can handle this job's request.
+ HttpPipelinedConnection* existing_available_pipeline_;
+
ScopedRunnableMethodFactory<Job> method_factory_;
DISALLOW_COPY_AND_ASSIGN(Job);

Powered by Google App Engine
This is Rietveld 408576698