[Contextual Search] Trigger translation on long-press.

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

 // Copyright 2015 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.chrome.browser.contextualsearch;
 
 import android.app.Activity;
 import android.content.Context;
 import android.text.TextUtils;
 import android.view.View;
@@ skipping 474 matching lines @@
                 mNetworkCommunicator.getBasePageUrl())) {
             mNetworkCommunicator.startSearchTermResolutionRequest(
                     mSelectionController.getSelectedText());
             didRequestSurroundings = true;
             // Cache the target languages in case they are needed for translation.
             if (!mPolicy.disableForceTranslationOnebox()) getReadableLanguages();
         } else {
             boolean shouldPrefetch = mPolicy.shouldPrefetchSearchResult(isTap);
             mSearchRequest = new ContextualSearchRequest(mSelectionController.getSelectedText(),
                     null, shouldPrefetch);
-            // TODO(donnd): figure out a way to do translation on long-press selections.
+            forceAutoDetectTranslateUnlessDisabled(mSearchRequest);
             mDidStartLoadingResolvedSearchRequest = false;
             mSearchPanel.displaySearchTerm(mSelectionController.getSelectedText());
             if (shouldPrefetch) loadSearchUrl();
         }
 
         if (!didRequestSurroundings) {
             // Gather surrounding text for Icing integration, which will make the selection and
             // a shorter version of the surroundings available for Conversational Search.
             // Although the surroundings are extracted, they will not be sent to the server as
             // part of search term resolution, just sent to Icing which keeps them local until
@@ skipping 225 matching lines @@
             searchTerm = mSelectionController.getSelectedText();
             alternateTerm = null;
             doPreventPreload = true;
         }
         if (!TextUtils.isEmpty(searchTerm)) {
             // TODO(donnd): Instead of preloading, we should prefetch (ie the URL should not
             // appear in the user's history until the user views it).  See crbug.com/406446.
             boolean shouldPreload = !doPreventPreload && mPolicy.shouldPrefetchSearchResult(true);
             mSearchRequest = new ContextualSearchRequest(searchTerm, alternateTerm, shouldPreload);
             // Trigger translation, if enabled.
-            if (!contextLanguage.isEmpty()) {
-                if (mPolicy.needsTranslation(contextLanguage, getReadableLanguages())) {
-                    boolean doForceTranslate = !mPolicy.disableForceTranslationOnebox();
-                    if (doForceTranslate) {
-                        mSearchRequest.forceTranslation(contextLanguage,
-                                mPolicy.bestTargetLanguage(getWritableLanguages()));
-                    }
-                    ContextualSearchUma.logTranslateOnebox(doForceTranslate);
-                }
-            }
+            forceTranslateIfNeeded(mSearchRequest, contextLanguage);
             mDidStartLoadingResolvedSearchRequest = false;
             if (mSearchPanel.isContentShowing()) {
                 mSearchRequest.setNormalPriority();
             }
             if (mSearchPanel.isContentShowing() || shouldPreload) {
                 loadSearchUrl();
             }
             mPolicy.logSearchTermResolutionDetails(searchTerm,
                     mNetworkCommunicator.getBasePageUrl());
         }
@@ skipping 29 matching lines @@
      */
     private boolean isTapSupported() {
         // Base page just started navigating away, so taps should be ignored.
         if (mDidBasePageLoadJustStart) return false;
 
         return mPolicy.isTapSupported();
     }
 
     // ============================================================================================
     // Translation support
+    // TODO(donnd): move to a separate file.
     // ============================================================================================
 
     /**
      * Gets the list of readable languages for the current user, with the first
      * item in the list being the user's primary language.
      * We assume that the user can read all languages that they can write.
      * @return The {@link List} of languages the user understands with their primary language first.
      */
     private List<String> getReadableLanguages() {
         // May be cached.
@@ skipping 54 matching lines @@
      * @return The given locale as a language code.
      */
     private String trimLocaleToLanguage(String locale) {
         // TODO(donnd): use getScript or getLanguageTag (both API 21), or some other standard way to
         // strip the country, instead of hard-coding the two character language code.
         // TODO(donnd): Shouldn't getLanguage() do this?
         String trimmedLocale = locale.substring(0, 2);
         return new Locale(trimmedLocale).getLanguage();
     }
 
+    /**
+     * Force translation from the given language for the current search request,
+     * unless disabled by experiment.  Also log whenever conditions are right to translate.
+     * @param searchRequest The search request to force translation upon.
+     * @param sourceLanguage The language to translate from, or an empty string if not known.
+     */
+    private void forceTranslateIfNeeded(ContextualSearchRequest searchRequest,
+            String sourceLanguage) {
+        if (!TextUtils.isEmpty(sourceLanguage)) {
+            if (mPolicy.needsTranslation(sourceLanguage, getReadableLanguages())) {
+                boolean doForceTranslate = !mPolicy.disableForceTranslationOnebox();
+                if (doForceTranslate && searchRequest != null) {
+                    searchRequest.forceTranslation(sourceLanguage,
+                            mPolicy.bestTargetLanguage(getWritableLanguages()));
+                }
+                // Log that conditions were right for translation, even though it may be disabled
+                // for an experiment so we can compare with the counter factual data.
+                ContextualSearchUma.logTranslateOnebox(doForceTranslate);
+            }
+        }
+    }
+
+    /**
+     * Force auto-detect translation for the current search request unless disabled by experiment.
+     * Also log that conditions are right to translate.
+     * @param searchRequest The search request to force translation upon.
+     */
+    private void forceAutoDetectTranslateUnlessDisabled(ContextualSearchRequest searchRequest) {
+        // Always trigger translation using auto-detect when we're not resolving,
+        // unless disabled by policy.
+        boolean shouldAutoDetectTranslate = !mPolicy.disableAutoDetectTranslationOnebox();
+        if (shouldAutoDetectTranslate && searchRequest != null) {
+            // The translation one-box won't actually show when the source text ends up being
+            // the same as the target text, so we err on over-triggering.
+            searchRequest.forceAutoDetectTranslation(
+                    mPolicy.bestTargetLanguage(getWritableLanguages()));
+        }
+        // Log that conditions were right for translation, even though it may be disabled
+        // for an experiment so we can compare with the counter factual data.
+        ContextualSearchUma.logTranslateOnebox(shouldAutoDetectTranslate);
+    }
+
     // ============================================================================================
     // OverlayContentDelegate
     // ============================================================================================
 
     @Override
     public OverlayContentDelegate getOverlayContentDelegate() {
         return new SearchOverlayContentDelegate();
     }
 
     /**
@@ skipping 409 matching lines @@
     private native void nativeDestroy(long nativeContextualSearchManager);
     private native void nativeStartSearchTermResolutionRequest(long nativeContextualSearchManager,
             String selection, boolean useResolvedSearchTerm, ContentViewCore baseContentViewCore,
             boolean maySendBasePageUrl);
     private native void nativeGatherSurroundingText(long nativeContextualSearchManager,
             String selection, boolean useResolvedSearchTerm, ContentViewCore baseContentViewCore,
             boolean maySendBasePageUrl);
     private native String nativeGetTargetLanguage(long nativeContextualSearchManager);
     private native String nativeGetAcceptLanguages(long nativeContextualSearchManager);
 }