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

Unified Diff: net/http/http_proxy_client_socket_pool.cc

Issue 851503003: Update from https://crrev.com/311076 (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 11 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_proxy_client_socket_pool.h ('k') | net/http/http_proxy_client_socket_pool_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_proxy_client_socket_pool.cc
diff --git a/net/http/http_proxy_client_socket_pool.cc b/net/http/http_proxy_client_socket_pool.cc
index 1f45882c526ed6bcdcffa2d21d2dc7ba55b11b4b..6c421c512151f8e10d4b775bd6582ddadb368e35 100644
--- a/net/http/http_proxy_client_socket_pool.cc
+++ b/net/http/http_proxy_client_socket_pool.cc
@@ -86,7 +86,6 @@ HttpProxyConnectJob::HttpProxyConnectJob(
const base::TimeDelta& timeout_duration,
TransportClientSocketPool* transport_pool,
SSLClientSocketPool* ssl_pool,
- HostResolver* host_resolver,
Delegate* delegate,
NetLog* net_log)
: ConnectJob(group_name, timeout_duration, priority, delegate,
@@ -94,7 +93,6 @@ HttpProxyConnectJob::HttpProxyConnectJob(
params_(params),
transport_pool_(transport_pool),
ssl_pool_(ssl_pool),
- resolver_(host_resolver),
using_spdy_(false),
protocol_negotiated_(kProtoUnknown),
weak_ptr_factory_(this) {
@@ -308,6 +306,9 @@ int HttpProxyConnectJob::DoHttpProxyConnectComplete(int result) {
SetSocket(transport_socket_.Pass());
}
+ if (result == ERR_HTTP_1_1_REQUIRED)
+ return ERR_PROXY_HTTP_1_1_REQUIRED;
+
return result;
}
@@ -395,12 +396,10 @@ HttpProxyConnectJobFactory::HttpProxyConnectJobFactory(
TransportClientSocketPool* transport_pool,
SSLClientSocketPool* ssl_pool,
HostResolver* host_resolver,
- const ProxyDelegate* proxy_delegate,
NetLog* net_log)
: transport_pool_(transport_pool),
ssl_pool_(ssl_pool),
host_resolver_(host_resolver),
- proxy_delegate_(proxy_delegate),
net_log_(net_log) {
base::TimeDelta max_pool_timeout = base::TimeDelta();
@@ -430,7 +429,6 @@ HttpProxyClientSocketPool::HttpProxyConnectJobFactory::NewConnectJob(
ConnectionTimeout(),
transport_pool_,
ssl_pool_,
- host_resolver_,
delegate,
net_log_));
}
@@ -448,7 +446,6 @@ HttpProxyClientSocketPool::HttpProxyClientSocketPool(
HostResolver* host_resolver,
TransportClientSocketPool* transport_pool,
SSLClientSocketPool* ssl_pool,
- const ProxyDelegate* proxy_delegate,
NetLog* net_log)
: transport_pool_(transport_pool),
ssl_pool_(ssl_pool),
@@ -458,7 +455,6 @@ HttpProxyClientSocketPool::HttpProxyClientSocketPool(
new HttpProxyConnectJobFactory(transport_pool,
ssl_pool,
host_resolver,
- proxy_delegate,
net_log)) {
// We should always have a |transport_pool_| except in unit tests.
if (transport_pool_)
« no previous file with comments | « net/http/http_proxy_client_socket_pool.h ('k') | net/http/http_proxy_client_socket_pool_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698