Enabled pressing TAB to cycle through the Omnibox results.

chrome/browser/instant/instant_controller.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/instant/instant_controller.h"
 
 #include "base/command_line.h"
 #include "base/message_loop.h"
 #include "base/metrics/histogram.h"
 #include "build/build_config.h"
@@ skipping 618 matching lines @@
       // To avoid extra load on other search engines we only enable previews if
       // they support the instant API.
       return false;
     }
   }
   *template_url = t_url;
 
   if (match.destination_url.SchemeIs(chrome::kJavaScriptScheme))
     return false;
 
   // Extension keywords don't have a real destionation URL.

[Peter Kasting, 2011/04/13 21:06:59]

Nit: While here, destionation -> destination

-  if (match.template_url && match.template_url->IsExtensionKeyword())
+  if (match.keyword_state == AutocompleteMatch::KEYWORD &&

[Peter Kasting, 2011/04/13 21:06:59]

I don't think we need to check the keyword state here.

+      match.keyword_url && match.keyword_url->IsExtensionKeyword())
     return false;
 
   // Was the host blacklisted?
   if (host_blacklist_ && host_blacklist_->count(match.destination_url.host()))
     return false;
 
   return true;
 }
 
 void InstantController::BlacklistFromInstant(TemplateURLID id) {
@@ skipping 16 matching lines @@
             &InstantController::DestroyLoaders));
   }
 }
 
 void InstantController::DestroyLoaders() {
   loaders_to_destroy_.reset();
 }
 
 const TemplateURL* InstantController::GetTemplateURL(
     const AutocompleteMatch& match) {
-  const TemplateURL* template_url = match.template_url;
+  const TemplateURL* template_url =
+      match.keyword_state == AutocompleteMatch::KEYWORD ? match.keyword_url :

[Peter Kasting, 2011/04/13 21:06:59]

I think you also want to allow DUAL_SHOWING_KEYWORD here.

I also think you want to write this like

if (state == KEYWORD or DUAL_SHOWING_KEYWORD) {
  template_url = keyword_url;
} else if (match.type == ...) { ...

Because if we're actively showing a keyword, the right template URL is the one
on the match, regardless of the default search provider.

+      NULL;
   if (match.type == AutocompleteMatch::SEARCH_WHAT_YOU_TYPED ||
       match.type == AutocompleteMatch::SEARCH_HISTORY ||
       match.type == AutocompleteMatch::SEARCH_SUGGEST) {
     TemplateURLModel* model = tab_contents_->profile()->GetTemplateURLModel();
     template_url = model ? model->GetDefaultSearchProvider() : NULL;
   }
   return template_url;
 }