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

Unified Diff: components/exo/wm_helper.cc

Issue 2440303002: Remove usage of FOR_EACH_OBSERVER macro in components/ (Closed)
Patch Set: remove superfluous return 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
« no previous file with comments | « components/exo/surface.cc ('k') | components/favicon/core/favicon_driver.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/exo/wm_helper.cc
diff --git a/components/exo/wm_helper.cc b/components/exo/wm_helper.cc
index 517a4ae414518b419f024c780dee897db8cef52a..6be7aff4f67848660249a06657e7583510a675df 100644
--- a/components/exo/wm_helper.cc
+++ b/components/exo/wm_helper.cc
@@ -72,39 +72,39 @@ void WMHelper::RemoveAccessibilityObserver(AccessibilityObserver* observer) {
void WMHelper::NotifyWindowActivated(aura::Window* gained_active,
aura::Window* lost_active) {
- FOR_EACH_OBSERVER(ActivationObserver, activation_observers_,
- OnWindowActivated(gained_active, lost_active));
+ for (ActivationObserver& observer : activation_observers_)
+ observer.OnWindowActivated(gained_active, lost_active);
}
void WMHelper::NotifyWindowFocused(aura::Window* gained_focus,
aura::Window* lost_focus) {
- FOR_EACH_OBSERVER(FocusObserver, focus_observers_,
- OnWindowFocused(gained_focus, lost_focus));
+ for (FocusObserver& observer : focus_observers_)
+ observer.OnWindowFocused(gained_focus, lost_focus);
}
void WMHelper::NotifyCursorVisibilityChanged(bool is_visible) {
- FOR_EACH_OBSERVER(CursorObserver, cursor_observers_,
- OnCursorVisibilityChanged(is_visible));
+ for (CursorObserver& observer : cursor_observers_)
+ observer.OnCursorVisibilityChanged(is_visible);
}
void WMHelper::NotifyCursorSetChanged(ui::CursorSetType cursor_set) {
- FOR_EACH_OBSERVER(CursorObserver, cursor_observers_,
- OnCursorSetChanged(cursor_set));
+ for (CursorObserver& observer : cursor_observers_)
+ observer.OnCursorSetChanged(cursor_set);
}
void WMHelper::NotifyMaximizeModeStarted() {
- FOR_EACH_OBSERVER(MaximizeModeObserver, maximize_mode_observers_,
- OnMaximizeModeStarted());
+ for (MaximizeModeObserver& observer : maximize_mode_observers_)
+ observer.OnMaximizeModeStarted();
}
void WMHelper::NotifyMaximizeModeEnded() {
- FOR_EACH_OBSERVER(MaximizeModeObserver, maximize_mode_observers_,
- OnMaximizeModeEnded());
+ for (MaximizeModeObserver& observer : maximize_mode_observers_)
+ observer.OnMaximizeModeEnded();
}
void WMHelper::NotifyAccessibilityModeChanged() {
- FOR_EACH_OBSERVER(AccessibilityObserver, accessibility_observers_,
- OnAccessibilityModeChanged());
+ for (AccessibilityObserver& observer : accessibility_observers_)
+ observer.OnAccessibilityModeChanged();
}
} // namespace exo
« no previous file with comments | « components/exo/surface.cc ('k') | components/favicon/core/favicon_driver.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698