Only set mLastSelectedText when selection handles are present

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 53b22d9fc5d6dd578eef933723738e70d60a56fe..9d1e5170d492045be04c4bdeea690810582acef0 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
@@ -1055,7 +1055,6 @@ public class SelectionPopupController extends ActionModeCallbackHelper {
 
     @CalledByNative
     private void onSelectionChanged(String text) {
-        mLastSelectedText = text;

[aelias_OOO_until_Jul13, 2017/06/06 22:14:43]

This doesn't seem correct because contextual search uses mLastSelectedText via
ContentViewCore.getSelectedText().  Also, are you sure it will always be updated
properly if it changes after the initial action mode creation for the purposes
of action mode share/search/etc?

Maybe we need to go back to holding a separate boolean after all?

         if (mSelectionClient != null) {
             mSelectionClient.onSelectionChanged(text);
         }