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

Unified Diff: chrome/browser/sync_file_system/sync_file_system_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/sync_file_system_service.cc
diff --git a/chrome/browser/sync_file_system/sync_file_system_service.cc b/chrome/browser/sync_file_system/sync_file_system_service.cc
index bf8d6e4e2e1071af07739d302816e3f57192a84c..d9666744e376f6e72aa20c55ad2a0a83f8bdb7b0 100644
--- a/chrome/browser/sync_file_system/sync_file_system_service.cc
+++ b/chrome/browser/sync_file_system/sync_file_system_service.cc
@@ -646,11 +646,10 @@ void SyncFileSystemService::OnRemoteServiceStateUpdated(
util::Log(logging::LOG_VERBOSE, FROM_HERE,
"OnRemoteServiceStateChanged: %d %s", state, description.c_str());
- FOR_EACH_OBSERVER(
- SyncEventObserver, observers_,
- OnSyncStateUpdated(GURL(),
- RemoteStateToSyncServiceState(state),
- description));
+ for (auto& observer : observers_) {
+ observer.OnSyncStateUpdated(GURL(), RemoteStateToSyncServiceState(state),
+ description);
+ }
RunForEachSyncRunners(&SyncProcessRunner::Schedule);
}
@@ -742,9 +741,8 @@ void SyncFileSystemService::OnFileStatusChanged(
SyncFileStatus sync_status,
SyncAction action_taken,
SyncDirection direction) {
- FOR_EACH_OBSERVER(
- SyncEventObserver, observers_,
- OnFileSynced(url, file_type, sync_status, action_taken, direction));
+ for (auto& observer : observers_)
+ observer.OnFileSynced(url, file_type, sync_status, action_taken, direction);
}
void SyncFileSystemService::UpdateSyncEnabledStatus(
« no previous file with comments | « chrome/browser/sync_file_system/mock_remote_file_sync_service.cc ('k') | chrome/browser/sync_file_system/task_logger.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698