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

Unified Diff: chrome/browser/ui/views/toolbar/app_menu_button.cc

Issue 2424853003: Remove FOR_EACH_OBSERVER macro usage in chrome/browser/ui (Closed)
Patch Set: explicit types 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: chrome/browser/ui/views/toolbar/app_menu_button.cc
diff --git a/chrome/browser/ui/views/toolbar/app_menu_button.cc b/chrome/browser/ui/views/toolbar/app_menu_button.cc
index b17474269fbe713e5c80de011c06b08e31577044..881ec2fbdcadc386449a5a0fdd39f8d4aedb4df1 100644
--- a/chrome/browser/ui/views/toolbar/app_menu_button.cc
+++ b/chrome/browser/ui/views/toolbar/app_menu_button.cc
@@ -74,7 +74,8 @@ void AppMenuButton::ShowMenu(bool for_drop) {
menu_model_.reset(new AppMenuModel(toolbar_view_, browser));
menu_->Init(menu_model_.get());
- FOR_EACH_OBSERVER(views::MenuListener, menu_listeners_, OnMenuOpened());
+ for (views::MenuListener& observer : menu_listeners_)
+ observer.OnMenuOpened();
base::TimeTicks menu_open_time = base::TimeTicks::Now();
menu_->RunMenu(this);
« no previous file with comments | « chrome/browser/ui/views/toolbar/app_menu.cc ('k') | chrome/browser/ui/views/website_settings/chosen_object_row.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698