Mozscreenshots ControlCenter localFile configuration is constantly triggering differences

RESOLVED FIXED

Status

P2
normal
RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: johannh, Assigned: johannh)

Tracking

({regression})

unspecified
regression
Points:
---

Firefox Tracking Flags

(firefox51 fixed)

Details

(Whiteboard: [fxprivacy] )

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
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 .

Updated

2 years ago
Keywords: regression
Priority: -- → P2
Whiteboard: [fxprivacy] [triage] → [fxprivacy]
Comment hidden (mozreview-request)
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+
(Assignee)

Comment 5

2 years ago
Yup, timing out on Windows again, I'll boot up my VM :(
Comment hidden (mozreview-request)
Comment on attachment 8781867 [details]
Bug 1295488 - Don't use a directory for mozscreenshots localFile tests.

https://reviewboard.mozilla.org/r/72210/#review69968
(Assignee)

Comment 9

2 years ago
https://hg.mozilla.org/integration/fx-team/rev/a6f67ab4a4ec4dd66f53c4b4423e5b8b63023388
Bug 1295488 - Don't use a directory for mozscreenshots localFile tests. r=MattN

Comment 10

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/a6f67ab4a4ec
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.