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

Side by Side Diff: jingle/glue/proxy_resolving_client_socket.cc

Issue 284423002: Remove HttpStreamFactory's NPN/SPDY globals, except for spdy_enabled. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « components/domain_reliability/uploader_unittest.cc ('k') | net/http/http_network_session.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "jingle/glue/proxy_resolving_client_socket.h" 5 #include "jingle/glue/proxy_resolving_client_socket.h"
6 6
7 #include "base/basictypes.h" 7 #include "base/basictypes.h"
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/bind_helpers.h" 9 #include "base/bind_helpers.h"
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
63 session_params.http_auth_handler_factory = 63 session_params.http_auth_handler_factory =
64 request_context->http_auth_handler_factory(); 64 request_context->http_auth_handler_factory();
65 session_params.network_delegate = request_context->network_delegate(); 65 session_params.network_delegate = request_context->network_delegate();
66 session_params.http_server_properties = 66 session_params.http_server_properties =
67 request_context->http_server_properties(); 67 request_context->http_server_properties();
68 session_params.net_log = request_context->net_log(); 68 session_params.net_log = request_context->net_log();
69 69
70 const net::HttpNetworkSession::Params* reference_params = 70 const net::HttpNetworkSession::Params* reference_params =
71 request_context->GetNetworkSessionParams(); 71 request_context->GetNetworkSessionParams();
72 if (reference_params) { 72 if (reference_params) {
73 // TODO(mmenke): Just copying specific parameters seems highly regression
74 // prone. Should have a better way to do this.
73 session_params.host_mapping_rules = reference_params->host_mapping_rules; 75 session_params.host_mapping_rules = reference_params->host_mapping_rules;
74 session_params.ignore_certificate_errors = 76 session_params.ignore_certificate_errors =
75 reference_params->ignore_certificate_errors; 77 reference_params->ignore_certificate_errors;
76 session_params.http_pipelining_enabled = 78 session_params.http_pipelining_enabled =
77 reference_params->http_pipelining_enabled; 79 reference_params->http_pipelining_enabled;
78 session_params.testing_fixed_http_port = 80 session_params.testing_fixed_http_port =
79 reference_params->testing_fixed_http_port; 81 reference_params->testing_fixed_http_port;
80 session_params.testing_fixed_https_port = 82 session_params.testing_fixed_https_port =
81 reference_params->testing_fixed_https_port; 83 reference_params->testing_fixed_https_port;
84 session_params.next_protos = reference_params->next_protos;
82 session_params.trusted_spdy_proxy = reference_params->trusted_spdy_proxy; 85 session_params.trusted_spdy_proxy = reference_params->trusted_spdy_proxy;
86 session_params.force_spdy_over_ssl = reference_params->force_spdy_over_ssl;
87 session_params.force_spdy_always = reference_params->force_spdy_always;
88 session_params.forced_spdy_exclusions =
89 reference_params->forced_spdy_exclusions;
90 session_params.use_alternate_protocols =
91 reference_params->use_alternate_protocols;
83 } 92 }
84 93
85 network_session_ = new net::HttpNetworkSession(session_params); 94 network_session_ = new net::HttpNetworkSession(session_params);
86 } 95 }
87 96
88 ProxyResolvingClientSocket::~ProxyResolvingClientSocket() { 97 ProxyResolvingClientSocket::~ProxyResolvingClientSocket() {
89 Disconnect(); 98 Disconnect();
90 } 99 }
91 100
92 int ProxyResolvingClientSocket::Read(net::IOBuffer* buf, int buf_len, 101 int ProxyResolvingClientSocket::Read(net::IOBuffer* buf, int buf_len,
(...skipping 296 matching lines...) Expand 10 before | Expand all | Expand 10 after
389 return false; 398 return false;
390 } 399 }
391 400
392 void ProxyResolvingClientSocket::CloseTransportSocket() { 401 void ProxyResolvingClientSocket::CloseTransportSocket() {
393 if (transport_.get() && transport_->socket()) 402 if (transport_.get() && transport_->socket())
394 transport_->socket()->Disconnect(); 403 transport_->socket()->Disconnect();
395 transport_.reset(); 404 transport_.reset();
396 } 405 }
397 406
398 } // namespace jingle_glue 407 } // namespace jingle_glue
OLDNEW
« no previous file with comments | « components/domain_reliability/uploader_unittest.cc ('k') | net/http/http_network_session.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698