Enabled pressing TAB to cycle through the Omnibox results.

chrome/browser/ui/views/location_bar/location_bar_view.cc

Index: chrome/browser/ui/views/location_bar/location_bar_view.cc
===================================================================
--- chrome/browser/ui/views/location_bar/location_bar_view.cc	(revision 93757)
+++ chrome/browser/ui/views/location_bar/location_bar_view.cc	(working copy)
@@ -1033,29 +1033,13 @@
 bool LocationBarView::SkipDefaultKeyEventProcessing(
     const views::KeyEvent& event) {
 #if defined(OS_WIN)
-  bool views_omnibox = views::Widget::IsPureViews();
-  if (views::FocusManager::IsTabTraversalKeyEvent(event)) {
-    if (HasValidSuggestText()) {
-      // Return true so that the edit sees the tab and commits the suggestion.
-      return true;
-    }
-    if (keyword_hint_view_->IsVisible() && !event.IsShiftDown()) {
-      // Return true so the edit gets the tab event and enters keyword mode.
-      return true;
-    }
-
-    // If the caret is not at the end, then tab moves the caret to the end.
-    if (!views_omnibox && !GetOmniboxViewWin()->IsCaretAtEnd())
-      return true;
-
-    // Tab while showing instant commits instant immediately.

[Peter Kasting, 2011/07/27 20:18:25]

I think this change depends on us ditching this behavior, which may require
other changes.  sky would know.

-    // Return true so that focus traversal isn't attempted. The edit ends
-    // up doing nothing in this case.
-    if (location_entry_->model()->AcceptCurrentInstantPreview())
-      return true;
+  if (views::FocusManager::IsTabTraversalKeyEvent(event) &&
+      (location_entry_->model()->popup_model()->IsOpen() ||
+      (keyword_hint_view_->IsVisible() && !event.IsShiftDown()))) {
+    return true;
   }
 
-  if (!views_omnibox)
+  if (!views::Widget::IsPureViews())
     return GetOmniboxViewWin()->SkipDefaultKeyEventProcessing(event);
   NOTIMPLEMENTED();
   return false;