Don't treat input with a scheme that's not "http" or "https" as a potential k...

chrome/browser/autocomplete/keyword_provider_unittest.cc

-// Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
+// Copyright (c) 2010 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 "chrome/browser/autocomplete/keyword_provider.h"
 #include "chrome/browser/search_engines/template_url.h"
 #include "chrome/browser/search_engines/template_url_model.h"
 #include "googleurl/src/gurl.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ skipping 80 matching lines @@
 
     // Matches should be limited to three, and sorted in quality order, not
     // alphabetical.
     {L"aaa",             2, {L"aaaa ", L"aaaaa "}},
     {L"a 1 2 3",         3, {L"aa 1 2 3", L"ab 1 2 3", L"aaaa 1 2 3"}},
     {L"www.a",           3, {L"aa ", L"ab ", L"aaaa "}},
     // Exact matches should prevent returning inexact matches.
     {L"aaaa foo",        1, {L"aaaa foo"}},
     {L"www.aaaa foo",    1, {L"aaaa foo"}},
 
-    // Clean up keyword input properly.
+    // Clean up keyword input properly.  "http" and "https" are the only
+    // allowed schemes.
     {L"www",             1, {L"www "}},
     {L"www.",            0, {}},
     {L"www.w w",         2, {L"www w", L"weasel w"}},
     {L"http://www",      1, {L"www "}},
     {L"http://www.",     0, {}},
     {L"ftp: blah",       0, {}},
-    {L"mailto:z",        1, {L"z "}},
+    {L"mailto:z",        0, {}},
+    {L"ftp://z",         0, {}},
+    {L"https://z",       1, {L"z "}},
   };
 
   RunTest<std::wstring>(edit_cases, arraysize(edit_cases),
                         &AutocompleteMatch::fill_into_edit);
 }
 
 TEST_F(KeywordProviderTest, URL) {
   test_data<GURL> url_cases[] = {
     // No query input -> empty destination URL.
     {L"z",               1, {GURL()}},
@@ skipping 38 matching lines @@
   RunTest<std::wstring>(contents_cases, arraysize(contents_cases),
                         &AutocompleteMatch::contents);
 }
 
 TEST_F(KeywordProviderTest, Description) {
   test_data<std::wstring> description_cases[] = {
     // Whole keyword should be returned for both exact and inexact matches.
     {L"z foo",           1, {L"(Keyword: z)"}},
     {L"a foo",           3, {L"(Keyword: aa)", L"(Keyword: ab)",
                              L"(Keyword: aaaa)"}},
-    {L"ftp://www.www w", 1, {L"(Keyword: www)"}},
+    {L"ftp://www.www w", 0, {}},
+    {L"http://www.ab w", 1, {L"(Keyword: ab)"}},
 
     // Keyword should be returned regardless of query input.
     {L"z",               1, {L"(Keyword: z)"}},
     {L"z    \t",         1, {L"(Keyword: z)"}},
     {L"z   a   b   c++", 1, {L"(Keyword: z)"}},
   };
 
   RunTest<std::wstring>(description_cases, arraysize(description_cases),
                         &AutocompleteMatch::description);
 }
 
 TEST_F(KeywordProviderTest, AddKeyword) {
   TemplateURL* template_url = new TemplateURL();
   std::wstring keyword(L"foo");
   std::string url("http://www.google.com/foo?q={searchTerms}");
   template_url->SetURL(url, 0, 0);
   template_url->set_keyword(keyword);
   template_url->set_short_name(L"Test");
   model_->Add(template_url);
   ASSERT_TRUE(template_url == model_->GetTemplateURLForKeyword(keyword));
 }
 
 TEST_F(KeywordProviderTest, RemoveKeyword) {
   std::wstring url(L"http://aaaa/?aaaa=1&b={searchTerms}&c");
   model_->Remove(model_->GetTemplateURLForKeyword(L"aaaa"));
   ASSERT_TRUE(model_->GetTemplateURLForKeyword(L"aaaa") == NULL);
 }