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

Unified Diff: ui/arc/notification/arc_custom_notification_item.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_custom_notification_item.cc
diff --git a/ui/arc/notification/arc_custom_notification_item.cc b/ui/arc/notification/arc_custom_notification_item.cc
index 24fcbb49476b7d71b9a3ba1e662953f6648b271f..f3998a5d5fb566995f5db5bfa03ac42f5a4d270e 100644
--- a/ui/arc/notification/arc_custom_notification_item.cc
+++ b/ui/arc/notification/arc_custom_notification_item.cc
@@ -49,7 +49,8 @@ ArcCustomNotificationItem::ArcCustomNotificationItem(
}
ArcCustomNotificationItem::~ArcCustomNotificationItem() {
- FOR_EACH_OBSERVER(Observer, observers_, OnItemDestroying());
+ for (auto& observer : observers_)
+ observer.OnItemDestroying();
}
void ArcCustomNotificationItem::UpdateWithArcNotificationData(
@@ -93,7 +94,8 @@ void ArcCustomNotificationItem::UpdateWithArcNotificationData(
*data->snapshot_image, data->snapshot_image_scale));
}
- FOR_EACH_OBSERVER(Observer, observers_, OnItemUpdated());
+ for (auto& observer : observers_)
+ observer.OnItemUpdated();
AddToMessageCenter();
}

Powered by Google App Engine
This is Rietveld 408576698