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

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 style violations 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 is
23 // used by the remoting host. The main difference from 23 // used by the remoting host. The main difference from
Sergey Ulanov 2013/06/21 01:21:57 This comment needs to be updated as this class is
24 // cricket::HttpPortAllocator is that it uses Chromium's HTTP stack 24 // cricket::HttpPortAllocator is that it uses Chromium's HTTP stack
25 // when creating relay sessions. It also configures itself according 25 // when creating relay sessions. It also configures itself according
26 // to the specified NetworkSettings. 26 // to the specified NetworkSettings.
27 class HostPortAllocator : public cricket::HttpPortAllocatorBase { 27 class ChromiumPortAllocator : public cricket::HttpPortAllocatorBase {
28 public: 28 public:
29 static scoped_ptr<HostPortAllocator> Create( 29 static scoped_ptr<ChromiumPortAllocator> Create(
30 const scoped_refptr<net::URLRequestContextGetter>& url_context, 30 const scoped_refptr<net::URLRequestContextGetter>& url_context,
31 const NetworkSettings& network_settings); 31 const NetworkSettings& network_settings);
32 32
33 virtual ~HostPortAllocator(); 33 virtual ~ChromiumPortAllocator();
34 34
35 // cricket::HttpPortAllocatorBase overrides. 35 // cricket::HttpPortAllocatorBase overrides.
36 virtual cricket::PortAllocatorSession* CreateSessionInternal( 36 virtual cricket::PortAllocatorSession* CreateSessionInternal(
37 const std::string& content_name, 37 const std::string& content_name,
38 int component, 38 int component,
39 const std::string& ice_username_fragment, 39 const std::string& ice_username_fragment,
40 const std::string& ice_password) OVERRIDE; 40 const std::string& ice_password) OVERRIDE;
41 41
42 private: 42 private:
43 HostPortAllocator( 43 ChromiumPortAllocator(
44 const scoped_refptr<net::URLRequestContextGetter>& url_context, 44 const scoped_refptr<net::URLRequestContextGetter>& url_context,
45 scoped_ptr<talk_base::NetworkManager> network_manager, 45 scoped_ptr<talk_base::NetworkManager> network_manager,
46 scoped_ptr<talk_base::PacketSocketFactory> socket_factory); 46 scoped_ptr<talk_base::PacketSocketFactory> socket_factory);
47 47
48 scoped_refptr<net::URLRequestContextGetter> url_context_; 48 scoped_refptr<net::URLRequestContextGetter> url_context_;
49 scoped_ptr<talk_base::NetworkManager> network_manager_; 49 scoped_ptr<talk_base::NetworkManager> network_manager_;
50 scoped_ptr<talk_base::PacketSocketFactory> socket_factory_; 50 scoped_ptr<talk_base::PacketSocketFactory> socket_factory_;
51 51
52 DISALLOW_COPY_AND_ASSIGN(HostPortAllocator); 52 DISALLOW_COPY_AND_ASSIGN(ChromiumPortAllocator);
53 }; 53 };
54 54
55 } // namespace remoting 55 } // namespace remoting
56 56
57 #endif // REMOTING_HOST_HOST_PORT_ALLOCATOR_H_ 57 #endif // REMOTING_HOST_HOST_PORT_ALLOCATOR_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698