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

Side by Side Diff: components/omnibox/browser/omnibox_field_trial.h

Issue 1224083010: Enable Answers in Suggest always. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix Android compile issue Created 5 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef COMPONENTS_OMNIBOX_BROWSER_OMNIBOX_FIELD_TRIAL_H_ 5 #ifndef COMPONENTS_OMNIBOX_BROWSER_OMNIBOX_FIELD_TRIAL_H_
6 #define COMPONENTS_OMNIBOX_BROWSER_OMNIBOX_FIELD_TRIAL_H_ 6 #define COMPONENTS_OMNIBOX_BROWSER_OMNIBOX_FIELD_TRIAL_H_
7 7
8 #include <map> 8 #include <map>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
(...skipping 222 matching lines...) Expand 10 before | Expand all | Expand 10 after
233 // --------------------------------------------------------- 233 // ---------------------------------------------------------
234 // For the HQPAllowMatchInScheme experiment that's part of the 234 // For the HQPAllowMatchInScheme experiment that's part of the
235 // bundled omnibox field trial. 235 // bundled omnibox field trial.
236 236
237 // Returns true if HQP should allow an input term to match in the 237 // Returns true if HQP should allow an input term to match in the
238 // scheme (e.g., http://) of a URL. Returns false if the allow 238 // scheme (e.g., http://) of a URL. Returns false if the allow
239 // match in scheme experiment isn't active. 239 // match in scheme experiment isn't active.
240 static bool HQPAllowMatchInSchemeValue(); 240 static bool HQPAllowMatchInSchemeValue();
241 241
242 // --------------------------------------------------------- 242 // ---------------------------------------------------------
243 // For the AnswersInSuggest experiment that's part of the bundled omnibox
244 // field trial.
245
246 // Returns true if the AnswersInSuggest feature should be enabled causing
247 // query responses such as current weather conditions or stock quotes
248 // to be provided in the Omnibox suggestion list. Considers both the
249 // field trial state as well as the overriding command-line flags.
250 static bool EnableAnswersInSuggest();
251
252 // ---------------------------------------------------------
253 // For SearchProvider related experiments. 243 // For SearchProvider related experiments.
254 244
255 // Returns true if the search provider should not be caching results. 245 // Returns true if the search provider should not be caching results.
256 static bool DisableResultsCaching(); 246 static bool DisableResultsCaching();
257 247
258 // Returns how the search provider should poll Suggest. Currently, we support 248 // Returns how the search provider should poll Suggest. Currently, we support
259 // measuring polling delay from the last keystroke or last suggest request. 249 // measuring polling delay from the last keystroke or last suggest request.
260 static void GetSuggestPollingStrategy(bool* from_last_keystroke, 250 static void GetSuggestPollingStrategy(bool* from_last_keystroke,
261 int* polling_delay_ms); 251 int* polling_delay_ms);
262 252
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
327 static const char kDisableProvidersRule[]; 317 static const char kDisableProvidersRule[];
328 static const char kShortcutsScoringMaxRelevanceRule[]; 318 static const char kShortcutsScoringMaxRelevanceRule[];
329 static const char kSearchHistoryRule[]; 319 static const char kSearchHistoryRule[];
330 static const char kDemoteByTypeRule[]; 320 static const char kDemoteByTypeRule[];
331 static const char kHQPBookmarkValueRule[]; 321 static const char kHQPBookmarkValueRule[];
332 static const char kHQPDiscountFrecencyWhenFewVisitsRule[]; 322 static const char kHQPDiscountFrecencyWhenFewVisitsRule[];
333 static const char kHQPAllowMatchInTLDRule[]; 323 static const char kHQPAllowMatchInTLDRule[];
334 static const char kHQPAllowMatchInSchemeRule[]; 324 static const char kHQPAllowMatchInSchemeRule[];
335 static const char kZeroSuggestRule[]; 325 static const char kZeroSuggestRule[];
336 static const char kZeroSuggestVariantRule[]; 326 static const char kZeroSuggestVariantRule[];
337 static const char kAnswersInSuggestRule[];
338 static const char kDisableResultsCachingRule[]; 327 static const char kDisableResultsCachingRule[];
339 static const char kMeasureSuggestPollingDelayFromLastKeystrokeRule[]; 328 static const char kMeasureSuggestPollingDelayFromLastKeystrokeRule[];
340 static const char kSuggestPollingDelayMsRule[]; 329 static const char kSuggestPollingDelayMsRule[];
341 static const char kHQPFixFrequencyScoringBugsRule[]; 330 static const char kHQPFixFrequencyScoringBugsRule[];
342 static const char kHQPNumTitleWordsRule[]; 331 static const char kHQPNumTitleWordsRule[];
343 static const char kHQPAlsoDoHUPLikeScoringRule[]; 332 static const char kHQPAlsoDoHUPLikeScoringRule[];
344 static const char kPreventUWYTDefaultForNonURLInputsRule[]; 333 static const char kPreventUWYTDefaultForNonURLInputsRule[];
345 334
346 // Parameter names used by the HUP new scoring experiments. 335 // Parameter names used by the HUP new scoring experiments.
347 static const char kHUPNewScoringEnabledParam[]; 336 static const char kHUPNewScoringEnabledParam[];
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
381 // prioritize different wildcard contexts, see the implementation. How to 370 // prioritize different wildcard contexts, see the implementation. How to
382 // interpret the value is left to the caller; this is rule-dependent. 371 // interpret the value is left to the caller; this is rule-dependent.
383 static std::string GetValueForRuleInContext( 372 static std::string GetValueForRuleInContext(
384 const std::string& rule, 373 const std::string& rule,
385 metrics::OmniboxEventProto::PageClassification page_classification); 374 metrics::OmniboxEventProto::PageClassification page_classification);
386 375
387 DISALLOW_IMPLICIT_CONSTRUCTORS(OmniboxFieldTrial); 376 DISALLOW_IMPLICIT_CONSTRUCTORS(OmniboxFieldTrial);
388 }; 377 };
389 378
390 #endif // COMPONENTS_OMNIBOX_BROWSER_OMNIBOX_FIELD_TRIAL_H_ 379 #endif // COMPONENTS_OMNIBOX_BROWSER_OMNIBOX_FIELD_TRIAL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698