| commit | b48a1f8833ee5d114179a26d746b164bacf3b5d0 | |
| author | brettw@gmail.com <brettw@gmail.com@8873c55e-713a-0410-88f8-23d9c3d90b1b> | |
| Wed, 26 Aug 2009 23:55:35 +0000 (26 23:55 +0000) | ||
| committer | brettw@gmail.com <brettw@gmail.com@8873c55e-713a-0410-88f8-23d9c3d90b1b> | |
| Wed, 26 Aug 2009 23:55:35 +0000 (26 23:55 +0000) | ||
| tree | 5e2f05a99d4566ee21f117f4f9c6e8fb4794b350 | treesnapshot (tar.gz zip) |
| parent | 745c5e6c55501b5c0102008fb737d71e013fe57b | commitdiff |
Update the unit tests to allow a colon in UNC hostnames on Windows. This test
was broken in r107 but we weren't running the tests properly so never noticed.
Colon is probably not allowed by Windows, but its the "natural" result of the
way the canonicalizer is written. There doesn't seem to be a strong argument
for why allowing it here would be bad, so we just tolerate it and the load will
fail later.
Review URL: http://codereview.chromium.org/173535
git-svn-id: http://google-url.googlecode.com/svn/trunk@115 8873c55e-713a-0410-88f8-23d9c3d90b1b
was broken in r107 but we weren't running the tests properly so never noticed.
Colon is probably not allowed by Windows, but its the "natural" result of the
way the canonicalizer is written. There doesn't seem to be a strong argument
for why allowing it here would be bad, so we just tolerate it and the load will
fail later.
Review URL: http://codereview.chromium.org/173535
git-svn-id: http://google-url.googlecode.com/svn/trunk@115 8873c55e-713a-0410-88f8-23d9c3d90b1b
| src/url_canon_unittest.cc | diffblobblamehistory |