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

Unified Diff: components/omnibox/omnibox_field_trial.h

Issue 645303003: Parametrize Suggest polling strategy and delay. Also add a parameter (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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
« no previous file with comments | « no previous file | components/omnibox/omnibox_field_trial.cc » ('j') | components/omnibox/omnibox_field_trial.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/omnibox/omnibox_field_trial.h
diff --git a/components/omnibox/omnibox_field_trial.h b/components/omnibox/omnibox_field_trial.h
index 07377bdc7052aa8085c479d91334ed37bdb3c757..b0d89c7b1cba82982041608054f38dc7549b891b 100644
--- a/components/omnibox/omnibox_field_trial.h
+++ b/components/omnibox/omnibox_field_trial.h
@@ -281,6 +281,17 @@ class OmniboxFieldTrial {
static bool DisplayHintTextWhenPossible();
// ---------------------------------------------------------
+ // For Search Provider related experiments.
Mark P 2014/10/21 21:28:22 nit: SearchProvider
Bart N. 2014/10/22 01:19:08 Done.
+
+ // Returns true if the search provider should not be caching results.
+ static bool DisableResultsCaching();
+
+ // Returns how the search provider should poll Suggest. Currently, we support
+ // measuring polling delay since the last keystroke or last suggest request.
+ static void GetSuggestPollingStrategy(bool* since_the_last_request,
+ int* polling_delay_ms);
+
+ // ---------------------------------------------------------
// Exposed publicly for the sake of unittests.
static const char kBundledExperimentFieldTrialName[];
// Rule names used by the bundled experiment.
@@ -297,6 +308,10 @@ class OmniboxFieldTrial {
static const char kAnswersInSuggestRule[];
static const char kAddUWYTMatchEvenIfPromotedURLsRule[];
static const char kDisplayHintTextWhenPossibleRule[];
+ // Search Provider experiments.
Mark P 2014/10/21 21:28:22 nit: comment unnecessary (we omit all similar comm
Bart N. 2014/10/22 01:19:08 Done.
+ static const char kDisableResultsCachingRule[];
+ static const char kSuggestPollingDelaySinceTheLastRequestRule[];
Mark P 2014/10/21 21:28:22 nit: I'd prefer this rule name had a verb such as
Bart N. 2014/10/22 01:19:08 Done.
+ static const char kSuggestPollingDelayMsRule[];
// Parameter names used by the HUP new scoring experiments.
static const char kHUPNewScoringEnabledParam[];
« no previous file with comments | « no previous file | components/omnibox/omnibox_field_trial.cc » ('j') | components/omnibox/omnibox_field_trial.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698