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

Unified Diff: remoting/protocol/libjingle_transport_factory.h

Issue 10160013: Implement HostPortAllocator. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 8 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 | « remoting/host/url_fetcher.cc ('k') | remoting/protocol/libjingle_transport_factory.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/protocol/libjingle_transport_factory.h
diff --git a/remoting/protocol/libjingle_transport_factory.h b/remoting/protocol/libjingle_transport_factory.h
index fb47b0037fb7008c024c01edcc0be64a34078485..0cead6d26a5f31bcbe1c445956cc3f5d4d6e3a58 100644
--- a/remoting/protocol/libjingle_transport_factory.h
+++ b/remoting/protocol/libjingle_transport_factory.h
@@ -22,9 +22,11 @@ namespace protocol {
class LibjingleTransportFactory : public TransportFactory {
public:
+ // Need to use cricket::HttpPortAllocatorBase pointer for the
+ // |port_allocator|, so that it is possible to configure
+ // |port_allocator| with STUN/Relay addresses.
+ // TODO(sergeyu): Reconsider this design.
LibjingleTransportFactory(
- scoped_ptr<talk_base::NetworkManager> network_manager,
- scoped_ptr<talk_base::PacketSocketFactory> socket_factory,
scoped_ptr<cricket::HttpPortAllocatorBase> port_allocator,
bool incoming_only);
« no previous file with comments | « remoting/host/url_fetcher.cc ('k') | remoting/protocol/libjingle_transport_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698