Enabled pressing TAB to cycle through the Omnibox results.

chrome/browser/autocomplete/autocomplete.h

Index: chrome/browser/autocomplete/autocomplete.h
===================================================================
--- chrome/browser/autocomplete/autocomplete.h	(revision 80563)
+++ chrome/browser/autocomplete/autocomplete.h	(working copy)
@@ -657,6 +657,16 @@
   // the popup to ensure it's not showing an out-of-date query.
   void ExpireCopiedEntries();
 
+  // Gets the selected keyword or keyword hint for the given match.

[Peter Kasting, 2011/04/07 20:19:21]

This function claims to be a getter but actually is something that updates the
keyword field on the supplied match.

Since I think this now only has one caller, when the controller wants to set the
keywords on the return result, I suggest simply inlining this functionality into
that loop.

+  void GetKeywordForMatch(AutocompleteMatch* match) const;
+
+  // Gets the selected keyword or keyword hint for the given text. Returns
+  // true if |keyword| represents a keyword hint, or false if |keyword|
+  // represents a selected keyword. (|keyword| will always be set [though

[Peter Kasting, 2011/04/07 20:19:21]

This function now seems to only have two cases: clear |keyword| and return
false, or set it and return true.  I suggest condensing it to just "string16
GetKeywordForText(const string16& text)".

+  // possibly to the empty string], and you cannot have both a selected keyword
+  // and a keyword hint simultaneously.)
+  bool GetKeywordForText(const string16& text, string16* keyword) const;
+
   SearchProvider* search_provider() const { return search_provider_; }
 
   // Getters
@@ -700,6 +710,8 @@
   // invokes |ExpireCopiedEntries|.
   base::OneShotTimer<AutocompleteController> expire_timer_;
 
+  Profile* profile_;
+
   // True if a query is not currently running.
   bool done_;