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

ui/views/widget/widget.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/widget/widget.h"
 
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/utf_string_conversions.h"
@@ skipping 1133 matching lines @@
 void Widget::OnKeyEvent(ui::KeyEvent* event) {
   ScopedEvent scoped(this, *event);
   static_cast<internal::RootView*>(GetRootView())->
       DispatchKeyEvent(event);
 }
 
 void Widget::OnMouseEvent(ui::MouseEvent* event) {
   ScopedEvent scoped(this, *event);
   View* root_view = GetRootView();
   switch (event->type()) {
-    case ui::ET_MOUSE_PRESSED:
+    case ui::ET_MOUSE_PRESSED: {
       last_mouse_event_was_move_ = false;
+
+      // We may get deleted by the time we return from OnMousePressed. So we
+      // use an observer to make sure we are still alive.
+      WidgetDeletionObserver widget_deletion_observer(this);
       // Make sure we're still visible before we attempt capture as the mouse
       // press processing may have made the window hide (as happens with menus).
-      if (root_view && root_view->OnMousePressed(*event) && IsVisible()) {
+      if (root_view && root_view->OnMousePressed(*event) &&
+          widget_deletion_observer.IsWidgetAlive() && IsVisible()) {
         is_mouse_button_pressed_ = true;
         if (!native_widget_->HasCapture())
           native_widget_->SetCapture();
         event->SetHandled();
       }
       return;
+    }
     case ui::ET_MOUSE_RELEASED:
       last_mouse_event_was_move_ = false;
       is_mouse_button_pressed_ = false;
       // Release capture first, to avoid confusion if OnMouseReleased blocks.
       if (native_widget_->HasCapture() &&
           ShouldReleaseCaptureOnMouseReleased()) {
         native_widget_->ReleaseCapture();
       }
       if (root_view)
         root_view->OnMouseReleased(*event);
@@ skipping 243 matching lines @@
   }
   return false;
 }
 
 void Widget::ReplaceInputMethod(InputMethod* input_method) {
   input_method_.reset(input_method);
   input_method->SetDelegate(native_widget_->GetInputMethodDelegate());
   input_method->Init(this);
 }
 
+////////////////////////////////////////////////////////////////////////////////
+// WidgetDeletionObserver:
+
+WidgetDeletionObserver::WidgetDeletionObserver(Widget* widget)
+    : widget_(widget) {
+  if (widget_)
+    widget_->AddObserver(this);
+}
+
+WidgetDeletionObserver::~WidgetDeletionObserver() {
+  CleanupWidget();
+}
+
+bool WidgetDeletionObserver::IsWidgetAlive() {
+  return widget_;
+}
+
+void WidgetDeletionObserver::OnWidgetDestroying(Widget* widget) {
+  CleanupWidget();
+}
+
+void WidgetDeletionObserver::CleanupWidget() {
+  if (widget_) {
+    widget_->RemoveObserver(this);
+    widget_ = NULL;
+  }
+}
+
 namespace internal {
 
 ////////////////////////////////////////////////////////////////////////////////
 // internal::NativeWidgetPrivate, NativeWidget implementation:
 
 internal::NativeWidgetPrivate* NativeWidgetPrivate::AsNativeWidgetPrivate() {
   return this;
 }
 
 }  // namespace internal
 }  // namespace views