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

Unified Diff: chrome/browser/history/url_index_private_data.h

Issue 11757004: Omnibox: Add Mid-Input Matching to HistoryQuick Provider (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 12 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/history/url_index_private_data.h
diff --git a/chrome/browser/history/url_index_private_data.h b/chrome/browser/history/url_index_private_data.h
index f32b4abff938da2ad34ef660b21259d22eb81ab4..a19459412eff85b09228612efce93d028cd230f5 100644
--- a/chrome/browser/history/url_index_private_data.h
+++ b/chrome/browser/history/url_index_private_data.h
@@ -50,19 +50,22 @@ class URLIndexPrivateData
// vector with all scored, matching history items. The |term_string| is
// broken down into individual terms (words), each of which must occur in the
// candidate history item's URL or page title for the item to qualify;
- // however, the terms do not necessarily have to be adjacent. Once we have
- // a set of candidates, they are filtered to insure that all |term_string|
- // terms, as separated by whitespace, occur within the candidate's URL
- // or page title. Scores are then calculated on no more than
- // |kItemsToScoreLimit| candidates, as the scoring of such a large number of
- // candidates may cause perceptible typing response delays in the omnibox.
- // This is likely to occur for short omnibox terms such as 'h' and 'w' which
+ // however, the terms do not necessarily have to be adjacent. We
+ // also allow breaking |term_string| at |cursor_position| (if
+ // set). Once we have a set of candidates, they are filtered to insure
Peter Kasting 2013/01/04 20:04:32 Nit: ensure
Mark P 2013/01/04 23:38:58 Done.
+ // that all |term_string| terms, as separated by whitespace and the
+ // cursor (if set), occur within the candidate's URL or page title.
+ // Scores are then calculated on no more than |kItemsToScoreLimit|
+ // candidates, as the scoring of such a large number of candidates may
+ // cause perceptible typing response delays in the omnibox. This is
+ // likely to occur for short omnibox terms such as 'h' and 'w' which
// will be found in nearly all history candidates. Results are sorted by
// descending score. The full results set (i.e. beyond the
// |kItemsToScoreLimit| limit) will be retained and used for subsequent calls
// to this function. |bookmark_service| is used to boost a result's score if
// its URL is referenced by one or more of the user's bookmarks.
ScoredHistoryMatches HistoryItemsForTerms(const string16& term_string,
+ size_t cursor_position,
BookmarkService* bookmark_service);
// Adds the history item in |row| to the index if it does not already already

Powered by Google App Engine
This is Rietveld 408576698