New MenuController Checks

ui/views/controls/menu/menu_controller.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef UI_VIEWS_CONTROLS_MENU_MENU_CONTROLLER_H_
 #define UI_VIEWS_CONTROLS_MENU_MENU_CONTROLLER_H_
 
 #include <stddef.h>
 
 #include <list>
@@ skipping 556 matching lines @@
 
   // If true, Run blocks. If false, Run doesn't block and this is used for
   // drag and drop. Note that the semantics for drag and drop are slightly
   // different: cancel timer is kicked off any time the drag moves outside the
   // menu, mouse events do nothing...
   bool blocking_run_;
 
   // If true, we're showing.
   bool showing_;
 
+  // If true running has began and there are still delegates which have not been

[sky, 2017/01/27 20:16:47]

Please add references to bug where with a TODO that this is temporary to track
down bug. Same thing for changes in MenuRunnerImpl.

[jonross, 2017/01/27 20:27:36]

Done.

+  // closed. We may not be showing.
+  bool running_;
+
   // Indicates what to exit.
   ExitType exit_type_;
 
   // Whether we did a capture. We do a capture only if we're blocking and
   // the mouse was down when Run.
   bool did_capture_;
 
   // As the user drags the mouse around pending_state_ changes immediately.
   // When the user stops moving/dragging the mouse (or clicks the mouse)
   // pending_state_ is committed to state_, potentially resulting in
@@ skipping 118 matching lines @@
 #if defined(USE_AURA)
   std::unique_ptr<MenuPreTargetHandler> menu_pre_target_handler_;
 #endif
 
   DISALLOW_COPY_AND_ASSIGN(MenuController);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_CONTROLS_MENU_MENU_CONTROLLER_H_