Bug 32247 - [GTK] Doesn't respect Content-Disposition for downloads, and provides no way for apps to do that
: [GTK] Doesn't respect Content-Disposition for downloads, and provides no way ...
Status: RESOLVED FIXED
Product: WebKit
Classification: Unclassified
Component: WebKit Gtk
: 528+ (Nightly build)
: PC Linux
: P2 Normal
Assigned To: Nobody
: Gtk
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-12-07 14:39 PST by Gustavo Noronha (kov)
Modified: 2009-12-07 18:46 PST (History)
2 users (show)

See Also:


Attachments
proposed fix (10.95 KB, patch)
2009-12-07 14:45 PST, Gustavo Noronha (kov)
xan.lopez: review+
gns: commit‑queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gustavo Noronha (kov) 2009-12-07 14:39:53 PST
Currently, when a server sends a response with Content-Disposition, we are not respecting it, and we give no means for the application to decided, during the mime type policy check, if it wants to perform a download based on that header. This is a rather severe problem for some users, because it makes us unable to work correctly when the 'Download' link in attachments webkit is able to render itself is clicked in GMail, for instance.
Comment 1 Gustavo Noronha (kov) 2009-12-07 14:45:23 PST
Created attachment 44433 [details]
proposed fix
Comment 2 Gustavo Noronha (kov) 2009-12-07 14:45:41 PST
Xan? Jan? =)
Comment 3 WebKit Review Bot 2009-12-07 14:48:57 PST
Attachment 44433 [details] did not pass style-queue:

Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebKit/gtk/tests/testwebframe.c:158:  test_webkit_web_frame_response is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
WebKit/gtk/tests/testwebframe.c:160:  Declaration has space between * and variable name in WebKitWebFrame* frame  [whitespace/declaration] [3]
WebKit/gtk/tests/testwebframe.c:161:  Declaration has space between * and variable name in WebKitNetworkResponse* response  [whitespace/declaration] [3]
WebKit/gtk/webkit/webkitwebframe.cpp:1158:  webkit_web_frame_get_network_response is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
WebKit/gtk/webkit/webkitwebframe.cpp:1162:  Use 0 instead of NULL.  [readability/null] [5]
WebKit/gtk/webkit/webkitwebframe.cpp:1166:  Use 0 instead of NULL.  [readability/null] [5]
WebKit/gtk/webkit/webkitwebframe.h:176:  Extra space before ( in function call  [whitespace/parens] [4]
WebKit/gtk/tests/testmimehandling.c:109:  Declaration has space between * and variable name in WebKitNetworkResponse* response  [whitespace/declaration] [3]
WebKit/gtk/tests/testmimehandling.c:110:  Declaration has space between * and variable name in SoupMessage* message  [whitespace/declaration] [3]
WebKit/gtk/tests/testmimehandling.c:111:  Declaration has space between * and variable name in char* disposition  [whitespace/declaration] [3]
Total errors found: 10
Comment 4 Xan Lopez 2009-12-07 15:09:47 PST
Comment on attachment 44433 [details]
proposed fix

Nice!
Comment 5 Gustavo Noronha (kov) 2009-12-07 18:46:36 PST
Landed as r51825.