Enabled pressing TAB to cycle through the Omnibox results.

chrome/browser/autocomplete/keyword_provider_unittest.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 "base/message_loop.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/autocomplete/autocomplete_match.h"
 #include "chrome/browser/autocomplete/keyword_provider.h"
 #include "chrome/browser/search_engines/template_url.h"
 #include "chrome/browser/search_engines/template_url_service.h"
@@ skipping 14 matching lines @@
   virtual ~KeywordProviderTest() { }
 
   virtual void SetUp();
   virtual void TearDown();
 
   template<class ResultType>
   void RunTest(test_data<ResultType>* keyword_cases,
                int num_cases,
                ResultType AutocompleteMatch::* member);
 
+  void RunTest(const char* input, const char* keyword);

[Peter Kasting, 2011/12/15 22:56:04]

I'd prefer not to add a function with the same name that means something totally
different from the one above.

You can just inline this function's body into the GetKeywordForInput testcase:

  EXPECT_EQ(ASCIIToUTF16("aa"),
            kw_provider_->GetKeywordForText(ASCIIToUTF16("aa")));
  ... (and similar) ...

+
  protected:
   scoped_refptr<KeywordProvider> kw_provider_;
   scoped_ptr<TemplateURLService> model_;
 };
 
 void KeywordProviderTest::SetUp() {
   static const TemplateURLService::Initializer kTestKeywordData[] = {
     { "aa", "aa.com?foo=%s", "aa" },
     { "aaaa", "http://aaaa/?aaaa=1&b=%s&c", "aaaa" },
     { "aaaaa", "%s", "aaaaa" },
@@ skipping 28 matching lines @@
     EXPECT_EQ(keyword_cases[i].num_results, matches.size()) <<
                 ASCIIToUTF16("Input was: ") + keyword_cases[i].input;
     if (matches.size() == keyword_cases[i].num_results) {
       for (size_t j = 0; j < keyword_cases[i].num_results; ++j) {
         EXPECT_EQ(keyword_cases[i].output[j], matches[j].*member);
       }
     }
   }
 }
 
+void KeywordProviderTest::RunTest(const char* input,
+    const char* keyword) {
+  string16 keyword16(ASCIIToUTF16(keyword));
+  string16 detected_keyword =
+      kw_provider_->GetKeywordForText(ASCIIToUTF16(input));
+  EXPECT_EQ(keyword16, detected_keyword);
+}
+
 // http://crbug.com/80612
 TEST_F(KeywordProviderTest, DISABLED_Edit) {
   test_data<string16> edit_cases[] = {
     // Searching for a nonexistent prefix should give nothing.
     {ASCIIToUTF16("Not Found"),       0, {}},
     {ASCIIToUTF16("aaaaaNot Found"),  0, {}},
 
     // Check that tokenization only collapses whitespace between first tokens,
     // no-query-input cases have a space appended, and action is not escaped.
     {ASCIIToUTF16("z foo"),           1, {ASCIIToUTF16("z foo")}},
@@ skipping 116 matching lines @@
   template_url->set_short_name(ASCIIToUTF16("Test"));
   model_->Add(template_url);
   ASSERT_TRUE(template_url == model_->GetTemplateURLForKeyword(keyword));
 }
 
 TEST_F(KeywordProviderTest, RemoveKeyword) {
   string16 url(ASCIIToUTF16("http://aaaa/?aaaa=1&b={searchTerms}&c"));
   model_->Remove(model_->GetTemplateURLForKeyword(ASCIIToUTF16("aaaa")));
   ASSERT_TRUE(model_->GetTemplateURLForKeyword(ASCIIToUTF16("aaaa")) == NULL);
 }
+
+TEST_F(KeywordProviderTest, GetKeywordForInput) {
+  RunTest("aa", "aa");
+  RunTest("aafoo", "");
+  RunTest("aa foo", "aa");
+}
+