Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1708)

Unified Diff: chrome/android/java_staging/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchFieldTrial.java

Issue 1141283003: Upstream oodles of Chrome for Android code into Chromium. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: final patch? Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java_staging/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchFieldTrial.java
diff --git a/chrome/android/java_staging/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchFieldTrial.java b/chrome/android/java_staging/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchFieldTrial.java
new file mode 100644
index 0000000000000000000000000000000000000000..810d4b1dc66cf0843673b390ef04de77efc88055
--- /dev/null
+++ b/chrome/android/java_staging/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchFieldTrial.java
@@ -0,0 +1,241 @@
+// 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.content.Context;
+import android.text.TextUtils;
+
+import org.chromium.base.CommandLine;
+import org.chromium.base.SysUtils;
+import org.chromium.chrome.browser.ChromeSwitches;
+import org.chromium.chrome.browser.ChromeVersionInfo;
+import org.chromium.components.variations.VariationsAssociatedData;
+import org.chromium.ui.base.DeviceFormFactor;
+
+import java.util.Arrays;
+import java.util.Locale;
+
+/**
+ * Provides Field Trial support for the Contextual Search application within Chrome for Android.
+ */
+public class ContextualSearchFieldTrial {
+
+ private static final String CONTEXTUAL_SEARCH_FIELD_TRIAL_NAME = "ContextualSearch";
+ private static final String CONTEXTUAL_SEARCH_ENABLED_PARAM = "enabled";
+ private static final String CONTEXTUAL_SEARCH_ENABLED_VALUE = "true";
+ private static final String CONTEXTUAL_SEARCH_DISABLE_FOR_CJK = "disable_for_cjk";
+ private static final String CONTEXTUAL_SEARCH_DISABLE_FOR_NON_ENGLISH =
+ "disable_for_non_english";
+ private static final String CONTEXTUAL_SEARCH_DISABLE_SURROUNDINGS_OBSERVERS =
+ "disable_surroundings_observers";
+ private static final String CONTEXTUAL_SEARCH_PROMO_ON_LONGPRESS_ONLY =
+ "promo_on_longpress_only";
+ static final String CONTEXTUAL_SEARCH_PROMO_ON_LIMITED_TAPS =
+ "promo_on_limited_taps";
+ static final String CONTEXTUAL_SEARCH_TAP_TRIGGERED_PROMO_LIMIT =
+ "tap_triggered_promo_limit";
+ private static final String CONTEXTUAL_SEARCH_NAVIGATION_DETECTION_DELAY =
+ "tap_navigation_detection_delay";
+ static final String CONTEXTUAL_SEARCH_TAP_RESOLVE_LIMIT_FOR_DECIDED =
+ "tap_resolve_limit_for_decided";
+ static final String CONTEXTUAL_SEARCH_TAP_PREFETCH_LIMIT_FOR_DECIDED =
+ "tap_prefetch_limit_for_decided";
+ static final String CONTEXTUAL_SEARCH_TAP_RESOLVE_LIMIT_FOR_UNDECIDED =
+ "tap_resolve_limit_for_undecided";
+ static final String CONTEXTUAL_SEARCH_TAP_PREFETCH_LIMIT_FOR_UNDECIDED =
+ "tap_prefetch_limit_for_undecided";
+
+ private static final String[] CJK_LANGUAGE_CODES = {"zh", "ja", "ko"};
+ private static final String ENGLISH_LANGUAGE_CODE = "en";
+
+ // The default navigation-detection-delay in milliseconds.
+ private static final int DEFAULT_TAP_NAVIGATION_DETECTION_DELAY = 16;
+
+ static final int UNLIMITED_TAPS = -1;
+ private static final int DEFAULT_TAP_RESOLVE_LIMIT_FOR_DECIDED = UNLIMITED_TAPS;
+ private static final int DEFAULT_TAP_PREFETCH_LIMIT_FOR_DECIDED = UNLIMITED_TAPS;
+ private static final int DEFAULT_TAP_RESOLVE_LIMIT_FOR_UNDECIDED = 100;
+ private static final int DEFAULT_TAP_PREFETCH_LIMIT_FOR_UNDECIDED = 10;
+
+ // Cached value to avoid repeated and redundant JNI operations.
+ private static Boolean sEnabled;
+
+ /**
+ * Don't instantiate.
+ */
+ private ContextualSearchFieldTrial() {}
+
+ /**
+ * Checks the current Variations parameters associated with the active group as well as the
+ * Chrome preference to determine if the service is enabled.
+ * @return Whether Contextual Search is enabled or not.
+ */
+ public static boolean isEnabled(Context context) {
+ if (sEnabled == null) {
+ sEnabled = detectEnabled(context);
+ }
+ return sEnabled.booleanValue();
+ }
+
+ private static boolean detectEnabled(Context context) {
+ if (DeviceFormFactor.isTablet(context) || SysUtils.isLowEndDevice()) {
+ return false;
+ }
+
+ // This is used for instrumentation tests (i.e. it is not a user-flippable flag). We cannot
+ // use Variations params because in the test harness, the initialization comes before any
+ // native methods are available. And the ContextualSearchManager is initialized very early
+ // in the Chrome initialization.
+ if (CommandLine.getInstance().hasSwitch(
+ ChromeSwitches.ENABLE_CONTEXTUAL_SEARCH_FOR_TESTING)) {
+ return true;
+ }
+
+ // Allow this user-flippable flag to disable the feature.
+ if (CommandLine.getInstance().hasSwitch(ChromeSwitches.DISABLE_CONTEXTUAL_SEARCH)) {
+ return false;
+ }
+
+ // Allow this user-flippable flag to override disabling due to language.
+ if (CommandLine.getInstance().hasSwitch(ChromeSwitches.ENABLE_CONTEXTUAL_SEARCH)) {
+ return true;
+ }
+
+ String languageCode = Locale.getDefault().getLanguage();
+ if (getBooleanParam(CONTEXTUAL_SEARCH_DISABLE_FOR_NON_ENGLISH)
+ && !languageCode.equals(ENGLISH_LANGUAGE_CODE)) {
+ return false;
+ }
+
+ if (getBooleanParam(CONTEXTUAL_SEARCH_DISABLE_FOR_CJK)
+ && Arrays.asList(CJK_LANGUAGE_CODES).contains(languageCode)) {
+ return false;
+ }
+
+ if (ChromeVersionInfo.isLocalBuild()) return true;
+
+ return getBooleanParam(CONTEXTUAL_SEARCH_ENABLED_PARAM);
+ }
+
+ /**
+ * @return Whether the promo is configured for Opt-out.
+ */
+ static boolean isPromoOptOut() {
+ return true;
+ }
+
+ /**
+ * Gets whether the promo should be triggered on longpress only.
+ * @return {@code true} iff Finch says we should trigger the promo only on touch-and-hold.
+ */
+ static boolean isPromoLongpressTriggeredOnly() {
+ return getBooleanParam(CONTEXTUAL_SEARCH_PROMO_ON_LONGPRESS_ONLY);
+ }
+
+ /**
+ * @return Whether the promo should be triggered by a limited number of taps.
+ */
+ public static boolean isPromoLimitedByTapCounts() {
+ return getBooleanParam(CONTEXTUAL_SEARCH_PROMO_ON_LIMITED_TAPS);
+ }
+
+ /**
+ * @return The maximum number of times the promo can be triggered by a tap, or
+ * {@code ContextualSearchUma#PROMO_TAPS_REMAINING_INVALID} if no value is present in the finch
+ * configuration.
+ */
+ static int getPromoTapTriggeredLimit() {
+ return getIntParamValueOrDefault(CONTEXTUAL_SEARCH_TAP_TRIGGERED_PROMO_LIMIT,
+ UNLIMITED_TAPS);
+ }
+
+ /**
+ * @return The delay to use for navigation-detection when triggering on a Tap.
+ */
+ static int getNavigationDetectionDelay() {
+ return getIntParamValueOrDefault(CONTEXTUAL_SEARCH_NAVIGATION_DETECTION_DELAY,
+ DEFAULT_TAP_NAVIGATION_DETECTION_DELAY);
+ }
+
+ /**
+ * @return The limit on the number of taps to resolve for decided users, or the default if no
+ * value is present in the Finch configuration.
+ */
+ static int getTapResolveLimitForDecided() {
+ return getIntParamValueOrDefault(CONTEXTUAL_SEARCH_TAP_RESOLVE_LIMIT_FOR_DECIDED,
+ DEFAULT_TAP_RESOLVE_LIMIT_FOR_DECIDED);
+ }
+
+ /**
+ * @return The limit on the number of prefetches to issue for decided users, or the default
+ * if no value is present.
+ */
+ static int getTapPrefetchLimitForDecided() {
+ return getIntParamValueOrDefault(CONTEXTUAL_SEARCH_TAP_PREFETCH_LIMIT_FOR_DECIDED,
+ DEFAULT_TAP_PREFETCH_LIMIT_FOR_DECIDED);
+ }
+
+ /**
+ * @return The limit on the number of taps to resolve for undecided users, or the default if no
+ * value is present in the Finch configuration.
+ */
+ static int getTapResolveLimitForUndecided() {
+ return getIntParamValueOrDefault(CONTEXTUAL_SEARCH_TAP_RESOLVE_LIMIT_FOR_UNDECIDED,
+ DEFAULT_TAP_RESOLVE_LIMIT_FOR_UNDECIDED);
+ }
+
+ /**
+ * @return The limit on the number of prefetches to issue for undecided users, or the default
+ * if no value is present.
+ */
+ static int getTapPrefetchLimitForUndecided() {
+ return getIntParamValueOrDefault(CONTEXTUAL_SEARCH_TAP_PREFETCH_LIMIT_FOR_UNDECIDED,
+ DEFAULT_TAP_PREFETCH_LIMIT_FOR_UNDECIDED);
+ }
+
+ // --------------------------------------------------------------------------------------------
+ // Helpers.
+ // --------------------------------------------------------------------------------------------
+
+ /**
+ * Gets a boolean Finch parameter, assuming the <paramName>="true" format. Also checks for a
+ * command-line switch with the same name, for easy local testing.
+ * @param paramName The name of the Finch parameter (or command-line switch) to get a value for.
+ * @return Whether the Finch param is defined with a value "true", if there's a command-line
+ * flag present with any value.
+ */
+ private static boolean getBooleanParam(String paramName) {
+ if (CommandLine.getInstance().hasSwitch(paramName)) {
+ return true;
+ }
+ return TextUtils.equals(CONTEXTUAL_SEARCH_ENABLED_VALUE,
+ VariationsAssociatedData.getVariationParamValue(CONTEXTUAL_SEARCH_FIELD_TRIAL_NAME,
+ paramName));
+ }
+
+ /**
+ * Returns an integer value for a Finch parameter, or the default value if no parameter exists
+ * in the current configuration. Also checks for a command-line switch with the same name.
+ * @param paramName The name of the Finch parameter (or command-line switch) to get a value for.
+ * @param defaultValue The default value to return when there's no param or switch.
+ * @return An integer value -- either the param or the default.
+ */
+ private static int getIntParamValueOrDefault(String paramName, int defaultValue) {
+ String value = CommandLine.getInstance().getSwitchValue(paramName);
+ if (TextUtils.isEmpty(value)) {
+ value = VariationsAssociatedData.getVariationParamValue(
+ CONTEXTUAL_SEARCH_FIELD_TRIAL_NAME, paramName);
+ }
+ if (!TextUtils.isEmpty(value)) {
+ try {
+ return Integer.parseInt(value);
+ } catch (NumberFormatException e) {
+ return defaultValue;
+ }
+ }
+
+ return defaultValue;
+ }
+}

Powered by Google App Engine
This is Rietveld 408576698