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

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

Issue 20777006: Omnibox: Create Bundled Field Trial; Convert SearchHistory trial to it (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years, 5 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
« no previous file with comments | « no previous file | chrome/browser/autocomplete/search_provider.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/autocomplete/search_provider.h
diff --git a/chrome/browser/autocomplete/search_provider.h b/chrome/browser/autocomplete/search_provider.h
index 3f9b080a20fc503842a26d42afc77d31239dbd33..74c5e7eba53c808efd636760cbcfdcd555843b67 100644
--- a/chrome/browser/autocomplete/search_provider.h
+++ b/chrome/browser/autocomplete/search_provider.h
@@ -435,11 +435,14 @@ class SearchProvider : public AutocompleteProvider,
// |time| is the time at which this query was last seen. |is_keyword|
// indicates whether the results correspond to the keyword provider or default
- // provider. |prevent_inline_autocomplete| is true if we should not inline
- // autocomplete this query.
+ // provider. |use_aggressive_method| says whether this function can use a
+ // method that gives high scores (1200+) rather than one that gives lower
+ // scores. When using the aggressive method, scores may exceed 1300
+ // unless |prevent_search_history_inlining| is set.
int CalculateRelevanceForHistory(const base::Time& time,
bool is_keyword,
- bool prevent_inline_autocomplete) const;
+ bool use_aggressive_method,
+ bool prevent_search_history_inlining) const;
// Creates an AutocompleteMatch for "Search <engine> for |query_string|" with
// the supplied relevance. Adds this match to |map|; if such a match already
@@ -531,9 +534,6 @@ class SearchProvider : public AutocompleteProvider,
// they will not be inlined.
bool prevent_search_history_inlining_;
- // If true, no search history query suggestions will be offered.
- bool disable_search_history_;
-
DISALLOW_COPY_AND_ASSIGN(SearchProvider);
};
« no previous file with comments | « no previous file | chrome/browser/autocomplete/search_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698