diff options
| author | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-19 20:25:05 +0000 |
|---|---|---|
| committer | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-19 20:25:05 +0000 |
| commit | 03a61e3d6731a54293958f2583d33d063e798af2 (patch) | |
| tree | 5cae3aa5078fb0c27e2a09953c0744d918675f23 /gfx | |
| parent | c9c8f440e4e80a289f65a841bd11aab9bcf44f47 (diff) | |
| download | chromium_src-03a61e3d6731a54293958f2583d33d063e798af2.zip chromium_src-03a61e3d6731a54293958f2583d33d063e798af2.tar.gz chromium_src-03a61e3d6731a54293958f2583d33d063e798af2.tar.bz2 | |
Changes FormatURL to not strip http if the host starts with ftp or https[!a-z]. This
is necessitated by the omnibox treating ftp.www.com as
ftp://www.com as well as security problems.
I also changed FormatURL to always reset the supplied
url_parse::Parsed. I did this as in writing my test I uncovered a long
standing bug where a component wasn't getting reset. It seems much
safer to always reset than rely on all code paths setting every
component.
BUG=41585, 41652
TEST=make sure the omnibox doesn't strip http off urls that start with
ftp.
Review URL: http://codereview.chromium.org/1574034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44943 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'gfx')
0 files changed, 0 insertions, 0 deletions