1585284 - Resolving the dataTransfer.files[n].text promise when triggered from a drag and drop event causes a NS_ERROR_FAILURE error

Status: RESOLVED FIXED

(Core :: DOM: File, defect, P2)

Description

[bluebottel]

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:69.0) Gecko/20100101 Firefox/69.0

Steps to reproduce:

Demo here: http://jsfiddle.net/0q5k6m9d/

Set up a drop zone and add a ondrop handler. In the ondrop handler function attempt to resolve the promise to get the file content like so

event.dataTransfer.files[0].text(fileContent => { ... }

I tested this on Ubuntu 18.04 and Windows 10 with Firefox 69.01

Actual results:

"NS_ERROR_FAILURE" is written to the console and the execution of the script is halted.

Expected results:

The file's content should be accessible after the promise resolves.

Comment 1

[Hsin-Yi Tsai (she/her) [:hsinyi]]

Looks the NS_ERROR_FAILURE may be thrown from Blob implementation... moving this bug to DOM:File ...

Comment 2

[Andrew Overholt [:overholt]]

Looks like baku (somewhat recently) implemented this in bug 1557121 so he may have the most informed opinion.

Comment 3

[Andrea Marchesini [:baku]]

Attachment: Bug 1585284 - Use nsIGlobalObject in any Blob/File CTOR, r?smaug

Comment 4

[Andrea Marchesini [:baku]]

Attachment: Bug 1585284 - Use BlobImpl in MutableBlobStorage instead of Blob, r?smaug

Comment 5

[Andrea Marchesini [:baku]]

Attachment: Bug 1585284 - Force global in Blob CTOR, r?smaug

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/30100f2c6128
Use nsIGlobalObject in any Blob/File CTOR, r=smaug
https://hg.mozilla.org/integration/autoland/rev/bee7aebbca69
Use BlobImpl in MutableBlobStorage instead of Blob, r=smaug
https://hg.mozilla.org/integration/autoland/rev/87469bcb7a13
Force global in Blob CTOR, r=smaug

Comment 7

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/30100f2c6128
https://hg.mozilla.org/mozilla-central/rev/bee7aebbca69
https://hg.mozilla.org/mozilla-central/rev/87469bcb7a13