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

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

Issue 158053002: Part 4 of search provider refactoring. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed comments Created 6 years, 10 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.h
diff --git a/chrome/browser/autocomplete/zero_suggest_provider.h b/chrome/browser/autocomplete/zero_suggest_provider.h
index c3fd7ef744b24a30fec467fb80e36fd1e03e41a9..b37d50e4cec34237d671f1f499fa2cc58964595a 100644
--- a/chrome/browser/autocomplete/zero_suggest_provider.h
+++ b/chrome/browser/autocomplete/zero_suggest_provider.h
@@ -72,6 +72,10 @@ class ZeroSuggestProvider : public BaseSearchProvider {
virtual ~ZeroSuggestProvider();
+ // BaseSearchProvider
+ virtual bool ShouldAppendExtraParams(
+ const SuggestResult& result) const OVERRIDE;
+
// The 4 functions below (that take classes defined in SearchProvider as
// arguments) were copied and trimmed from SearchProvider.
// TODO(hfung): Refactor them into a new base class common to both
@@ -92,19 +96,6 @@ class ZeroSuggestProvider : public BaseSearchProvider {
const TemplateURL* template_url,
MatchMap* map);
- // Creates an AutocompleteMatch with the provided |relevance| and |type| to
- // search |template_url| for |query_string|. |accepted_suggestion| will be
- // used to generate Assisted Query Stats.
- //
- // Adds this match to |map|; if such a match already exists, whichever one
- // has lower relevance is eliminated.
- void AddMatchToMap(int relevance,
- AutocompleteMatch::Type type,
- const TemplateURL* template_url,
- const base::string16& query_string,
- int accepted_suggestion,
- MatchMap* map);
-
// Returns an AutocompleteMatch for a navigational suggestion |navigation|.
AutocompleteMatch NavigationToMatch(const NavigationResult& navigation);

Powered by Google App Engine
This is Rietveld 408576698