Skip to content
Snippets Groups Projects
Commit e7f51241 authored by pkasting@chromium.org's avatar pkasting@chromium.org
Browse files

Don't treat ';' as an authority terminator when parsing a potential URL. This...

Don't treat ';' as an authority terminator when parsing a potential URL.  This was making the omnibox believe that "tl;dr" was a URL with "tl" as a host and ";dr" as a path, confusing it.

BUG=56967
TEST=Covered by unittests
Review URL: http://codereview.chromium.org/3440032

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60726 0039d316-1c4b-4281-b951-d872f2087c98
parent bd07f1d9
No related merge requests found
......@@ -22,7 +22,7 @@ deps = {
"/trunk/deps/support@20411",
"src/googleurl":
(Var("googlecode_url") % "google-url") + "/trunk@146",
(Var("googlecode_url") % "google-url") + "/trunk@147",
"src/seccompsandbox":
"http://seccompsandbox.googlecode.com/svn/trunk@119",
......
......@@ -217,6 +217,7 @@ TEST(AutocompleteTest, InputType) {
{ L"foo.com", AutocompleteInput::URL },
{ L"-.com", AutocompleteInput::UNKNOWN },
{ L"foo/bar", AutocompleteInput::URL },
{ L"foo;bar", AutocompleteInput::QUERY },
{ L"foo/bar baz", AutocompleteInput::UNKNOWN },
{ L"foo bar.com", AutocompleteInput::QUERY },
{ L"foo bar", AutocompleteInput::QUERY },
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment