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

Unified Diff: components/omnibox/browser/omnibox_field_trial.h

Issue 2541143002: Omnibox - Boost Frequency Scores Based on Number of Matching Pages (Closed)
Patch Set: improved comments and formatting Created 4 years 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: components/omnibox/browser/omnibox_field_trial.h
diff --git a/components/omnibox/browser/omnibox_field_trial.h b/components/omnibox/browser/omnibox_field_trial.h
index c4c6afe7d1a2e68a7720a87ff9c0894b9d979ab5..3cb96e29610f3db2feb16af2d4cedf5e9ceb7cf5 100644
--- a/components/omnibox/browser/omnibox_field_trial.h
+++ b/components/omnibox/browser/omnibox_field_trial.h
@@ -103,6 +103,15 @@ class OmniboxFieldTrial {
// given number. Omitted types are assumed to have multipliers of 1.0.
typedef std::map<AutocompleteMatchType::Type, float> DemotionMultipliers;
+ // A vector that maps from the number of matching pages to the document
+ // specificity score used in HistoryQuick provider / ScoredHistoryMatch
+ // scoring. The vector is sorted by the size_t (the number of matching pages).
+ // If an entry is omitted, the appropriate value is assumed to be the one in
+ // the later bucket. For example, with a vector containing {{1, 2.0},
+ // {3, 1.5}}, the score for 2 is inferred to be 1.5. Values beyond the
+ // end of the vector are assumed to have scores of 1.0.
+ typedef std::vector<std::pair<size_t, double>> NumMatchesScores;
+
// Do not change these values as they need to be in sync with values
// specified in experiment configs on the variations server.
enum EmphasizeTitlesCondition {
@@ -309,6 +318,10 @@ class OmniboxFieldTrial {
// of non-typed transitions is 1.) Returns 20 if the experiment isn't active.
static float HQPTypedValue();
+ // Returns NumMatchesScores; see comment by the declaration of it.
+ // Returns an empty NumMatchesScores if the experiment isn't active.
+ static NumMatchesScores HQPNumMatchesScores();
+
// ---------------------------------------------------------
// For the HQPNumTitleWords experiment that's part of the
// bundled omnibox field trial.
@@ -389,6 +402,7 @@ class OmniboxFieldTrial {
static const char kHQPFixFewVisitsBugRule[];
static const char kHQPFreqencyUsesSumRule[];
static const char kHQPMaxVisitsToScoreRule[];
+ static const char kHQPNumMatchesScoresRule[];
static const char kHQPNumTitleWordsRule[];
static const char kHQPAlsoDoHUPLikeScoringRule[];
static const char kHUPSearchDatabaseRule[];
« no previous file with comments | « no previous file | components/omnibox/browser/omnibox_field_trial.cc » ('j') | components/omnibox/browser/omnibox_field_trial.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698