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

Unified Diff: ui/wm/core/cursor_manager.cc

Issue 2416963006: Remove usage of FOR_EACH_OBSERVER macro in ui/wm/. (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
« no previous file with comments | « no previous file | ui/wm/core/default_activation_client.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/wm/core/cursor_manager.cc
diff --git a/ui/wm/core/cursor_manager.cc b/ui/wm/core/cursor_manager.cc
index 1704398745f0f9142b5682dbf7aaf206c0ba0144..1c3c6c99a12d50629540067e965465ead39962d7 100644
--- a/ui/wm/core/cursor_manager.cc
+++ b/ui/wm/core/cursor_manager.cc
@@ -110,8 +110,8 @@ void CursorManager::ShowCursor() {
if (cursor_lock_count_ == 0 &&
IsCursorVisible() != state_on_unlock_->visible()) {
delegate_->SetVisibility(state_on_unlock_->visible(), this);
- FOR_EACH_OBSERVER(aura::client::CursorClientObserver, observers_,
- OnCursorVisibilityChanged(true));
+ for (auto& observer : observers_)
+ observer.OnCursorVisibilityChanged(true);
}
}
@@ -121,8 +121,8 @@ void CursorManager::HideCursor() {
if (cursor_lock_count_ == 0 &&
IsCursorVisible() != state_on_unlock_->visible()) {
delegate_->SetVisibility(state_on_unlock_->visible(), this);
- FOR_EACH_OBSERVER(aura::client::CursorClientObserver, observers_,
- OnCursorVisibilityChanged(false));
+ for (auto& observer : observers_)
+ observer.OnCursorVisibilityChanged(false);
}
}
@@ -134,8 +134,8 @@ void CursorManager::SetCursorSet(ui::CursorSetType cursor_set) {
state_on_unlock_->set_cursor_set(cursor_set);
if (GetCursorSet() != state_on_unlock_->cursor_set()) {
delegate_->SetCursorSet(state_on_unlock_->cursor_set(), this);
- FOR_EACH_OBSERVER(aura::client::CursorClientObserver, observers_,
- OnCursorSetChanged(cursor_set));
+ for (auto& observer : observers_)
+ observer.OnCursorSetChanged(cursor_set);
}
}
@@ -218,8 +218,8 @@ void CursorManager::CommitCursor(gfx::NativeCursor cursor) {
void CursorManager::CommitVisibility(bool visible) {
// TODO(tdanderson): Find a better place for this so we don't
// notify the observers more than is necessary.
- FOR_EACH_OBSERVER(aura::client::CursorClientObserver, observers_,
- OnCursorVisibilityChanged(visible));
+ for (auto& observer : observers_)
+ observer.OnCursorVisibilityChanged(visible);
current_state_->SetVisible(visible);
}
« no previous file with comments | « no previous file | ui/wm/core/default_activation_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698