Pass TransportConfig to TransportFactory instead of Transport.

remoting/host/chromoting_host.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 "remoting/host/chromoting_host.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/logging.h"
 #include "base/message_loop_proxy.h"
@@ skipping 89 matching lines @@
     return;
   state_ = kStarted;
 
   // Create port allocator.
   // TODO(sergeyu): Replace the code below with HostPortAllocator when
   // it is implemented.
   scoped_ptr<talk_base::NetworkManager> network_manager(
       new talk_base::BasicNetworkManager());
   scoped_ptr<talk_base::PacketSocketFactory> socket_factory(
       new talk_base::BasicPacketSocketFactory());
-  scoped_ptr<cricket::PortAllocator> port_allocator;
+  scoped_ptr<cricket::HttpPortAllocatorBase> port_allocator(
+      new cricket::HttpPortAllocator(
+          network_manager.get(), socket_factory.get(), ""));
+
 
   // We always use PseudoTcp to provide a reliable channel. However
   // when it is used together with TCP the performance is veryey bad
   // so we explicitly disable TCP connections.
   int port_allocator_flags = cricket::PORTALLOCATOR_DISABLE_TCP;
-  if (network_settings_.nat_traversal_mode ==
+  if (network_settings_.nat_traversal_mode !=
       NetworkSettings::NAT_TRAVERSAL_ENABLED) {
-      port_allocator.reset(new cricket::HttpPortAllocator(
-          network_manager.get(), socket_factory.get(), ""));
-  } else {
     port_allocator_flags |= cricket::PORTALLOCATOR_DISABLE_STUN |
         cricket::PORTALLOCATOR_DISABLE_RELAY;
-    port_allocator.reset(
-        new cricket::BasicPortAllocator(network_manager.get(),
-                                        socket_factory.get()));
   }
   port_allocator->set_flags(port_allocator_flags);
   port_allocator->SetPortRange(network_settings_.min_port,
                                network_settings_.max_port);
 
   bool incoming_only = network_settings_.nat_traversal_mode ==
       NetworkSettings::NAT_TRAVERSAL_DISABLED;
 
   scoped_ptr<protocol::TransportFactory> transport_factory(
       new protocol::LibjingleTransportFactory(
@@ skipping 315 matching lines @@
                     OnShutdown());
 
   for (std::vector<base::Closure>::iterator it = shutdown_tasks_.begin();
        it != shutdown_tasks_.end(); ++it) {
     it->Run();
   }
   shutdown_tasks_.clear();
 }
 
 }  // namespace remoting