Parse input with explicit schemes better. Before, if the user typed "http://...

chrome/browser/autocomplete/autocomplete_unittest.cc

 // Copyright (c) 2006-2008 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/string_util.h"
 #include "chrome/browser/autocomplete/autocomplete.h"
 #include "chrome/common/notification_registrar.h"
 #include "chrome/common/notification_service.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 192 matching lines @@
   struct test_data {
     const wchar_t* input;
     const AutocompleteInput::Type type;
   } input_cases[] = {
     { L"", AutocompleteInput::INVALID },
     { L"?", AutocompleteInput::FORCED_QUERY },
     { L"?foo", AutocompleteInput::FORCED_QUERY },
     { L"?foo bar", AutocompleteInput::FORCED_QUERY },
     { L"?http://foo.com/bar", AutocompleteInput::FORCED_QUERY },
     { L"foo", AutocompleteInput::UNKNOWN },
+    { L"foo.c", AutocompleteInput::UNKNOWN },
     { L"foo.com", AutocompleteInput::URL },
     { L"-.com", AutocompleteInput::QUERY },
     { L"foo/bar", AutocompleteInput::URL },
     { L"foo/bar baz", AutocompleteInput::UNKNOWN },
     { L"http://foo/bar baz", AutocompleteInput::URL },
     { L"foo bar", AutocompleteInput::QUERY },
     { L"\"foo:bar\"", AutocompleteInput::QUERY },
     { L"link:foo.com", AutocompleteInput::UNKNOWN },
     { L"www.foo.com:81", AutocompleteInput::URL },
     { L"localhost:8080", AutocompleteInput::URL },
     { L"foo.com:123456", AutocompleteInput::QUERY },
     { L"foo.com:abc", AutocompleteInput::QUERY },
+    { L"user@foo.com", AutocompleteInput::UNKNOWN },
+    { L"user:pass@foo.com", AutocompleteInput::UNKNOWN },
+    { L"1.2", AutocompleteInput::UNKNOWN },
+    { L"1.2/45", AutocompleteInput::UNKNOWN },
+    { L"ps/2 games", AutocompleteInput::UNKNOWN },
     { L"en.wikipedia.org/wiki/James Bond", AutocompleteInput::URL },
     // In Chrome itself, mailto: will get handled by ShellExecute, but in
     // unittest mode, we don't have the data loaded in the external protocol
     // handler to know this.
     // { L"mailto:abuse@foo.com", AutocompleteInput::URL },
     { L"view-source:http://www.foo.com/", AutocompleteInput::URL },
     { L"javascript:alert(\"Hey there!\");", AutocompleteInput::URL },
 #if defined(OS_WIN)
     { L"C:\\Program Files", AutocompleteInput::URL },
     { L"\\\\Server\\Folder\\File", AutocompleteInput::URL },
 #endif  // defined(OS_WIN)
-    { L"http://foo.com/", AutocompleteInput::URL },
+    { L"http:foo", AutocompleteInput::URL },
+    { L"http://foo", AutocompleteInput::URL },
+    { L"http://foo.c", AutocompleteInput::URL },
+    { L"http://foo.com", AutocompleteInput::URL },
+    { L"http://-.com", AutocompleteInput::QUERY },
+    { L"http://foo.com:abc", AutocompleteInput::QUERY },
+    { L"http://foo.com:123456", AutocompleteInput::QUERY },
+    { L"http:user@foo.com", AutocompleteInput::URL },
+    { L"http://user@foo.com", AutocompleteInput::URL },
+    { L"http://user:pass@foo.com", AutocompleteInput::URL },
+    { L"http://1.2", AutocompleteInput::URL },
+    { L"http://1.2/45", AutocompleteInput::URL },
+    { L"http:ps/2 games", AutocompleteInput::URL },
+    { L"http://ps/2 games", AutocompleteInput::URL },
     { L"127.0.0.1", AutocompleteInput::URL },
     { L"127.0.1", AutocompleteInput::UNKNOWN },
     { L"127.0.1/", AutocompleteInput::UNKNOWN },
     { L"browser.tabs.closeButtons", AutocompleteInput::UNKNOWN },
     { L"\u6d4b\u8bd5", AutocompleteInput::UNKNOWN },
     { L"[2001:]", AutocompleteInput::QUERY },  // Not a valid IP
     { L"[2001:dB8::1]", AutocompleteInput::URL },
     { L"192.168.0.256", AutocompleteInput::QUERY },  // Invalid IPv4 literal.
     { L"[foo.com]", AutocompleteInput::QUERY },  // Invalid IPv6 literal.
   };
@@ skipping 78 matching lines @@
     EXPECT_EQ(input_cases[i].scheme.len, scheme.len) << "Input: " <<
         input_cases[i].input;
     EXPECT_EQ(input_cases[i].host.begin, host.begin) << "Input: " <<
         input_cases[i].input;
     EXPECT_EQ(input_cases[i].host.len, host.len) << "Input: " <<
         input_cases[i].input;
   }
 }
 
 }  // namespace