58 bytes, text/x-review-board-request
Through showing changing dates in the file list of file:///. We should point to a local file instead.
Probably simplest to resolve mozscreenshots.html to a file URI. IIRC there is an API to resolve a chrome URI (chrome://mozscreenshots/content/lib/mozscreenshots.html) to a file URI .
Priority: -- → P2
Whiteboard: [fxprivacy] [triage] → [fxprivacy]
Comment on attachment 8781867 [details] Bug 1295488 - Don't use a directory for mozscreenshots localFile tests. https://reviewboard.mozilla.org/r/72210/#review69772 Please review the localFile screenshots and the URLs in the address bar to make sure the URL doesn't change based upon the machine or other variables. If not, then r+. Linux opt looks fine: https://mozilla-releng-blobs.s3.amazonaws.com/blobs/try/sha512/40a7e521e296ed7ed62bc9ee1f7f5e3735f89453866f9fd39493bb9671e27a2eeac70827af114d93d8aff3ce18755b79fe04148e4139112cab002b7564f9f78f
Attachment #8781867 - Flags: review?(MattN+bmo) → review+
Yup, timing out on Windows again, I'll boot up my VM :(
Comment on attachment 8781867 [details] Bug 1295488 - Don't use a directory for mozscreenshots localFile tests. https://reviewboard.mozilla.org/r/72210/#review69968
https://hg.mozilla.org/integration/fx-team/rev/a6f67ab4a4ec4dd66f53c4b4423e5b8b63023388 Bug 1295488 - Don't use a directory for mozscreenshots localFile tests. r=MattN
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox51: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 51
Component: General → mozscreenshots
Product: Firefox → Testing
Target Milestone: Firefox 51 → ---
Version: Trunk → unspecified
You need to log in before you can comment on or make changes to this bug.