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

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

Issue 9417032: 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
diff --git a/chrome/browser/autocomplete/keyword_provider.h b/chrome/browser/autocomplete/keyword_provider.h
index 38673459252d3243926d68cc3a28d0e67f9adbb9..115eefc8a043d5f3046befbe056a8e9b49684375 100644
--- a/chrome/browser/autocomplete/keyword_provider.h
+++ b/chrome/browser/autocomplete/keyword_provider.h
@@ -55,6 +55,15 @@ class KeywordProvider : public AutocompleteProvider,
// 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.
@@ -71,6 +80,17 @@ class KeywordProvider : public AutocompleteProvider,
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;
@@ -93,15 +113,6 @@ class KeywordProvider : public AutocompleteProvider,
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(
@@ -139,6 +150,8 @@ class KeywordProvider : public AutocompleteProvider,
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