Selection Action mode triggered like a context menu

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index bf90b0ae15404e326ea89b1e84085293b60788ed..c5f8f5c308afd05e9531bf93c99577201a6f0a70 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -4293,8 +4293,11 @@ void RenderFrameImpl::ShowContextMenu(const blink::WebContextMenuData& data) {
   GetRenderWidget()->ConvertViewportToWindow(&position_in_window);
   params.x = position_in_window.x;
   params.y = position_in_window.y;
-  params.source_type =
-      GetRenderWidget()->input_handler().context_menu_source_type();
+  if (params.source_type == ui::MENU_SOURCE_NONE) {
+    params.source_type =
+        GetRenderWidget()->input_handler().context_menu_source_type();

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

pass this into Builder, instead of having this brittle check?

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

I decided to remove the SelectAll plumbing from this CL. I'll do the plumbing in
a separate CL.

+  }
+
   GetRenderWidget()->OnShowHostContextMenu(&params);
   if (GetRenderWidget()->has_host_context_menu_location()) {
     params.x = GetRenderWidget()->host_context_menu_location().x();