Add Yandex referral ID to search URL.

chrome/android/java/src/org/chromium/chrome/browser/locale/LocaleManager.java

 // Copyright 2016 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.locale;
 
 import android.content.Context;
 import android.content.Intent;
 import android.content.SharedPreferences;
 import android.support.annotation.IntDef;
 import android.support.annotation.Nullable;
 
 import org.chromium.base.Callback;
 import org.chromium.base.ContextUtils;
 import org.chromium.base.ThreadUtils;
 import org.chromium.base.VisibleForTesting;
+import org.chromium.base.annotations.CalledByNative;
 import org.chromium.chrome.R;
 import org.chromium.chrome.browser.AppHooks;
 import org.chromium.chrome.browser.ChromeFeatureList;
 import org.chromium.chrome.browser.preferences.PreferencesLauncher;
 import org.chromium.chrome.browser.preferences.SearchEnginePreference;
 import org.chromium.chrome.browser.search_engines.TemplateUrlService;
 import org.chromium.chrome.browser.snackbar.Snackbar;
 import org.chromium.chrome.browser.snackbar.SnackbarManager;
 import org.chromium.chrome.browser.snackbar.SnackbarManager.SnackbarController;
 
@@ skipping 51 matching lines @@
             Context context = ContextUtils.getApplicationContext();
             Intent intent = PreferencesLauncher.createIntentForSettingsPage(context,
                     SearchEnginePreference.class.getName());
             context.startActivity(intent);
         }
     };
 
     /**
      * @return An instance of the {@link LocaleManager}. This should only be called on UI thread.
      */
+    @CalledByNative
     public static LocaleManager getInstance() {
         assert ThreadUtils.runningOnUiThread();
         if (sInstance == null) {
             sInstance = AppHooks.get().createLocaleManager();
         }
         return sInstance;
     }
 
     /**
      * Starts listening to state changes of the phone.
@@ skipping 167 matching lines @@
     public int getSearchEnginePromoShowType() {
         if (!isSpecialLocaleEnabled()) return SEARCH_ENGINE_PROMO_DONT_SHOW;
         SharedPreferences preferences = ContextUtils.getAppSharedPreferences();
         if (preferences.getBoolean(PREF_PROMO_SHOWN, false)) {
             return SEARCH_ENGINE_PROMO_DONT_SHOW;
         }
         return SEARCH_ENGINE_PROMO_SHOW_SOGOU;
     }
 
     /**
+     * @return The referral ID to be passed when searching with Yandex as the DSE.
+     */
+    @CalledByNative
+    protected String getYandexReferralId() {

[Yusuf, 2017/05/09 23:18:52]

should this be a static, so that we avoid calling into Java in two separate
methods in the native side? Then we don't have to get the instance etc.

[Ted C, 2017/05/10 15:20:29]

I thought about that as well, but then it would prevent the subclass from
overwriting it as you can do that with statics.

+        return "";
+    }
+
+    /**
      * To be called after the user has made a selection from a search engine promo dialog.
      * @param type The type of search engine promo dialog that was shown.
      * @param keyword The keyword for the search engine chosen.
      */
     protected void onUserSearchEngineChoiceFromPromoDialog(
             @SearchEnginePromoType int type, String keyword) {
         TemplateUrlService.getInstance().setSearchEngine(keyword);
         ContextUtils.getAppSharedPreferences()
                 .edit()
                 .putInt(KEY_SEARCH_ENGINE_PROMO_SHOW_STATE, SEARCH_ENGINE_PROMO_CHECKED_AND_SHOWN)
                 .apply();
     }
 
     private SpecialLocaleHandler getSpecialLocaleHandler() {
         if (mLocaleHandler == null) mLocaleHandler = new SpecialLocaleHandler(getSpecialLocaleId());
         return mLocaleHandler;
     }
 
     /** Set a LocaleManager to be used for testing. */
     @VisibleForTesting
     public static void setInstanceForTest(LocaleManager instance) {
         assert sInstance == null;
         sInstance = instance;
     }
 }