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

Unified Diff: remoting/signaling/fake_signal_strategy.h

Issue 2798393007: Use SignalingAddress in SignalStrategy insterface. (Closed)
Patch Set: header Created 3 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/signaling/delegating_signal_strategy.cc ('k') | remoting/signaling/fake_signal_strategy.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/signaling/fake_signal_strategy.h
diff --git a/remoting/signaling/fake_signal_strategy.h b/remoting/signaling/fake_signal_strategy.h
index 0b5d912bd4d35a6df44cc8ecf75bd27b7ecb5165..c4da7e472a5f8ebee135aae8ae4fc1000626c583 100644
--- a/remoting/signaling/fake_signal_strategy.h
+++ b/remoting/signaling/fake_signal_strategy.h
@@ -15,6 +15,7 @@
#include "base/threading/non_thread_safe.h"
#include "remoting/signaling/iq_sender.h"
#include "remoting/signaling/signal_strategy.h"
+#include "remoting/signaling/signaling_address.h"
namespace base {
class SingleThreadTaskRunner;
@@ -29,7 +30,7 @@ class FakeSignalStrategy : public SignalStrategy,
// must belong to the current thread.
static void Connect(FakeSignalStrategy* peer1, FakeSignalStrategy* peer2);
- FakeSignalStrategy(const std::string& jid);
+ FakeSignalStrategy(const SignalingAddress& address);
~FakeSignalStrategy() override;
const std::list<buzz::XmlElement*>& received_messages() {
@@ -43,7 +44,7 @@ class FakeSignalStrategy : public SignalStrategy,
// Connects current FakeSignalStrategy to receive messages from |peer|.
void ConnectTo(FakeSignalStrategy* peer);
- void SetLocalJid(const std::string& jid);
+ void SetLocalAddress(const SignalingAddress& address);
// Simulate IQ messages re-ordering by swapping the delivery order of
// next pair of messages.
@@ -54,7 +55,7 @@ class FakeSignalStrategy : public SignalStrategy,
void Disconnect() override;
State GetState() const override;
Error GetError() const override;
- std::string GetLocalJid() const override;
+ const SignalingAddress& GetLocalAddress() const override;
void AddListener(Listener* listener) override;
void RemoveListener(Listener* listener) override;
bool SendStanza(std::unique_ptr<buzz::XmlElement> stanza) override;
@@ -76,7 +77,7 @@ class FakeSignalStrategy : public SignalStrategy,
scoped_refptr<base::SingleThreadTaskRunner> main_thread_;
- std::string jid_;
+ SignalingAddress address_;
PeerCallback peer_callback_;
base::ObserverList<Listener, true> listeners_;
« no previous file with comments | « remoting/signaling/delegating_signal_strategy.cc ('k') | remoting/signaling/fake_signal_strategy.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698