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

Side by Side Diff: components/pairing/fake_controller_pairing_controller.cc

Issue 2444753002: Reduce usage of FOR_EACH_OBSERVER macro in components/ (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 "components/pairing/fake_controller_pairing_controller.h" 5 #include "components/pairing/fake_controller_pairing_controller.h"
6 6
7 #include <map> 7 #include <map>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/location.h" 10 #include "base/location.h"
(...skipping 215 matching lines...) Expand 10 before | Expand all | Expand 10 after
226 226
227 void FakeControllerPairingController::StartSession() { 227 void FakeControllerPairingController::StartSession() {
228 CHECK(current_stage_ == STAGE_HOST_ENROLLMENT_SUCCESS); 228 CHECK(current_stage_ == STAGE_HOST_ENROLLMENT_SUCCESS);
229 ChangeStage(STAGE_FINISHED); 229 ChangeStage(STAGE_FINISHED);
230 } 230 }
231 231
232 void FakeControllerPairingController::ChangeStage(Stage new_stage) { 232 void FakeControllerPairingController::ChangeStage(Stage new_stage) {
233 if (current_stage_ == new_stage) 233 if (current_stage_ == new_stage)
234 return; 234 return;
235 current_stage_ = new_stage; 235 current_stage_ = new_stage;
236 FOR_EACH_OBSERVER(Observer, observers_, PairingStageChanged(new_stage)); 236 for (Observer& observer : observers_)
237 observer.PairingStageChanged(new_stage);
237 } 238 }
238 239
239 void FakeControllerPairingController::ChangeStageLater(Stage new_stage) { 240 void FakeControllerPairingController::ChangeStageLater(Stage new_stage) {
240 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask( 241 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
241 FROM_HERE, base::Bind(&FakeControllerPairingController::ChangeStage, 242 FROM_HERE, base::Bind(&FakeControllerPairingController::ChangeStage,
242 base::Unretained(this), new_stage), 243 base::Unretained(this), new_stage),
243 async_duration_); 244 async_duration_);
244 } 245 }
245 246
246 void FakeControllerPairingController::ExecuteDiscoveryEvent( 247 void FakeControllerPairingController::ExecuteDiscoveryEvent(
(...skipping 23 matching lines...) Expand all
270 FROM_HERE, 271 FROM_HERE,
271 base::Bind(&FakeControllerPairingController::ExecuteDiscoveryEvent, 272 base::Bind(&FakeControllerPairingController::ExecuteDiscoveryEvent,
272 base::Unretained(this), event_position), 273 base::Unretained(this), event_position),
273 async_duration_); 274 async_duration_);
274 } 275 }
275 276
276 void FakeControllerPairingController::DeviceFound( 277 void FakeControllerPairingController::DeviceFound(
277 const std::string& device_id) { 278 const std::string& device_id) {
278 CHECK(current_stage_ == STAGE_DEVICES_DISCOVERY); 279 CHECK(current_stage_ == STAGE_DEVICES_DISCOVERY);
279 discovered_devices_.insert(device_id); 280 discovered_devices_.insert(device_id);
280 FOR_EACH_OBSERVER(Observer, observers_, DiscoveredDevicesListChanged()); 281 for (Observer& observer : observers_)
282 observer.DiscoveredDevicesListChanged();
281 } 283 }
282 284
283 void FakeControllerPairingController::DeviceLost(const std::string& device_id) { 285 void FakeControllerPairingController::DeviceLost(const std::string& device_id) {
284 CHECK(current_stage_ == STAGE_DEVICES_DISCOVERY); 286 CHECK(current_stage_ == STAGE_DEVICES_DISCOVERY);
285 discovered_devices_.erase(device_id); 287 discovered_devices_.erase(device_id);
286 FOR_EACH_OBSERVER(Observer, observers_, DiscoveredDevicesListChanged()); 288 for (Observer& observer : observers_)
289 observer.DiscoveredDevicesListChanged();
287 } 290 }
288 291
289 void FakeControllerPairingController::PairingStageChanged(Stage new_stage) { 292 void FakeControllerPairingController::PairingStageChanged(Stage new_stage) {
290 Stage next_stage = STAGE_NONE; 293 Stage next_stage = STAGE_NONE;
291 switch (new_stage) { 294 switch (new_stage) {
292 case STAGE_DEVICES_DISCOVERY: { 295 case STAGE_DEVICES_DISCOVERY: {
293 discovered_devices_.clear(); 296 discovered_devices_.clear();
294 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask( 297 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
295 FROM_HERE, 298 FROM_HERE,
296 base::Bind(&FakeControllerPairingController::ExecuteDiscoveryEvent, 299 base::Bind(&FakeControllerPairingController::ExecuteDiscoveryEvent,
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
334 next_stage = STAGE_HOST_CONNECTION_LOST; 337 next_stage = STAGE_HOST_CONNECTION_LOST;
335 } 338 }
336 if (next_stage != STAGE_NONE) 339 if (next_stage != STAGE_NONE)
337 ChangeStageLater(next_stage); 340 ChangeStageLater(next_stage);
338 } 341 }
339 342
340 void FakeControllerPairingController::DiscoveredDevicesListChanged() { 343 void FakeControllerPairingController::DiscoveredDevicesListChanged() {
341 } 344 }
342 345
343 } // namespace pairing_chromeos 346 } // namespace pairing_chromeos
OLDNEW
« no previous file with comments | « components/pairing/bluetooth_host_pairing_controller.cc ('k') | components/pairing/fake_host_pairing_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698