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

Unified 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « jingle/notifier/base/proxy_resolving_client_socket.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: jingle/notifier/base/xmpp_client_socket_factory.cc
diff --git a/jingle/notifier/base/xmpp_client_socket_factory.cc b/jingle/notifier/base/xmpp_client_socket_factory.cc
index 87b439de7aaadf2ccf582ea5d4bddc1184de9ae0..22ae27cb277e712a46455c29d7f3d162082b1ec7 100644
--- a/jingle/notifier/base/xmpp_client_socket_factory.cc
+++ b/jingle/notifier/base/xmpp_client_socket_factory.cc
@@ -30,6 +30,7 @@ XmppClientSocketFactory::~XmppClientSocketFactory() {}
net::StreamSocket* XmppClientSocketFactory::CreateTransportClientSocket(
const net::HostPortPair& host_and_port) {
+ // TODO(akalin): Use socket pools.
net::StreamSocket* transport_socket = new ProxyResolvingClientSocket(
NULL,
request_context_getter_,
« 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