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

Side by Side Diff: remoting/signaling/delegating_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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/delegating_signal_strategy.h" 5 #include "remoting/signaling/delegating_signal_strategy.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/rand_util.h" 8 #include "base/rand_util.h"
9 #include "base/strings/string_number_conversions.h" 9 #include "base/strings/string_number_conversions.h"
10 #include "third_party/webrtc/libjingle/xmllite/xmlelement.h" 10 #include "third_party/webrtc/libjingle/xmllite/xmlelement.h"
(...skipping 28 matching lines...) Expand all
39 } 39 }
40 40
41 for (auto& listener : listeners_) { 41 for (auto& listener : listeners_) {
42 if (listener.OnSignalStrategyIncomingStanza(stanza.get())) 42 if (listener.OnSignalStrategyIncomingStanza(stanza.get()))
43 break; 43 break;
44 } 44 }
45 } 45 }
46 46
47 void DelegatingSignalStrategy::Connect() { 47 void DelegatingSignalStrategy::Connect() {
48 DCHECK(client_task_runner_->BelongsToCurrentThread()); 48 DCHECK(client_task_runner_->BelongsToCurrentThread());
49 FOR_EACH_OBSERVER(Listener, listeners_, 49 for (auto& observer : listeners_)
50 OnSignalStrategyStateChange(CONNECTED)); 50 observer.OnSignalStrategyStateChange(CONNECTED);
51 } 51 }
52 52
53 void DelegatingSignalStrategy::Disconnect() { 53 void DelegatingSignalStrategy::Disconnect() {
54 DCHECK(client_task_runner_->BelongsToCurrentThread()); 54 DCHECK(client_task_runner_->BelongsToCurrentThread());
55 } 55 }
56 56
57 SignalStrategy::State DelegatingSignalStrategy::GetState() const { 57 SignalStrategy::State DelegatingSignalStrategy::GetState() const {
58 DCHECK(client_task_runner_->BelongsToCurrentThread()); 58 DCHECK(client_task_runner_->BelongsToCurrentThread());
59 return CONNECTED; 59 return CONNECTED;
60 } 60 }
(...skipping 25 matching lines...) Expand all
86 delegate_task_runner_->PostTask(FROM_HERE, 86 delegate_task_runner_->PostTask(FROM_HERE,
87 base::Bind(send_iq_callback_, stanza->Str())); 87 base::Bind(send_iq_callback_, stanza->Str()));
88 return true; 88 return true;
89 } 89 }
90 90
91 std::string DelegatingSignalStrategy::GetNextId() { 91 std::string DelegatingSignalStrategy::GetNextId() {
92 return base::Uint64ToString(base::RandUint64()); 92 return base::Uint64ToString(base::RandUint64());
93 } 93 }
94 94
95 } // namespace remoting 95 } // namespace remoting
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698