Allow menus to stay open during a child view's drag and drop

ui/views/controls/menu/menu_delegate.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_DELEGATE_H_
 #define UI_VIEWS_CONTROLS_MENU_MENU_DELEGATE_H_
 
 #include <set>
 #include <string>
 
@@ skipping 114 matching lines @@
   virtual void ExecuteCommand(int id) {
   }
 
   // If nested menus are showing (nested menus occur when a menu shows a context
   // menu) this is invoked to determine if all the menus should be closed when
   // the user selects the menu with the command |id|. This returns true to
   // indicate that all menus should be closed. Return false if only the
   // context menu should be closed.
   virtual bool ShouldCloseAllMenusOnExecute(int id);
 
+  // Returns true if the menu should close upon a drag completing. Defaults to
+  // true.

[sky, 2014/07/14 15:19:05]

Document this is only invoked for menus originating outside of those on menu
items. Also, move this right above SelectionChanged() so that it's grouped with
the other dnd functions.

[Devlin, 2014/07/16 16:00:45]

Done.

+  virtual bool ShouldCloseOnDragComplete();
+
   // Executes the specified command. mouse_event_flags give the flags of the
   // mouse event that triggered this to be invoked (ui::MouseEvent
   // flags). mouse_event_flags is 0 if this is triggered by a user gesture
   // other than a mouse event.
   virtual void ExecuteCommand(int id, int mouse_event_flags);
 
   // Returns true if ExecuteCommand() should be invoked while leaving the
   // menu open. Default implementation returns true.
   virtual bool ShouldExecuteCommandWithoutClosingMenu(int id,
                                                       const ui::Event& e);
@@ skipping 96 matching lines @@
                                         int* left_margin,
                                         int* right_margin) const;
   // Returns true if the labels should reserve additional spacing for e.g.
   // submenu indicators at the end of the line.
   virtual bool ShouldReserveSpaceForSubmenuIndicator() const;
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_CONTROLS_MENU_MENU_DELEGATE_H_