Closed Bug 977437 Opened 10 years ago Closed 9 years ago

Add pref to disable the whitelist in the gstreamer media backend

Categories

(Core :: Audio/Video: Playback, defect, P5)

x86
macOS
defect

Tracking

()

RESOLVED WONTFIX

People

(Reporter: alessandro.d, Unassigned)

References

Details

Attachments

(1 file)

The patch adds a media.gstreamer.disable-whitelist pref (false by default) which when set to true lets gstreamer decode every format it supports. This is always going to stay false of course, but overriding it can be useful for local testing and XUL apps.
Attached patch whitelist.patchSplinter Review
Attachment #8382709 - Flags: feedback?(edwin)
Attachment #8382709 - Flags: feedback?(cpearce)
Blocks: 976992
btw with this patch applied and the pref turned on, HLS and DASH are suddenly supported :P
Comment on attachment 8382709 [details] [diff] [review]
whitelist.patch

Review of attachment 8382709 [details] [diff] [review]:
-----------------------------------------------------------------

No. We don't want to encourage media format proliferation in web content. Even with something that's preffed off by default. Sorry.
Attachment #8382709 - Flags: feedback?(edwin)
Attachment #8382709 - Flags: feedback?(cpearce)
Attachment #8382709 - Flags: feedback-
Hum. How does this encourage media proliferation in web content in practice? Are you worried that people would en masse turn the pref on? If so, how is that a concern for this pref specifically but not any other pref? (honest question)
I'd be worried that some website would use a format not supported, but viewable with this pref toggled, then give step-by-step instructions to users to turn it on. The other possibility is requiring the installation of an addon to flip the pref. It's stupid, but I can easily envision it happening.

Env var instead?
I really don't see why someone would go through the pain (for them and their users) of doing a step-by-step guide to enable a pref when transcoding in a supported format is likely less work. 

About addons... well, one can install addons to do everything, including decoding video directly (https://github.com/alessandrod/gst-browser-plugin).

The reason why I care about this patch is that it allows me to dogfood the gst media backend. Otherwise I never use it, since I don't use linux and I mostly ignore youtube. In a way it's a way to ensure I don't lose interest in hacking it :)
Component: Audio/Video → Audio/Video: Playback
gstreamer is going in bug 1234092
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: