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

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

Issue 16501002: Give more request types a TransportSecurityState. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Enforce CalledOnValidThread in all non-static methods. Created 7 years, 6 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 | « jingle/glue/chrome_async_socket_unittest.cc ('k') | net/http/http_network_layer_unittest.cc » ('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 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 request_context_getter->GetURLRequestContext(); 47 request_context_getter->GetURLRequestContext();
48 DCHECK(request_context); 48 DCHECK(request_context);
49 DCHECK(!dest_host_port_pair_.host().empty()); 49 DCHECK(!dest_host_port_pair_.host().empty());
50 DCHECK_GT(dest_host_port_pair_.port(), 0); 50 DCHECK_GT(dest_host_port_pair_.port(), 0);
51 DCHECK(proxy_url_.is_valid()); 51 DCHECK(proxy_url_.is_valid());
52 52
53 net::HttpNetworkSession::Params session_params; 53 net::HttpNetworkSession::Params session_params;
54 session_params.client_socket_factory = socket_factory; 54 session_params.client_socket_factory = socket_factory;
55 session_params.host_resolver = request_context->host_resolver(); 55 session_params.host_resolver = request_context->host_resolver();
56 session_params.cert_verifier = request_context->cert_verifier(); 56 session_params.cert_verifier = request_context->cert_verifier();
57 session_params.transport_security_state =
58 request_context->transport_security_state();
57 // TODO(rkn): This is NULL because ServerBoundCertService is not thread safe. 59 // TODO(rkn): This is NULL because ServerBoundCertService is not thread safe.
58 session_params.server_bound_cert_service = NULL; 60 session_params.server_bound_cert_service = NULL;
59 // transport_security_state is NULL because it's not thread safe.
60 session_params.transport_security_state = NULL;
61 session_params.proxy_service = request_context->proxy_service(); 61 session_params.proxy_service = request_context->proxy_service();
62 session_params.ssl_config_service = request_context->ssl_config_service(); 62 session_params.ssl_config_service = request_context->ssl_config_service();
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 =
(...skipping 318 matching lines...) Expand 10 before | Expand all | Expand 10 after
389 return false; 389 return false;
390 } 390 }
391 391
392 void ProxyResolvingClientSocket::CloseTransportSocket() { 392 void ProxyResolvingClientSocket::CloseTransportSocket() {
393 if (transport_.get() && transport_->socket()) 393 if (transport_.get() && transport_->socket())
394 transport_->socket()->Disconnect(); 394 transport_->socket()->Disconnect();
395 transport_.reset(); 395 transport_.reset();
396 } 396 }
397 397
398 } // namespace jingle_glue 398 } // namespace jingle_glue
OLDNEW
« no previous file with comments | « jingle/glue/chrome_async_socket_unittest.cc ('k') | net/http/http_network_layer_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698