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

Unified Diff: ui/arc/notification/arc_notification_surface_manager.cc

Issue 2416633002: Remove FOR_EACH_OBSERVER from arc/ since it's being deprecated (Closed)
Patch Set: remove braces 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: ui/arc/notification/arc_notification_surface_manager.cc
diff --git a/ui/arc/notification/arc_notification_surface_manager.cc b/ui/arc/notification/arc_notification_surface_manager.cc
index d816da8a234891c50acc7f88446cccabfb03685f..dbc03b45a3a8a6504731b674c2133e8fd2a7fb8f 100644
--- a/ui/arc/notification/arc_notification_surface_manager.cc
+++ b/ui/arc/notification/arc_notification_surface_manager.cc
@@ -47,7 +47,8 @@ void ArcNotificationSurfaceManager::AddSurface(
notification_surface_map_[surface->notification_id()] = surface;
- FOR_EACH_OBSERVER(Observer, observers_, OnNotificationSurfaceAdded(surface));
+ for (auto& observer : observers_)
+ observer.OnNotificationSurfaceAdded(surface);
}
void ArcNotificationSurfaceManager::RemoveSurface(
@@ -57,8 +58,8 @@ void ArcNotificationSurfaceManager::RemoveSurface(
return;
notification_surface_map_.erase(it);
- FOR_EACH_OBSERVER(Observer, observers_,
- OnNotificationSurfaceRemoved(surface));
+ for (auto& observer : observers_)
+ observer.OnNotificationSurfaceRemoved(surface);
}
void ArcNotificationSurfaceManager::AddObserver(Observer* observer) {
« components/arc/arc_bridge_service.cc ('K') | « ui/arc/notification/arc_custom_notification_item.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698