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

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

Issue 6268005: Removes the link in the omnibox for searching history. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix unit test Created 9 years, 11 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 | « chrome/app/theme/theme_resources.grd ('k') | chrome/browser/autocomplete/autocomplete.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/autocomplete/autocomplete.h
diff --git a/chrome/browser/autocomplete/autocomplete.h b/chrome/browser/autocomplete/autocomplete.h
index f91dbabbc27ab1dbadb189694b96dd1bb5c3b6eb..19a3722d5ec60f2cb2e82309665bb6bb63cf8e25 100644
--- a/chrome/browser/autocomplete/autocomplete.h
+++ b/chrome/browser/autocomplete/autocomplete.h
@@ -504,9 +504,7 @@ class AutocompleteResult {
void Validate() const;
#endif
- // Max number of matches we'll show from the various providers. We may end
- // up showing an additional shortcut for Destinations->History, see
- // AddHistoryContentsShortcut.
+ // Max number of matches we'll show from the various providers.
static const size_t kMaxMatches;
private:
@@ -543,7 +541,6 @@ class AutocompleteController : public ACProviderListener {
#ifdef UNIT_TEST
explicit AutocompleteController(const ACProviders& providers)
: providers_(providers),
- history_contents_provider_(NULL),
search_provider_(NULL),
updated_latest_result_(false),
delay_interval_has_passed_(false),
@@ -642,24 +639,12 @@ class AutocompleteController : public ACProviderListener {
// notificiations.
void CommitResult(bool notify_default_match);
- // Returns the matches from |provider| whose destination urls are not in
- // |latest_result_|.
- ACMatches GetMatchesNotInLatestResult(
- const AutocompleteProvider* provider) const;
-
- // If the HistoryContentsAutocomplete provider is done and there are more
- // matches in the database than currently shown, an entry is added to
- // |latest_result_| to show all history matches.
- void AddHistoryContentsShortcut();
-
// Updates |done_| to be accurate with respect to current providers' statuses.
void CheckIfDone();
// A list of all providers.
ACProviders providers_;
- HistoryContentsProvider* history_contents_provider_;
-
SearchProvider* search_provider_;
// Input passed to Start.
« no previous file with comments | « chrome/app/theme/theme_resources.grd ('k') | chrome/browser/autocomplete/autocomplete.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698