Bug 62502 - Allow setting content-disposition for blob-data
Summary: Allow setting content-disposition for blob-data
Status: UNCONFIRMED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-11 12:24 PDT by Sadrul Habib Chowdhury
Modified: 2011-06-12 08:07 PDT (History)
2 users (show)

See Also:


Attachments
patch (5.35 KB, patch)
2011-06-11 12:28 PDT, Sadrul Habib Chowdhury
webkit.review.bot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ec2-cr-linux-01 (1.61 MB, application/zip)
2011-06-11 12:55 PDT, WebKit Review Bot
no flags Details
Fix license headers + test (5.46 KB, patch)
2011-06-11 14:32 PDT, Sadrul Habib Chowdhury
no flags Details | Formatted Diff | Diff
patch (5.38 KB, patch)
2011-06-11 15:28 PDT, Sadrul Habib Chowdhury
no flags Details | Formatted Diff | Diff
patch (5.38 KB, patch)
2011-06-11 15:30 PDT, Sadrul Habib Chowdhury
webkit.review.bot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ec2-cr-linux-03 (1.30 MB, application/zip)
2011-06-11 16:46 PDT, WebKit Review Bot
no flags Details
Fixed test, for real this time! (5.58 KB, patch)
2011-06-11 17:19 PDT, Sadrul Habib Chowdhury
abarth: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Sadrul Habib Chowdhury 2011-06-11 12:24:39 PDT
It is currently possible to set the content-type for blob-data. This patch also allows setting content-disposition.
Comment 1 Sadrul Habib Chowdhury 2011-06-11 12:28:41 PDT
Created attachment 96858 [details]
patch

I have added the tests, but haven't yet been able to run; keep getting "unexpected no expected result found" errors. Will update the patch if I can get the tests to run successfully and any change is necessary. But please take a look if the approach is OK.
Comment 2 WebKit Review Bot 2011-06-11 12:55:04 PDT
Comment on attachment 96858 [details]
patch

Attachment 96858 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/8829578

New failing tests:
fast/files/blob-content-disposition.html
Comment 3 WebKit Review Bot 2011-06-11 12:55:09 PDT
Created attachment 96859 [details]
Archive of layout-test-results from ec2-cr-linux-01

The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: ec2-cr-linux-01  Port: Chromium  Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Comment 4 Adam Barth 2011-06-11 13:22:57 PDT
Comment on attachment 96858 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=96858&action=review

> Source/WebCore/fileapi/WebKitBlobBuilder.idl:2
> - * Copyright (C) 2010 Google Inc. All rights reserved.
> + * Copyright (C) 2011 Google Inc. All rights reserved.

nit: WebKit appends years to the copyright line rather than replacing them.  No idea why.
Comment 5 Sadrul Habib Chowdhury 2011-06-11 14:31:40 PDT
Comment on attachment 96858 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=96858&action=review

>> Source/WebCore/fileapi/WebKitBlobBuilder.idl:2
>> + * Copyright (C) 2011 Google Inc. All rights reserved.
> 
> nit: WebKit appends years to the copyright line rather than replacing them.  No idea why.

Changed. Thanks.
Comment 6 Sadrul Habib Chowdhury 2011-06-11 14:32:38 PDT
Created attachment 96860 [details]
Fix license headers + test
Comment 7 Sadrul Habib Chowdhury 2011-06-11 15:28:27 PDT
Created attachment 96862 [details]
patch
Comment 8 Sadrul Habib Chowdhury 2011-06-11 15:30:01 PDT
Created attachment 96863 [details]
patch
Comment 9 WebKit Review Bot 2011-06-11 16:46:37 PDT
Comment on attachment 96863 [details]
patch

Attachment 96863 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/8833016

New failing tests:
fast/files/blob-content-disposition.html
Comment 10 WebKit Review Bot 2011-06-11 16:46:41 PDT
Created attachment 96865 [details]
Archive of layout-test-results from ec2-cr-linux-03

The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: ec2-cr-linux-03  Port: Chromium  Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Comment 11 Sadrul Habib Chowdhury 2011-06-11 17:19:29 PDT
Created attachment 96867 [details]
Fixed test, for real this time!

Finally figured out how to test the test! Please take a look!
Comment 12 Adam Barth 2011-06-12 00:53:45 PDT
Comment on attachment 96867 [details]
Fixed test, for real this time!

Is this in the spec?  Marking this R- for now.  Please feel to renominate if this is in the spec (I couldn't find it).
Comment 13 Sadrul Habib Chowdhury 2011-06-12 08:07:23 PDT
This indeed is not in the spec (http://www.w3.org/TR/file-writer-api/#widl-BlobBuilder-getBlob).