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

Unified Diff: components/proximity_auth/cryptauth/cryptauth_enrollment_manager.cc

Issue 2423353002: 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 side-by-side diff with in-line comments
Download patch
Index: components/proximity_auth/cryptauth/cryptauth_enrollment_manager.cc
diff --git a/components/proximity_auth/cryptauth/cryptauth_enrollment_manager.cc b/components/proximity_auth/cryptauth/cryptauth_enrollment_manager.cc
index c2f8093c67694d0d8e45ac3b51a97196c780572a..ca65b47d3916ffaf384683b3635a54dad9762fb1 100644
--- a/components/proximity_auth/cryptauth/cryptauth_enrollment_manager.cc
+++ b/components/proximity_auth/cryptauth/cryptauth_enrollment_manager.cc
@@ -155,7 +155,8 @@ void CryptAuthEnrollmentManager::OnEnrollmentFinished(bool success) {
sync_request_->OnDidComplete(success);
cryptauth_enroller_.reset();
sync_request_.reset();
- FOR_EACH_OBSERVER(Observer, observers_, OnEnrollmentFinished(success));
+ for (auto& observer : observers_)
+ observer.OnEnrollmentFinished(success);
}
std::unique_ptr<SyncScheduler>
@@ -230,7 +231,8 @@ void CryptAuthEnrollmentManager::OnReenrollMessage() {
void CryptAuthEnrollmentManager::OnSyncRequested(
std::unique_ptr<SyncScheduler::SyncRequest> sync_request) {
- FOR_EACH_OBSERVER(Observer, observers_, OnEnrollmentStarted());
+ for (auto& observer : observers_)
+ observer.OnEnrollmentStarted();
sync_request_ = std::move(sync_request);
if (gcm_manager_->GetRegistrationId().empty() ||

Powered by Google App Engine
This is Rietveld 408576698