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

Unified Diff: chrome/browser/autocomplete/history_quick_provider_unittest.cc

Issue 9317011: Fine-tune HQP Candidate Result Scoring (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 11 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: chrome/browser/autocomplete/history_quick_provider_unittest.cc
===================================================================
--- chrome/browser/autocomplete/history_quick_provider_unittest.cc (revision 119730)
+++ chrome/browser/autocomplete/history_quick_provider_unittest.cc (working copy)
@@ -47,7 +47,7 @@
{"http://foo.com/dir/", "Dir", 2, 1, 10},
{"http://foo.com/dir/another/", "Dir", 5, 1, 0},
{"http://foo.com/dir/another/again/", "Dir", 5, 1, 0},
- {"http://foo.com/dir/another/again/myfile.html", "File", 10, 2, 0},
+ {"http://foo.com/dir/another/again/myfile.html", "File", 3, 2, 0},
{"http://visitedest.com/y/a", "VA", 10, 1, 20},
{"http://visitedest.com/y/b", "VB", 9, 1, 20},
{"http://visitedest.com/x/c", "VC", 8, 1, 20},
@@ -263,7 +263,7 @@
// Scores high because of visit count.
expected_urls.push_back("http://foo.com/dir/another/");
// Scores high because of high visit count.
- expected_urls.push_back("http://foo.com/dir/another/again/myfile.html");
+ expected_urls.push_back("http://foo.com/dir/another/again/");
RunTest(ASCIIToUTF16("foo"), expected_urls, "http://foo.com/", true,
ASCIIToUTF16("foo.com"));
}

Powered by Google App Engine
This is Rietveld 408576698