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

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

Issue 1877833002: Optimize shortcuts provider (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase on master and minor format fix Created 4 years, 8 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 | « components/omnibox/browser/autocomplete_result.cc ('k') | components/omnibox/browser/match_compare.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/omnibox/browser/history_url_provider_unittest.cc
diff --git a/components/omnibox/browser/history_url_provider_unittest.cc b/components/omnibox/browser/history_url_provider_unittest.cc
index a28f285ad735b356176b9fa863602ad46035e8b8..e1c1b19d27eebe313304d248440912fbc7e38151 100644
--- a/components/omnibox/browser/history_url_provider_unittest.cc
+++ b/components/omnibox/browser/history_url_provider_unittest.cc
@@ -332,10 +332,11 @@ void HistoryURLProviderTest::RunTest(
matches_ = autocomplete_->matches();
if (sort_matches_) {
TemplateURLService* service = client_->GetTemplateURLService();
- for (ACMatches::iterator i = matches_.begin(); i != matches_.end(); ++i)
+ for (ACMatches::iterator i = matches_.begin(); i != matches_.end(); ++i) {
i->ComputeStrippedDestinationURL(input, service);
- AutocompleteResult::DedupMatchesByDestination(
- input.current_page_classification(), false, &matches_);
+ }
+ AutocompleteResult::SortAndDedupMatches(input.current_page_classification(),
+ &matches_);
std::sort(matches_.begin(), matches_.end(),
&AutocompleteMatch::MoreRelevant);
}
« no previous file with comments | « components/omnibox/browser/autocomplete_result.cc ('k') | components/omnibox/browser/match_compare.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698