Better handling of UNKNOWN versus QUERY in a couple ways....

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 194 matching lines @@
     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"-.com", AutocompleteInput::UNKNOWN },
     { L"foo/bar", AutocompleteInput::URL },
     { L"foo/bar baz", AutocompleteInput::UNKNOWN },
+    { L"foo bar.com", AutocompleteInput::QUERY },
     { L"http://foo/bar baz", AutocompleteInput::URL },
     { L"foo bar", AutocompleteInput::QUERY },
+    { L"foo+bar", AutocompleteInput::QUERY },

[brettw, 2009/11/03 03:30:22]

Can you add "foo+bar.com"? It would make it a little more clear how this diffes
from "foo+bar"

     { 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", AutocompleteInput::URL },
     { L"http://foo", AutocompleteInput::URL },
     { L"http://foo.c", AutocompleteInput::URL },
     { L"http://foo.com", AutocompleteInput::URL },
     { L"http://foo_bar.com", AutocompleteInput::URL },
-    { L"http://-.com", AutocompleteInput::QUERY },
-    { L"http://_foo_.com", AutocompleteInput::QUERY },
+    { L"http://-.com", AutocompleteInput::UNKNOWN },
+    { L"http://_foo_.com", AutocompleteInput::UNKNOWN },
     { 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 },
@@ skipping 87 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