Platforms that can't --enable-gmp-sandbox shouldn't load GeckoMediaPlugins

NEW
Unassigned

Status

()

Core
WebRTC: Audio/Video
P3
normal
Rank:
25
3 years ago
4 months ago

People

(Reporter: jld, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

3 years ago
+++ This bug was initially created as a clone of Bug #1120045 +++

(In reply to Randell Jesup [:jesup] from bug 1120045 comment #7)
> Seems straightforward, unless I'm missing something:
> 
> 1. Platform or system that doesn't support sandboxing (BSD, etc):
>    Default: disallow GMP (all)
>    pref set: allow unsandboxed GMP/OpenH264 (but not CDMs)
> 2. Platform/system does allow sandboxing:
>    Allow GMP (all)

This indicates to me that --enable-gmp-sandbox / MOZ_GMP_SANDBOX should be changed to control whether GMP support is part of Gecko — and probably renamed to drop the “sandbox”.
(Reporter)

Comment 1

3 years ago
Also relevant here: B2G.  Currently --disable-gmp-sandbox and currently not using media plugins, but that might change.
(Reporter)

Comment 2

3 years ago
(In reply to Jed Davis [:jld] from comment #1)
> Also relevant here: B2G.  Currently --disable-gmp-sandbox and currently not
> using media plugins, but that might change.

s/might/will/.  Bug 1142899.
Jed - Can you take this?  Thanks
Rank: 15
Flags: needinfo?(jld)
Flags: firefox-backlog+
Priority: -- → P1
(Reporter)

Comment 4

3 years ago
Should B2G be exempt from this requirement, or will it also need GMP sandboxing?

In either case it might make sense to allow GMP sandboxing to remain optional on B2G for now, and then make it mandatory when and if it's all working.
Assignee: nobody → jld
Flags: needinfo?(jld) → needinfo?(mreavy)
(In reply to Jed Davis [:jld] from comment #4)
> Should B2G be exempt from this requirement, or will it also need GMP
> sandboxing?
> 
> In either case it might make sense to allow GMP sandboxing to remain
> optional on B2G for now, and then make it mandatory when and if it's all
> working.

I'm good with going forward in this bug under those assumptions, but before we go too far, we should reach out to B2G folks and cpearce and get their buy-in.
Flags: needinfo?(mreavy)

Updated

3 years ago
backlog: --- → webRTC+
Flags: firefox-backlog+
Rank: 15 → 25
Priority: P1 → P2
(Reporter)

Updated

3 years ago
See Also: → bug 1182565
(Reporter)

Updated

7 months ago
Assignee: jld → nobody
(Reporter)

Updated

7 months ago
See Also: → bug 1375863
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
You need to log in before you can comment on or make changes to this bug.