diff options
| author | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-24 21:36:12 +0000 |
|---|---|---|
| committer | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-24 21:36:12 +0000 |
| commit | 5e183ded3525704427b862268f08439543bf6aa5 (patch) | |
| tree | b4a5bd662cc352a0d3496b32ab7fc94b30a30224 /webkit/glue/weburlloader_impl.cc | |
| parent | 61731fe5a94e74fc1c4df426d0868da8b0d7a061 (diff) | |
| download | chromium_src-5e183ded3525704427b862268f08439543bf6aa5.zip chromium_src-5e183ded3525704427b862268f08439543bf6aa5.tar.gz chromium_src-5e183ded3525704427b862268f08439543bf6aa5.tar.bz2 | |
wstring: make net::GetSuggestedFilename return utf-16
We should only use FilePath when working with real paths on disk.
Note: I made a minor semantic change to downloads -- we no
longer preserve backslashes in server-suggested paths on non-Windows.
I think the previous behavior is crazy but from looking at the
blame I'm the one who wrote it originally, so I don't feel bad
about changing my mind.
BUG=69467
TEST=existing test still passes
Review URL: http://codereview.chromium.org/6241001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72390 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/glue/weburlloader_impl.cc')
| -rw-r--r-- | webkit/glue/weburlloader_impl.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/webkit/glue/weburlloader_impl.cc b/webkit/glue/weburlloader_impl.cc index d22b509..8904457 100644 --- a/webkit/glue/weburlloader_impl.cc +++ b/webkit/glue/weburlloader_impl.cc @@ -244,8 +244,8 @@ void PopulateURLResponse( // pass it to GetSuggestedFilename. std::string value; if (headers->EnumerateHeader(NULL, "content-disposition", &value)) { - response->setSuggestedFileName(FilePathToWebString( - net::GetSuggestedFilename(url, value, "", FilePath()))); + response->setSuggestedFileName( + net::GetSuggestedFilename(url, value, "", string16())); } Time time_val; |