Allows menu host windows to be enumerated in DragTargetWindowFinder

ui/views/controls/menu/menu_host.cc

 // 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.
 
 #include "ui/views/controls/menu/menu_host.h"
 
 #include "base/auto_reset.h"
 #include "base/debug/trace_event.h"
 #include "ui/events/gestures/gesture_recognizer.h"
 #include "ui/gfx/path.h"
@@ skipping 74 matching lines @@
   ReleaseMenuHostCapture();
   Hide();
   ignore_capture_lost_ = false;
 }
 
 void MenuHost::DestroyMenuHost() {
   HideMenuHost();
   destroying_ = true;
   static_cast<MenuHostRootView*>(GetRootView())->ClearSubmenu();
   Close();
+  // Since |submenu_| is not a child view it is necessary to notify its
+  // observers so that they can clean up the foucs and drag and drop state.
+  View::ViewHierarchyChangedDetails details(false, NULL, submenu_, NULL);
+  ViewHierarchyChanged(details);

[sadrul, 2014/03/26 18:01:07]

Can this be separated out in a different CL with corresponding tests?

[varkha, 2014/03/27 04:48:13]

See https://codereview.chromium.org/214083004/.

 }
 
 void MenuHost::SetMenuHostBounds(const gfx::Rect& bounds) {
   SetBounds(bounds);
 }
 
 void MenuHost::ReleaseMenuHostCapture() {
   if (native_widget_private()->HasCapture())
     native_widget_private()->ReleaseCapture();
 }
@@ skipping 29 matching lines @@
   if (destroying_)
     return;
 
   MenuController* menu_controller =
       submenu_->GetMenuItem()->GetMenuController();
   if (menu_controller && !menu_controller->drag_in_progress())
     menu_controller->CancelAll();
 }
 
 }  // namespace views