Only start action mode if there is a selection

content/public/android/java/src/org/chromium/content/browser/SelectionPopupController.java

Index: content/public/android/java/src/org/chromium/content/browser/SelectionPopupController.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/SelectionPopupController.java b/content/public/android/java/src/org/chromium/content/browser/SelectionPopupController.java
index 0a09e02ea4bef50d47500f3fdb303baffb3114d0..206765454cee9117d7da6f9efe7edb0acbadea53 100644
--- a/content/public/android/java/src/org/chromium/content/browser/SelectionPopupController.java
+++ b/content/public/android/java/src/org/chromium/content/browser/SelectionPopupController.java
@@ -226,7 +226,7 @@ public class SelectionPopupController extends ActionModeCallbackHelper {
     public void showActionModeOrClearOnFailure() {
         mPendingShowActionMode = false;
 
-        if (!isActionModeSupported()) return;
+        if (!isActionModeSupported() || !mHasSelection) return;

[Tima Vaisburd, 2017/04/25 21:34:49]

Do we ever create this ActinMode for insertion? At least createActionMenu()
calls isInsertion() (indirectly). For the editable situation the context menu
would make sense even if there is no text yet.

 
         // Just refresh the view if action mode already exists.
         if (isActionModeValid()) {