1623740 - Allow the setting (and locking) of the default Autoplay value

Status: RESOLVED FIXED

(Firefox :: Enterprise Policies, defect, P3)

Description

[Mike Kaply [:mkaply]]

In bug 1588197 we added support for enabling autoplay for domains, but we didn't provide a way to set the global default.

We should add that.

Comment 1

[Mike Kaply [:mkaply]]

Attachment: Bug 1623740 - Add policy for setting and locking autoplay default. r?mconley

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/49ad9716f043
Add policy for setting and locking autoplay default. r=mconley

Comment 3

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/49ad9716f043

Comment 4

[Mike Kaply [:mkaply]]

Comment on attachment 9134514 [details]
Bug 1623740 - Add policy for setting and locking autoplay default. r?mconley

ESR Uplift Approval Request

• If this is not a sec:{high,crit} bug, please state case for ESR consideration: Policy related
• User impact if declined: Unable to set default autoplay policy.
• Fix Landed on Version: 76
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Policy only, automated test.
• String or UUID changes made by this patch:

Comment 5

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9134514 [details]
Bug 1623740 - Add policy for setting and locking autoplay default. r?mconley

New policy addition needed to maintain parity with release. Approved for 68.8esr.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9134514 [details]
Bug 1623740 - Add policy for setting and locking autoplay default. r?mconley

Actually, this has conflicts trying to graft to ESR68. Please attach a rebased patch.

Comment 7

[Mike Kaply [:mkaply]]

Attachment: ESR patch

ESR did not have the ability to globally block audio/video.

The conflict source was the new dropdown added in prefs (nothing in policy)

I've left the ability to set a value of 5 in the policy for simplicity; I'll document in the policy docs that it won't work for ESR.

Comment 8

[Mike Kaply [:mkaply]]

Patch attached.

Comment 9

[Mike Kaply [:mkaply]]

Comment on attachment 9141054 [details]
ESR patch

ESR Uplift Approval Request

• If this is not a sec:{high,crit} bug, please state case for ESR consideration: Parity with Firefox 76 policy
• User impact if declined: Missing policy
• Fix Landed on Version: 76
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Policy only
• String or UUID changes made by this patch:

Comment 10

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9141054 [details]
ESR patch

Approved for 68.8esr.

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-esr68/rev/3cf737b05dd9