[TTS] Add an ACK message to SelectWordAroundCaret.

chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchSelectionController.java

Index: chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchSelectionController.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchSelectionController.java b/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchSelectionController.java
index c16b63a10a95e081fa8507759db6b768d765fb23..2cd3e65bbab65828b747a0dd52a23f127e1a1bd0 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchSelectionController.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchSelectionController.java
@@ -430,6 +430,19 @@ public class ContextualSearchSelectionController {
         }
     }
 
+    /**
+     * Acknowledges that a selectWordAroundCaret action has completed with the give result.
+     * @param didSelect Whether a word was actually selected or not.
+     * @param startAdjust The adjustment to the selection start offset needed to select the word.
+     *        This is typically a negative number (expressed in terms of number of characters).
+     * @param endAdjust The adjustment to the selection end offset needed to select the word.
+     *        This is typically a positive number (expressed in terms of number of characters).
+     */
+    void selectWordAroundCaretAck(boolean didSelect, int startAdjust, int endAdjust) {
+        // TODO(donnd): use startAdjust and endAdjust in upcoming text-extraction CL (started).

[aelias_OOO_until_Jul13, 2017/02/22 03:36:00]

You're planning to use these offsets to issue another IPC to extract the text? 
Could you show me the WIP patch for that?  Wouldn't it be cleaner to return the
extracted text contents as part of the ack you're adding now instead of making
these multiple round-trips?

[Donn Denman, 2017/02/22 23:13:01]

Actually we plan to just use these to convert some selection offsets before
sending to the server.  We don't plan to issue any additional IPC (and the WIP
patch isn't really ready to look at).

+        if (!didSelect) mHandler.handleInvalidTap();

[aelias_OOO_until_Jul13, 2017/02/22 03:36:00]

This seems a bit upside down.  How about doing nothing if the ack is false, and
kicking off the UI work if the ack is true?

[Donn Denman, 2017/02/22 23:13:01]

I think this will look much better once the dependent patch lands (which is a
high priority to do soon).  Then we'll take some action in both cases.

+    }
+
     // ============================================================================================
     // Invalid Tap Notification
     // ============================================================================================