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

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

Issue 8526010: Improve Autocomplete Matches and Handling of Large Results Sets (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 1 month 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.cc
===================================================================
--- chrome/browser/autocomplete/history_quick_provider.cc (revision 112051)
+++ chrome/browser/autocomplete/history_quick_provider.cc (working copy)
@@ -59,11 +59,6 @@
autocomplete_input_ = input;
- // Do some fixup on the user input before matching against it, so we provide
- // good results for local file paths, input with spaces, etc.
- if (!FixupUserInput(&autocomplete_input_))
- return;
-
// TODO(pkasting): We should just block here until this loads. Any time
// someone unloads the history backend, we'll get inconsistent inline
// autocomplete behavior here.
@@ -88,11 +83,7 @@
void HistoryQuickProvider::DoAutocomplete() {
// Get the matching URLs from the DB.
string16 term_string = autocomplete_input_.text();
- term_string = net::UnescapeURLComponent(term_string,
- net::UnescapeRule::SPACES | net::UnescapeRule::URL_SPECIAL_CHARS);
- history::String16Vector terms(
- history::String16VectorFromString16(term_string, false));
- ScoredHistoryMatches matches = GetIndex()->HistoryItemsForTerms(terms);
+ ScoredHistoryMatches matches = GetIndex()->HistoryItemsForTerms(term_string);
if (matches.empty())
return;
« no previous file with comments | « no previous file | chrome/browser/history/in_memory_url_index.h » ('j') | chrome/browser/history/in_memory_url_index_unittest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698