Pref Screenshots off in Beta/Release

RESOLVED FIXED

Status

()

enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: _6a68, Assigned: _6a68)

Tracking

unspecified
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

We're going to gradually roll out Screenshots in releases other than Nightly, so let's conditionally pref it off in those platforms.
Assignee: nobody → jhirsch
Oops! Updated the wrong bug.
Keywords: checkin-needed
Comment on attachment 8873931 [details]
Bug 1369824 - Pref Screenshots off in beta and release;

https://reviewboard.mozilla.org/r/145308/#review149492

::: browser/app/profile/firefox.js:1688
(Diff revision 1)
> +#ifdef NIGHTLY_BUILD
> +pref("extensions.screenshots.system-disabled", false);
> +#else
> +pref("extensions.screenshots.system-disabled", true);
> +#endif
> +pref("extensions.screenshots.system-disabled", true);

This looks wrong?
Comment on attachment 8873931 [details]
Bug 1369824 - Pref Screenshots off in beta and release;

Cancelling whilst waiting for an update.
Attachment #8873931 - Flags: review?(standard8)
Comment on attachment 8873931 [details]
Bug 1369824 - Pref Screenshots off in beta and release;

https://reviewboard.mozilla.org/r/145308/#review151336

That's better, thanks!
Attachment #8873931 - Flags: review?(standard8) → review+
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b86fec2a2ed8
Pref Screenshots off in beta and release; r=standard8
https://hg.mozilla.org/mozilla-central/rev/b86fec2a2ed8
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.