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

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

Issue 9419043: Revert 122412 - Enabled pressing TAB to traverse through the Omnibox results (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 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/keyword_provider.h
===================================================================
--- chrome/browser/autocomplete/keyword_provider.h (revision 122471)
+++ chrome/browser/autocomplete/keyword_provider.h (working copy)
@@ -55,15 +55,6 @@
// For testing.
KeywordProvider(ACProviderListener* listener, TemplateURLService* model);
- // Extracts the next whitespace-delimited token from input and returns it.
- // Sets |remaining_input| to everything after the first token (skipping over
- // the first intervening whitespace).
- // If |trim_leading_whitespace| is true then leading whitespace in
- // |*remaining_input| will be trimmed.
- static string16 SplitKeywordFromInput(const string16& input,
- bool trim_leading_whitespace,
- string16* remaining_input);
-
// Returns the replacement string from the user input. The replacement
// string is the portion of the input that does not contain the keyword.
// For example, the replacement string for "b blah" is blah.
@@ -80,17 +71,6 @@
const AutocompleteInput& input,
string16* remaining_input);
- // If |text| corresponds (in the sense of
- // TemplateURLModel::CleanUserInputKeyword()) to an enabled, substituting
- // keyword, returns that keyword; returns the empty string otherwise.
- string16 GetKeywordForText(const string16& text) const;
-
- // Creates a fully marked-up AutocompleteMatch for a specific keyword.
- AutocompleteMatch CreateAutocompleteMatch(
- const string16& text,
- const string16& keyword,
- const AutocompleteInput& input);
-
// AutocompleteProvider
virtual void Start(const AutocompleteInput& input,
bool minimal_changes) OVERRIDE;
@@ -113,6 +93,15 @@
string16* keyword,
string16* remaining_input);
+ // Extracts the next whitespace-delimited token from input and returns it.
+ // Sets |remaining_input| to everything after the first token (skipping over
+ // the first intervening whitespace).
+ // If |trim_leading_whitespace| is true then leading whitespace in
+ // |*remaining_input| will be trimmed.
+ static string16 SplitKeywordFromInput(const string16& input,
+ bool trim_leading_whitespace,
+ string16* remaining_input);
+
// Fills in the "destination_url" and "contents" fields of |match| with the
// provided user input and keyword data.
static void FillInURLAndContents(
@@ -150,8 +139,6 @@
const content::NotificationSource& source,
const content::NotificationDetails& details) OVERRIDE;
- TemplateURLService* GetTemplateURLService() const;
-
// Model for the keywords. This is only non-null when testing, otherwise the
// TemplateURLService from the Profile is used.
TemplateURLService* model_;
« no previous file with comments | « chrome/browser/autocomplete/history_url_provider_unittest.cc ('k') | chrome/browser/autocomplete/keyword_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698