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

Unified Diff: chrome/browser/history/scored_history_match.cc

Issue 10782017: Make Omnibox's HistoryQuickProvider's new scoring less aggressive (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/history/scored_history_match.cc
diff --git a/chrome/browser/history/scored_history_match.cc b/chrome/browser/history/scored_history_match.cc
index d71b9740811162c2d8a35be779587e368843cbd9..dee31c81b6ba49cffba66d8845b2f57b75755b40 100644
--- a/chrome/browser/history/scored_history_match.cc
+++ b/chrome/browser/history/scored_history_match.cc
@@ -119,13 +119,11 @@ ScoredHistoryMatch::ScoredHistoryMatch(const URLRow& row,
// Combine recency, popularity, and topicality scores into one.
// Example of how this functions: Suppose the omnibox has one
- // input term. Suppose we have a URL that has 8 typed visits with
+ // input term. Suppose we have a URL that has 30 typed visits with
// the most recent being within a day and the omnibox input term
// has a single URL hostname hit at a word boundary. Then this
- // URL will score 1300 ( = 8 * 162.5), which is exactly the value of
- // search what you type. That is, it's the boundary of what might
- // end up being inlined.
- raw_score = 162.5 * topicality_score * recency_score * popularity_score;
+ // URL will score 1200 ( = 30 * 40.0).
+ raw_score = 40.0 * topicality_score * recency_score * popularity_score;
raw_score =
(raw_score <= kint32max) ? static_cast<int>(raw_score) : kint32max;
} else { // "old" scoring
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698