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

Side by Side Diff: remoting/signaling/xmpp_signal_strategy.h

Issue 667123002: Standardize usage of virtual/override/final in remoting/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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
« no previous file with comments | « remoting/signaling/log_to_server.h ('k') | remoting/test/fake_network_manager.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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 // The XmppSignalStrategy encapsulates all the logic to perform the signaling 5 // The XmppSignalStrategy encapsulates all the logic to perform the signaling
6 // STUN/ICE for jingle via a direct XMPP connection. 6 // STUN/ICE for jingle via a direct XMPP connection.
7 // 7 //
8 // This class is not threadsafe. 8 // This class is not threadsafe.
9 9
10 #ifndef REMOTING_SIGNALING_XMPP_SIGNAL_STRATEGY_H_ 10 #ifndef REMOTING_SIGNALING_XMPP_SIGNAL_STRATEGY_H_
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 50
51 std::string username; 51 std::string username;
52 std::string auth_token; 52 std::string auth_token;
53 std::string auth_service; 53 std::string auth_service;
54 }; 54 };
55 55
56 XmppSignalStrategy( 56 XmppSignalStrategy(
57 net::ClientSocketFactory* socket_factory, 57 net::ClientSocketFactory* socket_factory,
58 scoped_refptr<net::URLRequestContextGetter> request_context_getter, 58 scoped_refptr<net::URLRequestContextGetter> request_context_getter,
59 const XmppServerConfig& xmpp_server_config); 59 const XmppServerConfig& xmpp_server_config);
60 virtual ~XmppSignalStrategy(); 60 ~XmppSignalStrategy() override;
61 61
62 // SignalStrategy interface. 62 // SignalStrategy interface.
63 virtual void Connect() override; 63 void Connect() override;
64 virtual void Disconnect() override; 64 void Disconnect() override;
65 virtual State GetState() const override; 65 State GetState() const override;
66 virtual Error GetError() const override; 66 Error GetError() const override;
67 virtual std::string GetLocalJid() const override; 67 std::string GetLocalJid() const override;
68 virtual void AddListener(Listener* listener) override; 68 void AddListener(Listener* listener) override;
69 virtual void RemoveListener(Listener* listener) override; 69 void RemoveListener(Listener* listener) override;
70 virtual bool SendStanza(scoped_ptr<buzz::XmlElement> stanza) override; 70 bool SendStanza(scoped_ptr<buzz::XmlElement> stanza) override;
71 virtual std::string GetNextId() override; 71 std::string GetNextId() override;
72 72
73 // buzz::XmppStanzaHandler interface. 73 // buzz::XmppStanzaHandler interface.
74 virtual bool HandleStanza(const buzz::XmlElement* stanza) override; 74 bool HandleStanza(const buzz::XmlElement* stanza) override;
75 75
76 // This method is used to update the auth info (for example when the OAuth 76 // This method is used to update the auth info (for example when the OAuth
77 // access token is renewed). It is OK to call this even when we are in the 77 // access token is renewed). It is OK to call this even when we are in the
78 // CONNECTED state. It will be used on the next Connect() call. 78 // CONNECTED state. It will be used on the next Connect() call.
79 void SetAuthInfo(const std::string& username, 79 void SetAuthInfo(const std::string& username,
80 const std::string& auth_token, 80 const std::string& auth_token,
81 const std::string& auth_service); 81 const std::string& auth_service);
82 82
83 // Use this method to override the default resource name used (optional). 83 // Use this method to override the default resource name used (optional).
84 // This will be used on the next Connect() call. 84 // This will be used on the next Connect() call.
(...skipping 21 matching lines...) Expand all
106 ObserverList<Listener, true> listeners_; 106 ObserverList<Listener, true> listeners_;
107 107
108 base::RepeatingTimer<XmppSignalStrategy> keep_alive_timer_; 108 base::RepeatingTimer<XmppSignalStrategy> keep_alive_timer_;
109 109
110 DISALLOW_COPY_AND_ASSIGN(XmppSignalStrategy); 110 DISALLOW_COPY_AND_ASSIGN(XmppSignalStrategy);
111 }; 111 };
112 112
113 } // namespace remoting 113 } // namespace remoting
114 114
115 #endif // REMOTING_SIGNALING_XMPP_SIGNAL_STRATEGY_H_ 115 #endif // REMOTING_SIGNALING_XMPP_SIGNAL_STRATEGY_H_
OLDNEW
« no previous file with comments | « remoting/signaling/log_to_server.h ('k') | remoting/test/fake_network_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698