Fix the hostname used to setup a TURN/TSL connection.

content/renderer/p2p/ipc_socket_factory.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/p2p/ipc_socket_factory.h"
 
 #include <algorithm>
 #include <deque>
 
 #include "base/compiler_specific.h"
@@ skipping 226 matching lines @@
   net::IPEndPoint remote_endpoint;
   if (!remote_address.IsNil() && !jingle_glue::SocketAddressToIPEndPoint(
       remote_address, &remote_endpoint) && !IsTcpClientSocket(type_)) {
     // Non TCP sockets must have a resolved remote address.
     return false;
   }
 
   // We need to send both resolved and unresolved address in Init. Unresolved
   // address will be used in case of TLS for certificate hostname matching.
   // Certificate will be tied to domain name not to IP address.
-  std::string remote_hostname = remote_address.hostname() + ":" +
-      remote_address.PortAsString();
-  P2PHostAndIPEndPoint remote_info(remote_hostname, remote_endpoint);
+  P2PHostAndIPEndPoint remote_info(remote_address.hostname(), remote_endpoint);

[Mallinath (Gone from Chromium), 2014/08/12 21:22:07]

are you sure remote_endpoint will have port, if remote_address has unresolved
address?

[jiayl, 2014/08/12 21:39:51]

Haha, I thought so but I was wrong. Added the code to set the port in that case.

 
   client->Init(type, local_endpoint, remote_info, this);
 
   return true;
 }
 
 void IpcPacketSocket::InitAcceptedTcp(
     P2PSocketClient* client,
     const rtc::SocketAddress& local_address,
     const rtc::SocketAddress& remote_address) {
@@ skipping 404 matching lines @@
 }
 
 rtc::AsyncResolverInterface*
 IpcPacketSocketFactory::CreateAsyncResolver() {
   scoped_ptr<AsyncAddressResolverImpl> resolver(
     new AsyncAddressResolverImpl(socket_dispatcher_));
   return resolver.release();
 }
 
 }  // namespace content