Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(369)

Unified Diff: trunk/src/chrome/browser/ui/views/toolbar/browser_action_view.h

Issue 416903002: Revert 285142 "Open the WrenchMenu on mouseover when dragging a ..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: trunk/src/chrome/browser/ui/views/toolbar/browser_action_view.h
===================================================================
--- trunk/src/chrome/browser/ui/views/toolbar/browser_action_view.h (revision 285300)
+++ trunk/src/chrome/browser/ui/views/toolbar/browser_action_view.h (working copy)
@@ -57,12 +57,6 @@
// 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 (i.e., not when there
- // 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() {}
};
@@ -207,7 +201,6 @@
// Overridden from views::View:
virtual void ViewHierarchyChanged(
const ViewHierarchyChangedDetails& details) OVERRIDE;
- virtual void OnDragDone() OVERRIDE;
private:
virtual ~BrowserActionButton();

Powered by Google App Engine
This is Rietveld 408576698