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

Side by Side Diff: remoting/protocol/jingle_session_manager.h

Issue 11419224: Add missing (and remove superfluous) 'explicit' from constructors. Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase + remove non-straightforward changes Created 7 years, 11 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 | Annotate | Revision Log
« no previous file with comments | « remoting/protocol/jingle_messages.h ('k') | remoting/protocol/negotiating_authenticator.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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_PROTOCOL_JINGLE_SESSION_MANAGER_H_ 5 #ifndef REMOTING_PROTOCOL_JINGLE_SESSION_MANAGER_H_
6 #define REMOTING_PROTOCOL_JINGLE_SESSION_MANAGER_H_ 6 #define REMOTING_PROTOCOL_JINGLE_SESSION_MANAGER_H_
7 7
8 #include <map> 8 #include <map>
9 #include <list> 9 #include <list>
10 #include <string> 10 #include <string>
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
43 // creating outgoing sessions. 43 // creating outgoing sessions.
44 class JingleSessionManager : public SessionManager, 44 class JingleSessionManager : public SessionManager,
45 public SignalStrategy::Listener { 45 public SignalStrategy::Listener {
46 public: 46 public:
47 // When |fetch_stun_relay_config| is set to true then 47 // When |fetch_stun_relay_config| is set to true then
48 // JingleSessionManager will also try to query configuration of STUN 48 // JingleSessionManager will also try to query configuration of STUN
49 // and Relay servers from the signaling server. 49 // and Relay servers from the signaling server.
50 // 50 //
51 // TODO(sergeyu): Move NAT-traversal config fetching to a separate 51 // TODO(sergeyu): Move NAT-traversal config fetching to a separate
52 // class. 52 // class.
53 explicit JingleSessionManager( 53 JingleSessionManager(
54 scoped_ptr<TransportFactory> transport_factory, 54 scoped_ptr<TransportFactory> transport_factory,
55 bool fetch_stun_relay_config); 55 bool fetch_stun_relay_config);
56 virtual ~JingleSessionManager(); 56 virtual ~JingleSessionManager();
57 57
58 // SessionManager interface. 58 // SessionManager interface.
59 virtual void Init(SignalStrategy* signal_strategy, 59 virtual void Init(SignalStrategy* signal_strategy,
60 SessionManager::Listener* listener) OVERRIDE; 60 SessionManager::Listener* listener) OVERRIDE;
61 virtual scoped_ptr<Session> Connect( 61 virtual scoped_ptr<Session> Connect(
62 const std::string& host_jid, 62 const std::string& host_jid,
63 scoped_ptr<Authenticator> authenticator, 63 scoped_ptr<Authenticator> authenticator,
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
103 103
104 SessionsMap sessions_; 104 SessionsMap sessions_;
105 105
106 DISALLOW_COPY_AND_ASSIGN(JingleSessionManager); 106 DISALLOW_COPY_AND_ASSIGN(JingleSessionManager);
107 }; 107 };
108 108
109 } // namespace protocol 109 } // namespace protocol
110 } // namespace remoting 110 } // namespace remoting
111 111
112 #endif // REMOTING_PROTOCOL_JINGLE_SESSION_MANAGER_H_ 112 #endif // REMOTING_PROTOCOL_JINGLE_SESSION_MANAGER_H_
OLDNEW
« no previous file with comments | « remoting/protocol/jingle_messages.h ('k') | remoting/protocol/negotiating_authenticator.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698