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..ebba98b64107d0fea4529847e44c933006613466 100644
--- a/chrome/browser/ui/views/toolbar/browser_action_view.h
+++ b/chrome/browser/ui/views/toolbar/browser_action_view.h
@@ -57,6 +57,10 @@ 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.

[Finnur, 2014/07/18 10:40:40]

... as in the extension being uninstalled?

[Devlin, 2014/07/18 15:44:32]

Not quite.  We delete and recreate all the views every time there's a change in
the browser actions order.  Which is kinda excessive (and results in an ugly
"flicker" on repaint), but that's a separate issue to fix. :)  Problem is, the
view is only notified that the drag is done if it wasn't deleted as part of the
drag - which these are if there's actually a movement.

Comment updated to make it more clear.

+    virtual void OnBrowserActionViewDragDone() = 0;
+
    protected:
     virtual ~Delegate() {}
   };
@@ -201,6 +205,7 @@ class BrowserActionButton : public views::MenuButton,
   // Overridden from views::View:
   virtual void ViewHierarchyChanged(
       const ViewHierarchyChangedDetails& details) OVERRIDE;
+  virtual void OnDragDone() OVERRIDE;
 
  private:
   virtual ~BrowserActionButton();