Selection Action mode triggered like a context menu

content/browser/renderer_host/input/touch_selection_controller_client_aura.cc

Index: content/browser/renderer_host/input/touch_selection_controller_client_aura.cc
diff --git a/content/browser/renderer_host/input/touch_selection_controller_client_aura.cc b/content/browser/renderer_host/input/touch_selection_controller_client_aura.cc
index b07e3adb4aca2e4d9258d31f622678dd86411881..fb8acd698b2467986eba4d5fffe254bf2691f01a 100644
--- a/content/browser/renderer_host/input/touch_selection_controller_client_aura.cc
+++ b/content/browser/renderer_host/input/touch_selection_controller_client_aura.cc
@@ -165,6 +165,12 @@ bool TouchSelectionControllerClientAura::HandleContextMenu(
     UpdateQuickMenu();
     return true;
   }
+
+  const bool from_touch = params.source_type == ui::MENU_SOURCE_LONG_PRESS ||

[boliu, 2017/05/23 17:34:13]

were you intending to change aura? I guess blink changes made this necessary?

[amaralp1, 2017/05/24 08:30:34]

Yes, the now blink sends context menu events when touch selection handles are
shown and aura is not expecting them so we need to ignore them in aura.
Eventually we should change aura's implementation to rely on the context menu
events instead of selection handle changes.

+                          params.source_type == ui::MENU_SOURCE_TOUCH;
+  if (from_touch && !params.selection_text.empty())
+    return true;
+
   rwhva_->selection_controller()->HideAndDisallowShowingAutomatically();
   return false;
 }