Closed Bug 1223825 Opened 4 years ago Closed 4 years ago

Change Directory.path to include the directory's name

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla45
Tracking Status
firefox45 --- fixed

People

(Reporter: jwatt, Assigned: jwatt)

Details

Attachments

(1 file)

https://wicg.github.io/directory-upload/proposal.html#dom-directory-path

Has been changed to say:

  On getting, this attribute must return the path from root of the
  present working directory, including its name.

Our current code goes out of its way to omit the Directory's name from its path, so it needs to be updated to include the Directory's name.
Attached patch patchSplinter Review
Attachment #8686083 - Flags: review?(amarchesini)
Comment on attachment 8686083 [details] [diff] [review]
patch

Review of attachment 8686083 [details] [diff] [review]:
-----------------------------------------------------------------

any test for this?
Attachment #8686083 - Flags: review?(amarchesini) → review+
I'm on PTO just now, but I'll follow-up on getting tests for this after I'm back.
https://hg.mozilla.org/mozilla-central/rev/f7db31318197
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.