Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(300)

Unified Diff: net/http/http_content_disposition_unittest.cc

Issue 10447019: Prefer 'filename' parameters if both 'name' and 'filename' parameters are specified in a Content-Di… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge only Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/http/http_content_disposition.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_content_disposition_unittest.cc
diff --git a/net/http/http_content_disposition_unittest.cc b/net/http/http_content_disposition_unittest.cc
index 7587c3e9e92b4516dd47f95dee1822119db09fd0..240c69962e2ec8b90638792228105c9ef0cf4257 100644
--- a/net/http/http_content_disposition_unittest.cc
+++ b/net/http/http_content_disposition_unittest.cc
@@ -147,6 +147,11 @@ TEST(HttpContentDispositionTest, Filename) {
{"attachment; filename*=windows-1252''caf%E9.png", "",
L"caf\x00e9.png"},
+ // Multiple filename, filename*, name parameters specified.
+ {"attachment; name=\"foo\"; filename=\"bar\"", "", L"bar"},
+ {"attachment; filename=\"bar\"; name=\"foo\"", "", L"bar"},
+ {"attachment; filename=\"bar\"; filename*=utf-8''baz", "", L"baz"},
+
// http://greenbytes.de/tech/tc2231/ filename* test cases.
// attwithisofn2231iso
{"attachment; filename*=iso-8859-1''foo-%E4.html", "",
« no previous file with comments | « net/http/http_content_disposition.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698