1528343 - Add an option to block muted videos

Status: NEW

(Core :: Audio/Video: Playback, enhancement, P3)

Description

[chsiang]

Target to land before March 8th, pending Dale's availability
Eric to link UX specs here

Comment 1

[Eric Pang [:epang] UX]

Link to UX Specs: https://mozilla.invisionapp.com/share/P2QIWZ0ZWQ7

Comment 2

[chsiang]

thanks Eric.
Hi Dale,
Here is the bug after you get other stuff done.

Comment 3

[Dale Harvey (:daleharvey)]

Hey Johann, interested in your take on this

I think this is potentially problematic UI, we dont have any ability as far as I know to hook blocking videos on the same permission as block-autoplay, which means a new permission I believe

What do we do if we have block-autoplay-video=allowed and block-autoplay=blocked? There isnt anywhere to configure the whitelist for block-autoplay-video and is a user going to be able to understand how they interact if we add another whitelist?

Cheers
Dale

Comment 4

[Johann Hofmann [:johannh]]

Yeah, having two permissions sounds like the horrible thing we're doing with cookieBehavior and and lifetimePolicy. Let me give you this advice: If you plan to show it in a single menulist in the UI it should be a single permission.

The permission manager is just an origin to integer store, you could invent another value to signify that all autoplay should be blocked, other projects have done similar things and in my experience it has worked much better than consolidating two prefs.

Comment 5

[Jeff Fortin]

See also bug #1650711 ?