1289634 - Enable Widevine EME and don't enable Primetime EME in default build configuration

Status: RESOLVED FIXED

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

Description

[Chris Pearce [:cpearce (Not reading bugmail)]]

Primetime only works in Mozilla built Firefox builds because it requires a signed plugin container. Widevine however, works in Firefox build by anyone. So we should enable Widevine by default, and but not enable Primetime, in a default build.

However, we should only download and activate the Widevine CDM on proprietary platforms (MacOSX, Windows), as users on open source platforms (i.e. Linux) have not opted in to downloading and running arbitrary binaries. We should however on Linux prompt the user the first time the CDM is requested, and ask them if they want to "enable DRM", and if they click OK, we can then download and use the CDM.

So I guess we want to add a --disable-eme={widevine,primetime} config option, and have Widevine on by default.

Comment 1

[Chris Pearce [:cpearce (Not reading bugmail)]]

glandium: any comments/guidance on how we should arrange the new/existing ac_add_options here to enable Widevine and disable Adobe Primetime by default?

Comment 2

[Mike Hommey [:glandium]]

You can just add "default='widevine'" to the existing option.
Then in our mozconfigs, you can --enable-eme=+adobe, and that should pick both. Or if you want to be more explicit, you can still do --enable-eme=adobe,widevine.

Comment 3

[Stebs]

(In reply to Chris Pearce (:cpearce) from comment #0)
> We should however on Linux prompt the user the first time the CDM
> is requested, and ask them if they want to "enable DRM", and if they click
> OK, we can then download and use the CDM.
Shouldn't browser.eme.ui.enabled also be set to true on linux?

Comment 4

[Chris Pearce [:cpearce (Not reading bugmail)]]

(In reply to Stebs from comment #3)
> (In reply to Chris Pearce (:cpearce) from comment #0)
> > We should however on Linux prompt the user the first time the CDM
> > is requested, and ask them if they want to "enable DRM", and if they click
> > OK, we can then download and use the CDM.
> Shouldn't browser.eme.ui.enabled also be set to true on linux?

That gets enabled automatically if any CDM is enabled.

Comment 5

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Bug 1289634 - Enable Widevine EME by default in browser builds.

Review commit: https://reviewboard.mozilla.org/r/69456/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69456/

Comment 6

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Bug 1289634 - Remove Widevine from Mozilla official mozconfigs, since it's enabled by default now.

Review commit: https://reviewboard.mozilla.org/r/69458/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69458/

Comment 7

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Bug 1289634 - Disable Widevine CDM by default on Linux, but prompt user to enable and download on first use.

We don't want the Widevine CDM to be downloaded by default on Linux, as
the CDM is proprietary software and the user hasn't opted in to having such
software on their system. By leaving the CDM visible but disabled, we'll
prompt the user the first time EME is used, and they can approve the
download.

This means the Widevine CDM won't be downloaded by default. The user will
need to opt-in to enablding DRM playback before we'll download proprietary
CDMs.

Review commit: https://reviewboard.mozilla.org/r/69460/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69460/

Comment 8

[Gerald Squelart (he/him) (not at Mozilla since 2022-09-15)]

Comment on attachment 8778061 [details]
Bug 1289634 - Disable Widevine CDM by default on Linux, but prompt user to enable and download on first use.

https://reviewboard.mozilla.org/r/69460/#review66596

Comment 9

[Mike Hommey [:glandium]]

Comment on attachment 8778059 [details]
Bug 1289634 - Enable Widevine EME by default in browser builds.

https://reviewboard.mozilla.org/r/69456/#review66660

Comment 10

[Mike Hommey [:glandium]]

Comment on attachment 8778060 [details]
Bug 1289634 - Remove Widevine from Mozilla official mozconfigs, since it's enabled by default now.

https://reviewboard.mozilla.org/r/69458/#review66662

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5eaaa9b300cc
Enable Widevine EME by default in browser builds. r=glandium
https://hg.mozilla.org/integration/autoland/rev/8425f7f24096
Remove Widevine from Mozilla official mozconfigs, since it's enabled by default now. r=glandium
https://hg.mozilla.org/integration/autoland/rev/14ff1748f948
Disable Widevine CDM by default on Linux, but prompt user to enable and download on first use. r=gerald

Comment 12

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/5eaaa9b300cc
https://hg.mozilla.org/mozilla-central/rev/8425f7f24096
https://hg.mozilla.org/mozilla-central/rev/14ff1748f948

Comment 13

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=fbe876a60b42

Comment 14

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=68b989dc2669

Comment 15

[Chris Pearce [:cpearce (Not reading bugmail)]]

Comment on attachment 8778059 [details]
Bug 1289634 - Enable Widevine EME by default in browser builds.

Approval Request Comment
[Feature/regressing bug #]: Widevine EME on Linux
[User impact if declined]: Widevine EME won't be enabled or visible in the Firefox UI on Linux.
[Describe test coverage new/current, TreeHerder]: We have plenty of EME mochitests, which run on Linux.
[Risks and why]: Low; this is tweaking the build system. It'll be obvious if it's not working!
[String/UUID change made/needed]: None

Comment 16

[Chris Pearce [:cpearce (Not reading bugmail)]]

Comment on attachment 8778060 [details]
Bug 1289634 - Remove Widevine from Mozilla official mozconfigs, since it's enabled by default now.

Approval Request Comment
[Feature/regressing bug #]: Widevine EME on Linux
[User impact if declined]: Widevine EME won't be enabled or visible in the Firefox UI on Linux.
[Describe test coverage new/current, TreeHerder]: We have plenty of EME mochitests, which run on Linux.
[Risks and why]: Low; this is tweaking the build system. It'll be obvious if it's not working!
[String/UUID change made/needed]: None

Comment 17

[Chris Pearce [:cpearce (Not reading bugmail)]]

Comment on attachment 8778061 [details]
Bug 1289634 - Disable Widevine CDM by default on Linux, but prompt user to enable and download on first use.

Approval Request Comment
[Feature/regressing bug #]: Widevine EME on Linux
[User impact if declined]: Widevine EME won't be enabled or visible in the Firefox UI on Linux.
[Describe test coverage new/current, TreeHerder]: We have plenty of EME mochitests, which run on Linux.
[Risks and why]: Low; this is tweaking the build system. It'll be obvious if it's not working!
[String/UUID change made/needed]: None

Comment 18

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Comment on attachment 8778059 [details]
Bug 1289634 - Enable Widevine EME by default in browser builds.

Widevine EME on Linux, Aurora50+

Comment 19

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Comment on attachment 8778060 [details]
Bug 1289634 - Remove Widevine from Mozilla official mozconfigs, since it's enabled by default now.

Widevine EME on Linux, Aurora50+

Comment 20

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Comment on attachment 8778061 [details]
Bug 1289634 - Disable Widevine CDM by default on Linux, but prompt user to enable and download on first use.

Widevine EME on Linux, Aurora50+

Comment 21

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=0d5ce136db93

Comment 22

[Liz Henry (:lizzard) (relman/hg->git project)]

Comment on attachment 8778059 [details]
Bug 1289634 - Enable Widevine EME by default in browser builds.

Fix the defaults and config for Widevine. OK for beta 3.

Comment 23

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://hg.mozilla.org/releases/mozilla-beta/rev/4f3cf594c468
https://hg.mozilla.org/releases/mozilla-beta/rev/546319b9e7bb
https://hg.mozilla.org/releases/mozilla-beta/rev/b950c832d4df

Comment 24

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://hg.mozilla.org/releases/mozilla-beta/rev/4f3cf594c468
https://hg.mozilla.org/releases/mozilla-beta/rev/546319b9e7bb
https://hg.mozilla.org/releases/mozilla-beta/rev/b950c832d4df

Comment 25

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/c44ddf4b793791d33ebcacd5eff4a66fe15be3cc
https://hg.mozilla.org/releases/mozilla-aurora/rev/7525e55ce030f3d977d3650fab4c026c3614f66b
https://hg.mozilla.org/releases/mozilla-aurora/rev/e74314c1717e9406417014786f00d06f7c0e8b00