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

Unified Diff: chrome/browser/sync_file_system/local/local_file_sync_service.cc

Issue 2425553002: Remove FOR_EACH_OBSERVER macro usage in chrome/browser/sync_file_system (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: chrome/browser/sync_file_system/local/local_file_sync_service.cc
diff --git a/chrome/browser/sync_file_system/local/local_file_sync_service.cc b/chrome/browser/sync_file_system/local/local_file_sync_service.cc
index cc6dad428a41d23f9a6b83a4de0de30a33925e42..d1478919c54620ba77cec2d25b6aa6a436277651 100644
--- a/chrome/browser/sync_file_system/local/local_file_sync_service.cc
+++ b/chrome/browser/sync_file_system/local/local_file_sync_service.cc
@@ -329,8 +329,8 @@ void LocalFileSyncService::OnChangesAvailableInOrigins(
if (!need_notification)
return;
int64_t num_changes = origin_change_map_.GetTotalChangeCount();
- FOR_EACH_OBSERVER(Observer, change_observers_,
- OnLocalChangeAvailable(num_changes));
+ for (auto& observer : change_observers_)
+ observer.OnLocalChangeAvailable(num_changes);
}
void LocalFileSyncService::SetOriginEnabled(const GURL& origin, bool enabled) {
@@ -375,8 +375,8 @@ void LocalFileSyncService::DidInitializeFileSystemContext(
origin_change_map_.SetOriginChangeCount(
app_origin, backend->change_tracker()->num_changes());
int64_t num_changes = origin_change_map_.GetTotalChangeCount();
- FOR_EACH_OBSERVER(Observer, change_observers_,
- OnLocalChangeAvailable(num_changes));
+ for (auto& observer : change_observers_)
+ observer.OnLocalChangeAvailable(num_changes);
}
callback.Run(status);
}

Powered by Google App Engine
This is Rietveld 408576698