Enabled pressing TAB to traverse through the Omnibox results, removed moving the caret to the end...

chrome/browser/autocomplete/keyword_provider.h

Index: chrome/browser/autocomplete/keyword_provider.h
===================================================================
--- chrome/browser/autocomplete/keyword_provider.h	(revision 122771)
+++ chrome/browser/autocomplete/keyword_provider.h	(working copy)
@@ -55,6 +55,15 @@
   // 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,

[Peter Kasting, 2012/02/24 02:16:34]

Nit: Indent these even, not 2

+    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 @@
       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,

[Peter Kasting, 2012/02/24 02:16:34]

Nit: Can fit first arg on above line and then indent even

+      const string16& keyword,
+      const AutocompleteInput& input);
+
   // AutocompleteProvider
   virtual void Start(const AutocompleteInput& input,
                      bool minimal_changes) OVERRIDE;
@@ -93,15 +113,6 @@
                                       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 @@
                        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_;