Enabled pressing TAB to cycle through the Omnibox results.

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

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/views/location_bar/location_bar_view.h"
 
 #if defined(TOOLKIT_USES_GTK)
 #include <gtk/gtk.h>
 #endif
 
@@ skipping 1015 matching lines @@
 #endif
 }
 
 std::string LocationBarView::GetClassName() const {
   return kViewClassName;
 }
 
 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;
 #else
   // This method is not used for Linux ports. See FocusManager::OnKeyEvent() in
   // src/views/focus/focus_manager.cc for details.
   return false;
 #endif
 }
 
@@ skipping 185 matching lines @@
 bool LocationBarView::HasValidSuggestText() const {
   return suggested_text_view_ && !suggested_text_view_->size().IsEmpty() &&
       !suggested_text_view_->GetText().empty();
 }
 
 OmniboxViewWin* LocationBarView::GetOmniboxViewWin() {
   CHECK(!views::Widget::IsPureViews());
   return static_cast<OmniboxViewWin*>(location_entry_.get());
 }
 #endif