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

Issue 2417463009: Remove usage of FOR_EACH_OBSERVER macro in ui/views/. (Closed)

Created:
4 years, 2 months ago by loyso (OOO)
Modified:
4 years, 2 months ago
Reviewers:
Avi (use Gerrit)
CC:
chromium-reviews, tfarina
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Remove usage of FOR_EACH_OBSERVER macro in ui/views/. Observer lists now support range-based for loops. BUG=655021 Closed in favor of https://codereview.chromium.org/2418983002/

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+50 lines, -56 lines) Patch
M ui/views/animation/bounds_animator.cc View 1 chunk +4 lines, -6 lines 0 comments Download
M ui/views/focus/focus_manager.cc View 2 chunks +4 lines, -4 lines 0 comments Download
M ui/views/focus/widget_focus_manager.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M ui/views/mus/pointer_watcher_event_router.cc View 2 chunks +8 lines, -10 lines 0 comments Download
M ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc View 2 chunks +4 lines, -6 lines 0 comments Download
M ui/views/widget/desktop_aura/x11_desktop_handler.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M ui/views/widget/widget.cc View 8 chunks +22 lines, -20 lines 0 comments Download
M ui/views/win/windows_session_change_observer.cc View 2 chunks +4 lines, -6 lines 0 comments Download

Messages

Total messages: 9 (7 generated)
loyso (OOO)
4 years, 2 months ago (2016-10-14 03:53:06 UTC) #3
Avi (use Gerrit)
4 years, 2 months ago (2016-10-14 04:01:31 UTC) #6
lgtm

Powered by Google App Engine
This is Rietveld 408576698