Closed Bug 958888 Opened 6 years ago Closed 6 years ago

Use OS.File in the getAvailableFileName function (in WebappsInstaller)

Categories

(Firefox Graveyard :: Web Apps, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 29

People

(Reporter: marco, Assigned: marco)

References

Details

Attachments

(1 file)

Attached patch PatchSplinter Review
No description provided.
Attachment #8358865 - Flags: review?(myk)
Comment on attachment 8358865 [details] [diff] [review]
Patch

Nice!
Attachment #8358865 - Flags: review?(myk) → review+
Hey Myk, would you mind testing this on Mac?
Assignee: nobody → mar.castelluccio
Status: NEW → ASSIGNED
Flags: needinfo?(myk)
Flags: needinfo?(myk)
Priority: -- → P3
Erm, sorry, I cancelled that needinfo request accidentally.  I'll test on Mac, and I'm re-adding the needinfo request to track that action.
Flags: needinfo?(myk)
I tested it; it works!
Flags: needinfo?(myk)
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/bb0585e8c50e
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 29
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.