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

ui/views/widget/widget.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_WIDGET_WIDGET_H_
 #define UI_VIEWS_WIDGET_WIDGET_H_
 
 #include <set>
 #include <stack>
 #include <vector>
@@ skipping 775 matching lines @@
   // Creates the RootView to be used within this Widget. Subclasses may override
   // to create custom RootViews that do specialized event processing.
   // TODO(beng): Investigate whether or not this is needed.
   virtual internal::RootView* CreateRootView();
 
   // Provided to allow the NativeWidget implementations to destroy the RootView
   // _before_ the focus manager/tooltip manager.
   // TODO(beng): remove once we fold those objects onto this one.
   void DestroyRootView();
 
+  bool child_drag_in_progress() const { return child_drag_in_progress_; }
+  void set_child_drag_in_progress(bool child_drag_in_progress) {
+    child_drag_in_progress_ = child_drag_in_progress;
+  }
+
  private:
   friend class ComboboxTest;
   friend class TextfieldTest;
 
   // Sets the value of |disable_inactive_rendering_|. If the value changes,
   // both the NonClientView and WidgetDelegate are notified.
   void SetInactiveRenderingDisabled(bool value);
 
   // Persists the window's restored position and "show" state using the
   // window delegate.
@@ skipping 45 matching lines @@
   // must be destroyed AFTER root_view_. This is enforced in DestroyRootView().
   scoped_ptr<FocusManager> focus_manager_;
 
   // A theme provider to use when no other theme provider is specified.
   scoped_ptr<ui::DefaultThemeProvider> default_theme_provider_;
 
   // Valid for the lifetime of RunShellDrag(), indicates the view the drag
   // started from.
   View* dragged_view_;
 
+  // Whether or not a child of this widget is in the middle of a drag.
+  bool child_drag_in_progress_;

[Devlin, 2014/07/09 18:28:44]

This name isn't strictly true, since only the top-level widget will know about
this, but I think that's all that's needed (since it is the top-level widget
which closes the menu when it loses focus).  We could do this for each parent,
though, without much trouble.

+
   // See class documentation for Widget above for a note about ownership.
   InitParams::Ownership ownership_;
 
   // See set_is_secondary_widget().
   bool is_secondary_widget_;
 
   // The current frame type in use by this window. Defaults to
   // FRAME_TYPE_DEFAULT.
   FrameType frame_type_;
 
@@ skipping 57 matching lines @@
   bool movement_disabled_;
 
   ScopedObserver<ui::NativeTheme, ui::NativeThemeObserver> observer_manager_;
 
   DISALLOW_COPY_AND_ASSIGN(Widget);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_WIDGET_WIDGET_H_