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

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

Issue 455563002: Prefetch cache pt1 - MRU (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@pfq-base
Patch Set: Get rid of C++11 features... :( Created 6 years, 4 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/search_provider.cc
diff --git a/chrome/browser/autocomplete/search_provider.cc b/chrome/browser/autocomplete/search_provider.cc
index c059c6ec2030c57e826980c33907b53beb5e7646..4dca4d4f6aacc24304378a6a96706e03250f4b73 100644
--- a/chrome/browser/autocomplete/search_provider.cc
+++ b/chrome/browser/autocomplete/search_provider.cc
@@ -136,7 +136,8 @@ SearchProvider::SearchProvider(AutocompleteProviderListener* listener,
AutocompleteProvider::TYPE_SEARCH),
listener_(listener),
suggest_results_pending_(0),
- providers_(template_url_service) {
+ providers_(template_url_service),
+ answers_cache_(1) {
}
// static
@@ -669,9 +670,9 @@ net::URLFetcher* SearchProvider::CreateSuggestFetcher(
search_term_args.session_token = GetSessionToken();
if (!prefetch_data_.full_query_text.empty()) {
search_term_args.prefetch_query =
- base::UTF16ToUTF8(last_answer_seen_.full_query_text);
+ base::UTF16ToUTF8(prefetch_data_.full_query_text);
search_term_args.prefetch_query_type =
- base::UTF16ToUTF8(last_answer_seen_.query_type);
+ base::UTF16ToUTF8(prefetch_data_.query_type);
}
}
GURL suggest_url(template_url->suggestions_url_ref().ReplaceSearchTerms(
@@ -1256,14 +1257,9 @@ void SearchProvider::RegisterDisplayedAnswers(
return;
// Valid answer encountered, cache it for further queries.
- last_answer_seen_.full_query_text = match->fill_into_edit;
- last_answer_seen_.query_type = match->answer_type;
+ answers_cache_.UpdateRecentAnswers(match->fill_into_edit, match->answer_type);
}
void SearchProvider::DoAnswersQuery(const AutocompleteInput& input) {
- // If the query text starts with trimmed input, this is valid prefetch data.
- prefetch_data_ = StartsWith(last_answer_seen_.full_query_text,
- base::CollapseWhitespace(input.text(), false),
- false) ?
- last_answer_seen_ : AnswersQueryData();
+ prefetch_data_ = answers_cache_.GetTopAnswerEntry(input.text());
}
« no previous file with comments | « chrome/browser/autocomplete/search_provider.h ('k') | chrome/browser/autocomplete/search_provider_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698