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

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

Issue 5774004: Make HistoryContentsProvider results deletable (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Move DeleteMatch to common base class, update includes, etc. Created 10 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: chrome/browser/autocomplete/history_contents_provider.cc
diff --git a/chrome/browser/autocomplete/history_contents_provider.cc b/chrome/browser/autocomplete/history_contents_provider.cc
index 5b14a3be4568f0d19351b4f2c77c1991974e14e8..37f314403dacfac7b095bc3d8e9689208385aac5 100644
--- a/chrome/browser/autocomplete/history_contents_provider.cc
+++ b/chrome/browser/autocomplete/history_contents_provider.cc
@@ -53,7 +53,7 @@ using history::HistoryDatabase;
HistoryContentsProvider::HistoryContentsProvider(ACProviderListener* listener,
Profile* profile)
- : AutocompleteProvider(listener, profile, "HistoryContents"),
+ : HistoryProvider(listener, profile, "HistoryContents"),
star_title_count_(0),
star_contents_count_(0),
title_count_(0),
@@ -222,7 +222,7 @@ AutocompleteMatch HistoryContentsProvider::ResultToMatch(
int score) {
// TODO(sky): if matched title highlight matching words in title.
// Also show star in popup.
- AutocompleteMatch match(this, score, false, MatchInTitle(result) ?
+ AutocompleteMatch match(this, score, true, MatchInTitle(result) ?
AutocompleteMatch::HISTORY_TITLE : AutocompleteMatch::HISTORY_BODY);
match.contents = StringForURLDisplay(result.url(), true, trim_http_);
match.fill_into_edit =

Powered by Google App Engine
This is Rietveld 408576698