From 67c74d5e81e1a0a634fea79cfc5d5975c33d0e17 Mon Sep 17 00:00:00 2001 From: "sergeyu@chromium.org" Date: Fri, 29 Jun 2012 02:16:21 +0000 Subject: Roll libjingle 153:157 Among other things this roll fixes relay server support (bug 134007). BUG=134007 Review URL: https://chromiumcodereview.appspot.com/10705008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144860 0039d316-1c4b-4281-b951-d872f2087c98 --- remoting/client/plugin/pepper_port_allocator.cc | 7 ++++++- remoting/host/host_port_allocator.cc | 6 +++++- 2 files changed, 11 insertions(+), 2 deletions(-) (limited to 'remoting') diff --git a/remoting/client/plugin/pepper_port_allocator.cc b/remoting/client/plugin/pepper_port_allocator.cc index ae0a3a5..bf8ac98 100644 --- a/remoting/client/plugin/pepper_port_allocator.cc +++ b/remoting/client/plugin/pepper_port_allocator.cc @@ -88,7 +88,6 @@ PepperPortAllocatorSession::PepperPortAllocatorSession( stun_address_resolver_(instance_), stun_port_(0), relay_response_received_(false) { - set_flags(cricket::PORTALLOCATOR_DISABLE_TCP); if (stun_hosts.size() > 0) { stun_address_ = stun_hosts[0]; } @@ -325,6 +324,12 @@ PepperPortAllocator::PepperPortAllocator( instance_(instance), network_manager_(network_manager.Pass()), socket_factory_(socket_factory.Pass()) { + // TCP transport is disabled becase PseudoTCP works poorly over + // it. ENABLE_SHARED_UFRAG flag is specified so that the same + // username fragment is shared between all candidates for this + // channel. + set_flags(cricket::PORTALLOCATOR_DISABLE_TCP | + cricket::PORTALLOCATOR_ENABLE_SHARED_UFRAG); } PepperPortAllocator::~PepperPortAllocator() { diff --git a/remoting/host/host_port_allocator.cc b/remoting/host/host_port_allocator.cc index bf7fd18..d9323d1 100644 --- a/remoting/host/host_port_allocator.cc +++ b/remoting/host/host_port_allocator.cc @@ -134,7 +134,11 @@ scoped_ptr HostPortAllocator::Create( // We always use PseudoTcp to provide a reliable channel. It // provides poor performance when combined with TCP-based transport, // so we have to disable TCP ports. - int flags = cricket::PORTALLOCATOR_DISABLE_TCP; + // ENABLE_SHARED_UFRAG flag is + // specified so that the same username fragment is shared between + // all candidates for this channel. + int flags = cricket::PORTALLOCATOR_DISABLE_TCP | + cricket::PORTALLOCATOR_ENABLE_SHARED_UFRAG; if (network_settings.nat_traversal_mode != NetworkSettings::NAT_TRAVERSAL_ENABLED) { flags |= cricket::PORTALLOCATOR_DISABLE_STUN | -- cgit v1.1