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

Unified Diff: ui/views/controls/menu/menu_runner_impl_adapter.cc

Issue 2394123002: Views: Expose an on_closed callback via the MenuRunner constructor. (Closed)
Patch Set: default arg 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 | « ui/views/controls/menu/menu_runner_impl_adapter.h ('k') | ui/views/controls/menu/menu_runner_impl_cocoa.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/views/controls/menu/menu_runner_impl_adapter.cc
diff --git a/ui/views/controls/menu/menu_runner_impl_adapter.cc b/ui/views/controls/menu/menu_runner_impl_adapter.cc
index 932af7ec8611d57830277c8bee5f281bfbf74e8a..31241bfa0fa73c07c5529ec3e4adefb9620973d9 100644
--- a/ui/views/controls/menu/menu_runner_impl_adapter.cc
+++ b/ui/views/controls/menu/menu_runner_impl_adapter.cc
@@ -10,10 +10,12 @@
namespace views {
namespace internal {
-MenuRunnerImplAdapter::MenuRunnerImplAdapter(ui::MenuModel* menu_model)
- : menu_model_adapter_(new MenuModelAdapter(menu_model)),
- impl_(new MenuRunnerImpl(menu_model_adapter_->CreateMenu())) {
-}
+MenuRunnerImplAdapter::MenuRunnerImplAdapter(
+ ui::MenuModel* menu_model,
+ const base::Closure& on_menu_done_callback)
+ : menu_model_adapter_(
+ new MenuModelAdapter(menu_model, on_menu_done_callback)),
+ impl_(new MenuRunnerImpl(menu_model_adapter_->CreateMenu())) {}
bool MenuRunnerImplAdapter::IsRunning() const {
return impl_->IsRunning();
« no previous file with comments | « ui/views/controls/menu/menu_runner_impl_adapter.h ('k') | ui/views/controls/menu/menu_runner_impl_cocoa.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698