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

Side by Side Diff: remoting/jingle_glue/chromium_port_allocator.h

Issue 17101034: Add static Create method to LibjingleTransportFactory (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed erroneous comments Created 7 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef REMOTING_HOST_HOST_PORT_ALLOCATOR_H_ 5 #ifndef REMOTING_JINGLE_GLUE_CHROMIUM_PORT_ALLOCATOR_H_
6 #define REMOTING_HOST_HOST_PORT_ALLOCATOR_H_ 6 #define REMOTING_JINGLE_GLUE_CHROMIUM_PORT_ALLOCATOR_H_
7 7
8 #include <set> 8 #include <set>
9 9
10 #include "base/memory/ref_counted.h" 10 #include "base/memory/ref_counted.h"
11 #include "base/memory/scoped_ptr.h" 11 #include "base/memory/scoped_ptr.h"
12 #include "third_party/libjingle/source/talk/p2p/client/httpportallocator.h" 12 #include "third_party/libjingle/source/talk/p2p/client/httpportallocator.h"
13 13
14 namespace net { 14 namespace net {
15 class URLRequestContextGetter; 15 class URLRequestContextGetter;
16 } // namespace net 16 } // namespace net
17 17
18 namespace remoting { 18 namespace remoting {
19 19
20 struct NetworkSettings; 20 struct NetworkSettings;
21 21
22 // An implementation of cricket::PortAllocator for libjingle that is 22 // An implementation of cricket::PortAllocator for libjingle that
23 // used by the remoting host. The main difference from 23 // uses Chromium's network stack and configures itself according to
24 // cricket::HttpPortAllocator is that it uses Chromium's HTTP stack
25 // when creating relay sessions. It also configures itself according 24 // when creating relay sessions. It also configures itself according
Sergey Ulanov 2013/06/21 02:05:28 I think you want to remove this line and 'to' from
26 // to the specified NetworkSettings. 25 // to the specified NetworkSettings.
27 class HostPortAllocator : public cricket::HttpPortAllocatorBase { 26 class ChromiumPortAllocator : public cricket::HttpPortAllocatorBase {
28 public: 27 public:
29 static scoped_ptr<HostPortAllocator> Create( 28 static scoped_ptr<ChromiumPortAllocator> Create(
30 const scoped_refptr<net::URLRequestContextGetter>& url_context, 29 const scoped_refptr<net::URLRequestContextGetter>& url_context,
31 const NetworkSettings& network_settings); 30 const NetworkSettings& network_settings);
32 31
33 virtual ~HostPortAllocator(); 32 virtual ~ChromiumPortAllocator();
34 33
35 // cricket::HttpPortAllocatorBase overrides. 34 // cricket::HttpPortAllocatorBase overrides.
36 virtual cricket::PortAllocatorSession* CreateSessionInternal( 35 virtual cricket::PortAllocatorSession* CreateSessionInternal(
37 const std::string& content_name, 36 const std::string& content_name,
38 int component, 37 int component,
39 const std::string& ice_username_fragment, 38 const std::string& ice_username_fragment,
40 const std::string& ice_password) OVERRIDE; 39 const std::string& ice_password) OVERRIDE;
41 40
42 private: 41 private:
43 HostPortAllocator( 42 ChromiumPortAllocator(
44 const scoped_refptr<net::URLRequestContextGetter>& url_context, 43 const scoped_refptr<net::URLRequestContextGetter>& url_context,
45 scoped_ptr<talk_base::NetworkManager> network_manager, 44 scoped_ptr<talk_base::NetworkManager> network_manager,
46 scoped_ptr<talk_base::PacketSocketFactory> socket_factory); 45 scoped_ptr<talk_base::PacketSocketFactory> socket_factory);
47 46
48 scoped_refptr<net::URLRequestContextGetter> url_context_; 47 scoped_refptr<net::URLRequestContextGetter> url_context_;
49 scoped_ptr<talk_base::NetworkManager> network_manager_; 48 scoped_ptr<talk_base::NetworkManager> network_manager_;
50 scoped_ptr<talk_base::PacketSocketFactory> socket_factory_; 49 scoped_ptr<talk_base::PacketSocketFactory> socket_factory_;
51 50
52 DISALLOW_COPY_AND_ASSIGN(HostPortAllocator); 51 DISALLOW_COPY_AND_ASSIGN(ChromiumPortAllocator);
53 }; 52 };
54 53
55 } // namespace remoting 54 } // namespace remoting
56 55
57 #endif // REMOTING_HOST_HOST_PORT_ALLOCATOR_H_ 56 #endif // REMOTING_HOST_HOST_PORT_ALLOCATOR_H_
OLDNEW
« no previous file with comments | « remoting/host/session_manager_factory.cc ('k') | remoting/jingle_glue/chromium_port_allocator.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698