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

Side by Side Diff: jingle/notifier/base/xmpp_client_socket_factory.cc

Issue 10389098: [Sync] Make ChromeAsyncSocket use only the hostname on connect (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments Created 8 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 | « jingle/notifier/base/proxy_resolving_client_socket.cc ('k') | no next file » | 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/notifier/base/xmpp_client_socket_factory.h" 5 #include "jingle/notifier/base/xmpp_client_socket_factory.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "jingle/notifier/base/fake_ssl_client_socket.h" 8 #include "jingle/notifier/base/fake_ssl_client_socket.h"
9 #include "jingle/notifier/base/proxy_resolving_client_socket.h" 9 #include "jingle/notifier/base/proxy_resolving_client_socket.h"
10 #include "net/socket/client_socket_factory.h" 10 #include "net/socket/client_socket_factory.h"
(...skipping 12 matching lines...) Expand all
23 request_context_getter_(request_context_getter), 23 request_context_getter_(request_context_getter),
24 ssl_config_(ssl_config), 24 ssl_config_(ssl_config),
25 use_fake_ssl_client_socket_(use_fake_ssl_client_socket) { 25 use_fake_ssl_client_socket_(use_fake_ssl_client_socket) {
26 CHECK(client_socket_factory_); 26 CHECK(client_socket_factory_);
27 } 27 }
28 28
29 XmppClientSocketFactory::~XmppClientSocketFactory() {} 29 XmppClientSocketFactory::~XmppClientSocketFactory() {}
30 30
31 net::StreamSocket* XmppClientSocketFactory::CreateTransportClientSocket( 31 net::StreamSocket* XmppClientSocketFactory::CreateTransportClientSocket(
32 const net::HostPortPair& host_and_port) { 32 const net::HostPortPair& host_and_port) {
33 // TODO(akalin): Use socket pools.
33 net::StreamSocket* transport_socket = new ProxyResolvingClientSocket( 34 net::StreamSocket* transport_socket = new ProxyResolvingClientSocket(
34 NULL, 35 NULL,
35 request_context_getter_, 36 request_context_getter_,
36 ssl_config_, 37 ssl_config_,
37 host_and_port); 38 host_and_port);
38 return (use_fake_ssl_client_socket_ ? 39 return (use_fake_ssl_client_socket_ ?
39 new FakeSSLClientSocket(transport_socket) : transport_socket); 40 new FakeSSLClientSocket(transport_socket) : transport_socket);
40 } 41 }
41 42
42 net::SSLClientSocket* XmppClientSocketFactory::CreateSSLClientSocket( 43 net::SSLClientSocket* XmppClientSocketFactory::CreateSSLClientSocket(
43 net::ClientSocketHandle* transport_socket, 44 net::ClientSocketHandle* transport_socket,
44 const net::HostPortPair& host_and_port) { 45 const net::HostPortPair& host_and_port) {
45 net::SSLClientSocketContext context; 46 net::SSLClientSocketContext context;
46 context.cert_verifier = 47 context.cert_verifier =
47 request_context_getter_->GetURLRequestContext()->cert_verifier(); 48 request_context_getter_->GetURLRequestContext()->cert_verifier();
48 // TODO(rkn): context.server_bound_cert_service is NULL because the 49 // TODO(rkn): context.server_bound_cert_service is NULL because the
49 // ServerBoundCertService class is not thread safe. 50 // ServerBoundCertService class is not thread safe.
50 return client_socket_factory_->CreateSSLClientSocket( 51 return client_socket_factory_->CreateSSLClientSocket(
51 transport_socket, host_and_port, ssl_config_, NULL, context); 52 transport_socket, host_and_port, ssl_config_, NULL, context);
52 } 53 }
53 54
54 55
55 } // namespace 56 } // namespace
OLDNEW
« no previous file with comments | « jingle/notifier/base/proxy_resolving_client_socket.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698