Found via bug 898598. The give-away is the line in osfile_win_allthreads.jsm: this._lastModificationDate = lastAccessDate; :)
Created attachment 782958 [details] [diff] [review] osfile-last-modified-time.patch The tests seem to pass with this change, but I'm not sure if anything else is necessary?
Assignee: nobody → mhammond
Attachment #782958 - Flags: review?(dteller)
Ahaha, that would certainly be useful.
Comment on attachment 782958 [details] [diff] [review] osfile-last-modified-time.patch Review of attachment 782958 [details] [diff] [review]: ----------------------------------------------------------------- Looks good, thanks.
Attachment #782958 - Flags: review?(dteller) → review+
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
Any chance of landing this in Firefox 24 for the ESR? It looks pretty straightforward, and this bug is preventing us from using OS.File for various fairly impactful things involving hundreds or thousands of files in Zotero. At best, it will require us to maintain two versions of code (asynchronous and synchronous) for another year, and Zotero users on the ESR will have significantly worse performance.
Looks pretty straightforward, I don't see any reason not to uplift. Mark, want to request approvals?
Comment on attachment 782958 [details] [diff] [review] osfile-last-modified-time.patch [Approval Request Comment] Bug caused by (feature/regressing bug #): Unknown User impact if declined: Addon authors are unable to OS.File in some contexts. Testing completed (on m-c, etc.): Landed on M-C on July 31. Risk to taking this patch (and alternatives if risky): Low risk, limited to OS.File functionality on Windows. String or IDL/UUID changes made by this patch: None Please see comment 6 and comment 7
status-firefox24: --- → affected
status-firefox25: --- → affected
status-firefox26: --- → verified disabled
Sorry for the spam, it seems I mis-clicked on the flags
status-firefox26: verified disabled → fixed
This landed on m-c for Fx25, so this doesn't need uplifting to Aurora. https://hg.mozilla.org/releases/mozilla-beta/rev/6c752934f8b5
status-firefox24: affected → fixed
status-firefox25: affected → fixed
status-firefox26: fixed → ---
Assuming no verification needed here. Please add the verifyme keyword and remove the [qa-] whiteboard tag to request verification.
You need to log in before you can comment on or make changes to this bug.