[SmartText selection] Add LocaleList to the provider interface

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

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.content.browser;
 
 import android.content.Intent;
 import android.graphics.drawable.Drawable;
+import android.os.LocaleList;

[Ted C, 2017/04/13 17:17:10]

LocaleList is api 24...are any of the apis below called for older versions of
Android?  If so, that probably crashes.

[Tima Vaisburd, 2017/04/13 18:11:32]

The implementation only calls to TextClassifier API which is O and above.
Do you suggest making this insterface robust for all Android versions, e.g. by
providing an array of strings and converting them on the implementation side?

 import android.view.View.OnClickListener;
 
 import org.chromium.base.annotations.SuppressFBWarnings;
 
 /**
  * The interface that controls contextual text selection.
  */
 @SuppressFBWarnings("UUF_UNUSED_PUBLIC_OR_PROTECTED_FIELD")
 public interface ContextSelectionProvider {
     /**
@@ skipping 52 matching lines @@
     }
 
     /**
      * Sends asynchronous request to obtain the selection, analyze its type and suggest
      * better selection boundaries.
      * @param text  The textual context that encloses the selected text.
      * @param start The start index of the selected text inside the textual context.
      * @param end   The index pointing to the first character that comes after
      *              the selected text inside the textual context.
      */
-    public void sendSuggestAndClassifyRequest(CharSequence text, int start, int end);
+    public void sendSuggestAndClassifyRequest(
+            CharSequence text, int start, int end, LocaleList locales);
 
     /**
      * Sends asynchronous request to obtain the selection and analyze its type.
      * @param text  The textual context that encloses the selected text.
      * @param start The start index of the selected text inside the textual context.
      * @param end   The index pointing to the first character that comes after
      *              the selected text inside the textual context.
      */
-    public void sendClassifyRequest(CharSequence text, int start, int end);
+    public void sendClassifyRequest(CharSequence text, int start, int end, LocaleList locales);
 
     /**
      * Cancel all asynchronous requests.
      */
     public void cancelAllRequests();
 }