If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Directory Upload API doesn't work when DnD folder

RESOLVED FIXED in Firefox 54

Status

()

Core
DOM
RESOLVED FIXED
7 months ago
7 months ago

People

(Reporter: baku, Assigned: baku)

Tracking

unspecified
mozilla54
Points:
---

Firefox Tracking Flags

(firefox54 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 months ago
I know that this API is not enabled, but I was testing this issue and I found this bug.
(Assignee)

Comment 1

7 months ago
Created attachment 8838011 [details] [diff] [review]
aa.patch
Assignee: nobody → amarchesini
Attachment #8838011 - Flags: review?(bugs)

Comment 2

7 months ago
Comment on attachment 8838011 [details] [diff] [review]
aa.patch

I guess we can do this.
Attachment #8838011 - Flags: review?(bugs) → review+

Comment 3

7 months ago
Comment on attachment 8838011 [details] [diff] [review]
aa.patch

>+      // Entries API.
>+      if (blinkFileSystemEnabled) {
>       // This is rather ugly. Pass the directories as Files using SetFiles,
>       // but then if blink filesystem API is enabled, it wants
>       // FileOrDirectory array.
This comment is missing some indentation now

Comment 4

7 months ago
Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c9127259d432
Directory Upload API should work correctly with a folder is dropped, r=smaug

Comment 5

7 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/c9127259d432
Status: NEW → RESOLVED
Last Resolved: 7 months ago
status-firefox54: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in before you can comment on or make changes to this bug.