Bug 30433 - Chrome doesn't set Content-Type for file upload when the file extension is not recognized
Summary: Chrome doesn't set Content-Type for file upload when the file extension is no...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Forms (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-10-16 01:14 PDT by Adam Barth
Modified: 2009-10-16 09:20 PDT (History)
1 user (show)

See Also:

Attachments
Patch v1 (2.27 KB, patch)
2009-10-16 01:18 PDT, Adam Barth 		no flags Details | Formatted Diff | Diff
Patch v1 (1.52 KB, patch)
2009-10-16 01:20 PDT, Adam Barth 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Barth 2009-10-16 01:14:30 PDT 
http://code.google.com/p/chromium/issues/detail?id=6800

Patch forthcoming, but I haven't the foggiest idea how to write a test case.
Comment 1 Adam Barth 2009-10-16 01:18:29 PDT 
Created attachment 41275 [details]
Patch v1
Comment 2 Adam Barth 2009-10-16 01:20:13 PDT 
Created attachment 41276 [details]
Patch v1
Comment 3 Mark Rowe (bdash) 2009-10-16 03:15:20 PDT 
Comment on attachment 41276 [details]
Patch v1

> Index: WebCore/platform/chromium/MimeTypeRegistryChromium.cpp

It’s sad that this file doesn’t match the case used in the name of the class and in the equivalent files used by other platforms.

r=me
Comment 4 Adam Barth 2009-10-16 09:20:40 PDT 
Comment on attachment 41276 [details]
Patch v1

Clearing flags on attachment: 41276

Committed r49682: <http://trac.webkit.org/changeset/49682>
Comment 5 Adam Barth 2009-10-16 09:20:45 PDT 
All reviewed patches have been landed.  Closing bug.