Use showing_context_menu_ from RenderWidgetHostViewBase to avoid MouseLeave event on Context Menu

content/browser/renderer_host/render_widget_host_view_aura.cc

Index: content/browser/renderer_host/render_widget_host_view_aura.cc
diff --git a/content/browser/renderer_host/render_widget_host_view_aura.cc b/content/browser/renderer_host/render_widget_host_view_aura.cc
index d06fde40a5576d801c9d86386f99a1ca76309f37..31b448c5316267824f9e33305f9b83137a0217bc 100644
--- a/content/browser/renderer_host/render_widget_host_view_aura.cc
+++ b/content/browser/renderer_host/render_widget_host_view_aura.cc
@@ -474,7 +474,6 @@ RenderWidgetHostViewAura::RenderWidgetHostViewAura(RenderWidgetHost* host,
 #if defined(OS_WIN)
       legacy_render_widget_host_HWND_(NULL),
       legacy_window_destroyed_(false),
-      showing_context_menu_(false),
 #endif
       has_snapped_to_boundary_(false),
       is_guest_view_hack_(is_guest_view_hack),
@@ -798,11 +797,6 @@ bool RenderWidgetHostViewAura::CanRendererHandleEvent(
     const ui::MouseEvent* event,
     bool mouse_locked,
     bool selection_popup) {

[sadrul, 2016/02/12 15:30:12]

Oh, and while you are here, make this a const function please.

-#if defined(OS_WIN)
-  bool showing_context_menu = showing_context_menu_;
-  showing_context_menu_ = false;
-#endif
-
   if (event->type() == ui::ET_MOUSE_CAPTURE_CHANGED)
     return false;
 
@@ -813,7 +807,7 @@ bool RenderWidgetHostViewAura::CanRendererHandleEvent(
     // Don't forward the mouse leave message which is received when the context
     // menu is displayed by the page. This confuses the page and causes state
     // changes.
-    if (showing_context_menu)
+    if (showing_context_menu_)

[sadrul, 2016/02/12 01:06:52]

IsShowingContextMenu()

       return false;
 #endif
     return true;
@@ -2722,7 +2716,6 @@ bool RenderWidgetHostViewAura::OnShowContextMenu(
     *last_context_menu_params_ = params;
     return false;
   }
-  showing_context_menu_ = true;
 #endif
   return true;
 }