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_job.h

Issue 2771263002: Retry upon 421 status code without IP pooling. (Closed)
Patch Set: Rebase. Created 3 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_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 9201edce42cd196435d48a163e54d88a3333698f..0ae7222b79239c2f25219d91a8f6a3eb125b4a5d 100644
--- a/net/http/http_stream_factory_impl_job.h
+++ b/net/http/http_stream_factory_impl_job.h
@@ -170,6 +170,7 @@ class HttpStreamFactoryImpl::Job {
const SSLConfig& proxy_ssl_config,
HostPortPair destination,
GURL origin_url,
+ bool enable_ip_based_pooling,
NetLog* net_log);
// Constructor for the alternative Job. The Job is owned by |delegate|, hence
@@ -191,6 +192,7 @@ class HttpStreamFactoryImpl::Job {
GURL origin_url,
AlternativeService alternative_service,
const ProxyServer& alternative_proxy_server,
+ bool enable_ip_based_pooling,
NetLog* net_log);
virtual ~Job();
@@ -392,6 +394,7 @@ class HttpStreamFactoryImpl::Job {
static int OnHostResolution(SpdySessionPool* spdy_session_pool,
const SpdySessionKey& spdy_session_key,
const GURL& origin_url,
+ bool enable_ip_based_pooling,
const AddressList& addresses,
const NetLogWithSource& net_log);
@@ -424,6 +427,10 @@ class HttpStreamFactoryImpl::Job {
// request.
const ProxyServer alternative_proxy_server_;
+ // Enable pooling to a SpdySession with matching IP and certificate
+ // even if the SpdySessionKey is different.
+ const bool enable_ip_based_pooling_;
+
// Unowned. |this| job is owned by |delegate_|.
Delegate* delegate_;
@@ -502,6 +509,7 @@ class HttpStreamFactoryImpl::JobFactory {
HostPortPair destination,
GURL origin_url,
AlternativeService alternative_service,
+ bool enable_ip_based_pooling,
NetLog* net_log) = 0;
// Creates an alternative proxy server Job.
@@ -516,6 +524,7 @@ class HttpStreamFactoryImpl::JobFactory {
HostPortPair destination,
GURL origin_url,
const ProxyServer& alternative_proxy_server,
+ bool enable_ip_based_pooling,
NetLog* net_log) = 0;
// Creates a non-alternative Job.
@@ -529,6 +538,7 @@ class HttpStreamFactoryImpl::JobFactory {
const SSLConfig& proxy_ssl_config,
HostPortPair destination,
GURL origin_url,
+ bool enable_ip_based_pooling,
NetLog* net_log) = 0;
};

Powered by Google App Engine
This is Rietveld 408576698