Open the WrenchMenu on mouseover when dragging a browser action

chrome/browser/ui/views/toolbar/browser_action_view.h

Index: chrome/browser/ui/views/toolbar/browser_action_view.h
diff --git a/chrome/browser/ui/views/toolbar/browser_action_view.h b/chrome/browser/ui/views/toolbar/browser_action_view.h
index 24ff613d8d368e0756ccdad03925c33a1308090b..fd0391c0c0887d802747c64690da43aa612938e6 100644
--- a/chrome/browser/ui/views/toolbar/browser_action_view.h
+++ b/chrome/browser/ui/views/toolbar/browser_action_view.h
@@ -57,6 +57,12 @@ class BrowserActionView : public views::View {
     // Whether the container for this button is shown inside a menu.
     virtual bool ShownInsideMenu() const = 0;
 
+    // Notifies that a drag completed. Note this will only happen if the view
+    // wasn't removed during the drag-and-drop process (which happens if there

[Finnur, 2014/07/18 15:53:51]

s/which happens if/not when/

Right?

[Devlin, 2014/07/18 18:58:52]

Yeah, that looks better.  Done.

+    // was a move in the browser actions, since we re-create the views each
+    // time we re-order the browser actions).
+    virtual void OnBrowserActionViewDragDone() = 0;
+
    protected:
     virtual ~Delegate() {}
   };
@@ -201,6 +207,7 @@ class BrowserActionButton : public views::MenuButton,
   // Overridden from views::View:
   virtual void ViewHierarchyChanged(
       const ViewHierarchyChangedDetails& details) OVERRIDE;
+  virtual void OnDragDone() OVERRIDE;
 
  private:
   virtual ~BrowserActionButton();