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

Unified Diff: components/omnibox/browser/scored_history_match.cc

Issue 1286093006: Launch HQP & HUP score changes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 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: components/omnibox/browser/scored_history_match.cc
diff --git a/components/omnibox/browser/scored_history_match.cc b/components/omnibox/browser/scored_history_match.cc
index 0c058d6af9771d2250169ba573c4d335eaf18c4b..2ae28b38fe53d5c200efb3bebc78232022b21ff4 100644
--- a/components/omnibox/browser/scored_history_match.cc
+++ b/components/omnibox/browser/scored_history_match.cc
@@ -112,7 +112,11 @@ bool ScoredHistoryMatch::allow_tld_matches_ = false;
bool ScoredHistoryMatch::allow_scheme_matches_ = false;
size_t ScoredHistoryMatch::num_title_words_to_allow_ = 10u;
bool ScoredHistoryMatch::hqp_experimental_scoring_enabled_ = false;
-float ScoredHistoryMatch::topicality_threshold_ = -1;
+float ScoredHistoryMatch::topicality_threshold_ = 0.8f;
+// Default HQP relevance buckets. See GetFinalRelevancyScore()
+// for more details on these numbers.
+char ScoredHistoryMatch::hqp_relevance_buckets_str_[] =
+ "0.0:400,1.5:600,5.0:900,10.5:1203,15.0:1300,20.0:1399";
std::vector<ScoredHistoryMatch::ScoreMaxRelevance>*
ScoredHistoryMatch::hqp_relevance_buckets_ = nullptr;
@@ -540,8 +544,7 @@ float ScoredHistoryMatch::GetTopicalityScore(
const float final_topicality_score = topicality_score / num_terms;
// Demote the URL if the topicality score is less than threshold.
- if (hqp_experimental_scoring_enabled_ &&
- (final_topicality_score < topicality_threshold_)) {
+ if (final_topicality_score < topicality_threshold_) {
return 0.0;
}
@@ -645,7 +648,8 @@ float ScoredHistoryMatch::GetFinalRelevancyScore(
void ScoredHistoryMatch::InitHQPExperimentalParams() {
// These are default HQP relevance scoring buckets.
// See GetFinalRelevancyScore() for details.
- std::string hqp_relevance_buckets_str = "0.0:400,1.5:600,12.0:1300,20.0:1399";
+ std::string hqp_relevance_buckets_str = std::string(
+ hqp_relevance_buckets_str_);
// Fetch the experiment params if they are any.
hqp_experimental_scoring_enabled_ =
« no previous file with comments | « components/omnibox/browser/scored_history_match.h ('k') | components/omnibox/browser/scored_history_match_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698