Selection Action mode triggered like a context menu

third_party/WebKit/Source/core/input/GestureManager.cpp

Index: third_party/WebKit/Source/core/input/GestureManager.cpp
diff --git a/third_party/WebKit/Source/core/input/GestureManager.cpp b/third_party/WebKit/Source/core/input/GestureManager.cpp
index 1554c160209746c014aef38a7062dd7aeea547db..e414ea2ccf0db5b9edef735e03cd645ef696a8c2 100644
--- a/third_party/WebKit/Source/core/input/GestureManager.cpp
+++ b/third_party/WebKit/Source/core/input/GestureManager.cpp
@@ -301,6 +301,9 @@ WebInputEventResult GestureManager::HandleGestureTap(
         tapped_position_in_viewport, tapped_node,
         dom_tree_changed || style_changed);
   }
+
+  frame_->GetEventHandler().SendContextMenuEventForTouchSelection(

[aelias_OOO_until_Jul13, 2017/05/23 20:52:05]

How about putting this behind "if (event_result ==
WebInputEventResult::kHandledSystem)"?  I think if this was kHandledApplication
for example, we don't want to show the menu.

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

Done.

+      kMenuSourceTouch);
   return event_result;
 }
 
@@ -333,6 +336,8 @@ WebInputEventResult GestureManager::HandleGestureLongPress(
   if (inner_node && inner_node->GetLayoutObject() &&
       selection_controller_->HandleGestureLongPress(hit_test_result)) {
     mouse_event_manager_->FocusDocumentView();
+    frame_->GetEventHandler().SendContextMenuEventForTouchSelection(
+        kMenuSourceLongPress);
     return WebInputEventResult::kHandledSystem;
   }