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

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

Issue 1185723002: Prepare HistoryProvider for componentization (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Response to review Created 5 years, 6 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/zero_suggest_provider.cc
diff --git a/chrome/browser/autocomplete/zero_suggest_provider.cc b/chrome/browser/autocomplete/zero_suggest_provider.cc
index 812cef888ab6c5315f4e9b372046356d2a317471..4bc9c36e24441ca1af19f9f8d5138af6bb037169 100644
--- a/chrome/browser/autocomplete/zero_suggest_provider.cc
+++ b/chrome/browser/autocomplete/zero_suggest_provider.cc
@@ -76,10 +76,12 @@ const int kDefaultZeroSuggestRelevance = 100;
// static
ZeroSuggestProvider* ZeroSuggestProvider::Create(
+ AutocompleteProviderClient* client,
AutocompleteProviderListener* listener,
TemplateURLService* template_url_service,
Profile* profile) {
- return new ZeroSuggestProvider(listener, template_url_service, profile);
+ return new ZeroSuggestProvider(client, listener, template_url_service,
+ profile);
}
// static
@@ -121,7 +123,7 @@ void ZeroSuggestProvider::Start(const AutocompleteInput& input,
// most visited field trials.
if (CanSendURL(input.current_url(), suggest_url, default_provider,
current_page_classification_,
- template_url_service_->search_terms_data(), client_.get()) &&
+ template_url_service_->search_terms_data(), client_) &&
!OmniboxFieldTrial::InZeroSuggestPersonalizedFieldTrial() &&
!OmniboxFieldTrial::InZeroSuggestMostVisitedFieldTrial()) {
// Update suggest_url to include the current_page_url.
@@ -189,13 +191,14 @@ void ZeroSuggestProvider::ResetSession() {
}
ZeroSuggestProvider::ZeroSuggestProvider(
- AutocompleteProviderListener* listener,
- TemplateURLService* template_url_service,
- Profile* profile)
- : BaseSearchProvider(template_url_service,
- scoped_ptr<AutocompleteProviderClient>(
- new ChromeAutocompleteProviderClient(profile)),
- AutocompleteProvider::TYPE_ZERO_SUGGEST),
+ AutocompleteProviderClient* client,
+
+ AutocompleteProviderListener* listener,
+ TemplateURLService* template_url_service,
+ Profile* profile)
+ : BaseSearchProvider(AutocompleteProvider::TYPE_ZERO_SUGGEST,
+ client,
+ template_url_service),
listener_(listener),
profile_(profile),
results_from_cache_(false),
@@ -444,8 +447,7 @@ bool ZeroSuggestProvider::ShouldShowNonContextualZeroSuggest(
if (!ZeroSuggestEnabled(suggest_url,
template_url_service_->GetDefaultSearchProvider(),
current_page_classification_,
- template_url_service_->search_terms_data(),
- client_.get()))
+ template_url_service_->search_terms_data(), client_))
return false;
// If we cannot send URLs, then only the MostVisited and Personalized
« no previous file with comments | « chrome/browser/autocomplete/zero_suggest_provider.h ('k') | chrome/browser/autocomplete/zero_suggest_provider_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698