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

Side by Side Diff: chromeos/dbus/fake_power_manager_client.cc

Issue 2413963002: Replace FOR_EACH_OBSERVER in chromeos/ with range-based for (Closed)
Patch Set: Run the script again with '/mg' 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
« no previous file with comments | « chromeos/dbus/fake_cras_audio_client.cc ('k') | chromeos/dbus/fake_session_manager_client.cc » ('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) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 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 "chromeos/dbus/fake_power_manager_client.h" 5 #include "chromeos/dbus/fake_power_manager_client.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/bind_helpers.h" 8 #include "base/bind_helpers.h"
9 #include "base/location.h" 9 #include "base/location.h"
10 #include "base/threading/thread_task_runner_handle.h" 10 #include "base/threading/thread_task_runner_handle.h"
(...skipping 135 matching lines...) Expand 10 before | Expand all | Expand 10 after
146 } 146 }
147 147
148 bool FakePowerManagerClient::PopVideoActivityReport() { 148 bool FakePowerManagerClient::PopVideoActivityReport() {
149 CHECK(!video_activity_reports_.empty()); 149 CHECK(!video_activity_reports_.empty());
150 bool fullscreen = video_activity_reports_.front(); 150 bool fullscreen = video_activity_reports_.front();
151 video_activity_reports_.pop_front(); 151 video_activity_reports_.pop_front();
152 return fullscreen; 152 return fullscreen;
153 } 153 }
154 154
155 void FakePowerManagerClient::SendSuspendImminent() { 155 void FakePowerManagerClient::SendSuspendImminent() {
156 FOR_EACH_OBSERVER(Observer, observers_, SuspendImminent()); 156 for (auto& observer : observers_)
157 observer.SuspendImminent();
157 if (render_process_manager_delegate_) 158 if (render_process_manager_delegate_)
158 render_process_manager_delegate_->SuspendImminent(); 159 render_process_manager_delegate_->SuspendImminent();
159 } 160 }
160 161
161 void FakePowerManagerClient::SendSuspendDone() { 162 void FakePowerManagerClient::SendSuspendDone() {
162 if (render_process_manager_delegate_) 163 if (render_process_manager_delegate_)
163 render_process_manager_delegate_->SuspendDone(); 164 render_process_manager_delegate_->SuspendDone();
164 165
165 FOR_EACH_OBSERVER(Observer, observers_, SuspendDone(base::TimeDelta())); 166 for (auto& observer : observers_)
167 observer.SuspendDone(base::TimeDelta());
166 } 168 }
167 169
168 void FakePowerManagerClient::SendDarkSuspendImminent() { 170 void FakePowerManagerClient::SendDarkSuspendImminent() {
169 FOR_EACH_OBSERVER(Observer, observers_, DarkSuspendImminent()); 171 for (auto& observer : observers_)
172 observer.DarkSuspendImminent();
170 } 173 }
171 174
172 void FakePowerManagerClient::SendPowerButtonEvent( 175 void FakePowerManagerClient::SendPowerButtonEvent(
173 bool down, 176 bool down,
174 const base::TimeTicks& timestamp) { 177 const base::TimeTicks& timestamp) {
175 FOR_EACH_OBSERVER(Observer, observers_, 178 for (auto& observer : observers_)
176 PowerButtonEventReceived(down, timestamp)); 179 observer.PowerButtonEventReceived(down, timestamp);
177 } 180 }
178 181
179 void FakePowerManagerClient::UpdatePowerProperties( 182 void FakePowerManagerClient::UpdatePowerProperties(
180 const power_manager::PowerSupplyProperties& power_props) { 183 const power_manager::PowerSupplyProperties& power_props) {
181 props_ = power_props; 184 props_ = power_props;
182 NotifyObservers(); 185 NotifyObservers();
183 } 186 }
184 187
185 void FakePowerManagerClient::NotifyObservers() { 188 void FakePowerManagerClient::NotifyObservers() {
186 FOR_EACH_OBSERVER(Observer, observers_, PowerChanged(props_)); 189 for (auto& observer : observers_)
190 observer.PowerChanged(props_);
187 } 191 }
188 192
189 void FakePowerManagerClient::HandleSuspendReadiness() { 193 void FakePowerManagerClient::HandleSuspendReadiness() {
190 CHECK(num_pending_suspend_readiness_callbacks_ > 0); 194 CHECK(num_pending_suspend_readiness_callbacks_ > 0);
191 195
192 --num_pending_suspend_readiness_callbacks_; 196 --num_pending_suspend_readiness_callbacks_;
193 } 197 }
194 198
195 } // namespace chromeos 199 } // namespace chromeos
OLDNEW
« no previous file with comments | « chromeos/dbus/fake_cras_audio_client.cc ('k') | chromeos/dbus/fake_session_manager_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698