Context menu on views must show on mouse down for non-WIN.

chrome/browser/ui/views/browser_action_view.cc

Index: chrome/browser/ui/views/browser_action_view.cc
diff --git a/chrome/browser/ui/views/browser_action_view.cc b/chrome/browser/ui/views/browser_action_view.cc
index 7a8a81ef112bd77d42c6ea474a75a621e6945147..f2102e2c0c25f3f880ea82bda4b4e0e78f4212e8 100644
--- a/chrome/browser/ui/views/browser_action_view.cc
+++ b/chrome/browser/ui/views/browser_action_view.cc
@@ -314,10 +314,6 @@ bool BrowserActionButton::OnMousePressed(const ui::MouseEvent& event) {
                        TextButton::OnMousePressed(event);
   }
 
-  // See comments in MenuButton::Activate() as to why this is needed.

[sky, 2013/04/04 20:36:04]

Why are you removing this?

[varunjain, 2013/04/04 21:02:08]

Now that we are showing context menu on mouse press for all views, this is not
needed here since it would happen through normal views workflow in
View::ProcessedMousePressed. However, this should not be skipped on OS_WIN.
Adding it back with appropriate conditional.

-  SetMouseHandler(NULL);
-
-  ShowContextMenu(gfx::Point(), true);
   return false;
 }