Views: Expose an on_closed callback via the MenuRunner constructor.

ui/views/controls/menu/menu_runner.h

Index: ui/views/controls/menu/menu_runner.h
diff --git a/ui/views/controls/menu/menu_runner.h b/ui/views/controls/menu/menu_runner.h
index 1bbc905e2ba9670e3c66c721adf606320b0087a5..0be64f9bca928c1a637703276e5c19fe1eb05fa1 100644
--- a/ui/views/controls/menu/menu_runner.h
+++ b/ui/views/controls/menu/menu_runner.h
@@ -9,6 +9,7 @@
 
 #include <memory>
 
+#include "base/callback_forward.h"
 #include "base/compiler_specific.h"
 #include "base/macros.h"
 #include "ui/base/ui_base_types.h"
@@ -105,9 +106,15 @@ class VIEWS_EXPORT MenuRunner {
     NORMAL_EXIT
   };
 
-  // Creates a new MenuRunner.
-  // |run_types| is a bitmask of RunTypes.
+  // Creates a new MenuRunner, which may use a native menu if available.
+  // |run_types| is a bitmask of RunTypes. If provided,
+  // |on_menu_closed_callback| is invoked when the menu is closed.
+  MenuRunner(ui::MenuModel* menu_model,
+             int32_t run_types,
+             const base::Closure& on_menu_closed_callback);

[sky, 2016/10/13 17:44:07]

Combine this with the next constructor and use a default value? (style guide now
allows default values in cases like this).

[tapted, 2016/10/13 21:31:40]

Done.

   MenuRunner(ui::MenuModel* menu_model, int32_t run_types);
+
+  // Creates a runner for a custom-created toolkit-views menu.
   MenuRunner(MenuItemView* menu, int32_t run_types);
   ~MenuRunner();