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

Side by Side Diff: remoting/signaling/fake_signal_strategy.cc

Issue 2425483002: Remove usage of FOR_EACH_OBSERVER macro in remoting (Closed)
Patch Set: Created 4 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
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 #include "remoting/signaling/fake_signal_strategy.h" 5 #include "remoting/signaling/fake_signal_strategy.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/location.h" 10 #include "base/location.h"
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
58 } 58 }
59 } 59 }
60 60
61 void FakeSignalStrategy::SetLocalJid(const std::string& jid) { 61 void FakeSignalStrategy::SetLocalJid(const std::string& jid) {
62 DCHECK(CalledOnValidThread()); 62 DCHECK(CalledOnValidThread());
63 jid_ = jid; 63 jid_ = jid;
64 } 64 }
65 65
66 void FakeSignalStrategy::Connect() { 66 void FakeSignalStrategy::Connect() {
67 DCHECK(CalledOnValidThread()); 67 DCHECK(CalledOnValidThread());
68 FOR_EACH_OBSERVER(Listener, listeners_, 68 for (auto& observer : listeners_)
69 OnSignalStrategyStateChange(CONNECTED)); 69 observer.OnSignalStrategyStateChange(CONNECTED);
70 } 70 }
71 71
72 void FakeSignalStrategy::Disconnect() { 72 void FakeSignalStrategy::Disconnect() {
73 DCHECK(CalledOnValidThread()); 73 DCHECK(CalledOnValidThread());
74 FOR_EACH_OBSERVER(Listener, listeners_, 74 for (auto& observer : listeners_)
75 OnSignalStrategyStateChange(DISCONNECTED)); 75 observer.OnSignalStrategyStateChange(DISCONNECTED);
76 } 76 }
77 77
78 SignalStrategy::State FakeSignalStrategy::GetState() const { 78 SignalStrategy::State FakeSignalStrategy::GetState() const {
79 return CONNECTED; 79 return CONNECTED;
80 } 80 }
81 81
82 SignalStrategy::Error FakeSignalStrategy::GetError() const { 82 SignalStrategy::Error FakeSignalStrategy::GetError() const {
83 return OK; 83 return OK;
84 } 84 }
85 85
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
150 if (listener.OnSignalStrategyIncomingStanza(stanza_ptr)) 150 if (listener.OnSignalStrategyIncomingStanza(stanza_ptr))
151 break; 151 break;
152 } 152 }
153 } 153 }
154 154
155 void FakeSignalStrategy::SetPeerCallback(const PeerCallback& peer_callback) { 155 void FakeSignalStrategy::SetPeerCallback(const PeerCallback& peer_callback) {
156 peer_callback_ = peer_callback; 156 peer_callback_ = peer_callback;
157 } 157 }
158 158
159 } // namespace remoting 159 } // namespace remoting
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698