Fix some wrong use of NS_NewNativeLocalFile in dom

RESOLVED FIXED in Firefox 52

Status

()

Product:
Core
Component:
DOM: Core & HTML
Type:
defect
Status:
RESOLVED FIXED
Opened:
2 years ago
Updated:
3 months ago

People

(Reporter: baku, Assigned: baku)

Tracking

Version:
unspecified
Target:
mozilla53
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox51 wontfix, firefox52 fixed, firefox53 fixed)

Details

Attachments

(1 attachment)

fileSystem2.patch
3.39 KB, patch
smaug
: review+
gchang
: approval-mozilla-aurora+
gchang
: approval-mozilla-beta-
Details | Diff | Splinter Review
Assignee

Description

•
2 years ago 
No description provided.
Assignee

Comment 1

•
2 years ago
Assignee: nobody → amarchesini
Attachment #8827744 - Flags: review?(bugs)

Comment 2

•
2 years ago 
Comment on attachment 8827744 [details] [diff] [review]
fileSystem2.patch

Do we need this in aurora/beta too? That other bug definitely, so possibly also this?

Is this tested on Windows with non-ascii folders and file names?
Attachment #8827744 - Flags: review?(bugs) → review+

Comment 3

•
2 years ago 
Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5cf462d962ce
Replace NS_NewNativeLocalFile with NS_NewLocalFile in dom, r=smaug
Assignee

Comment 4

•
2 years ago 
Comment on attachment 8827744 [details] [diff] [review]
fileSystem2.patch

Approval Request Comment
[Feature/Bug causing the regression]: Directory upload API, mainly.
[User impact if declined]: on windows, if a directory is called with a non-ascii name, that name can be wrongly encoded.
[Is this code covered by automated tests?]: none
[Has the fix been verified in Nightly?]: not yet 
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: no
[Why is the change risky/not risky?]: We use a similar method but avoiding the double encoding.
[String changes made/needed]: none
Attachment #8827744 - Flags: approval-mozilla-beta?
Attachment #8827744 - Flags: approval-mozilla-aurora?

Comment 5

•
2 years ago
bugherder 
https://hg.mozilla.org/mozilla-central/rev/5cf462d962ce
Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox53: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53

Comment 6

•
2 years ago 
Comment on attachment 8827744 [details] [diff] [review]
fileSystem2.patch

It's too late for 51 as we've built 51 RC2. Mark 51 won't fix and Beta51-. We can take this in Aurora52. Aurora52+.
Attachment #8827744 - Flags: approval-mozilla-beta?
Attachment #8827744 - Flags: approval-mozilla-beta-
Attachment #8827744 - Flags: approval-mozilla-aurora?
Attachment #8827744 - Flags: approval-mozilla-aurora+
Blocks: 1331820
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.