Make media.eme.enabled locking work properly
Categories
(Firefox :: Enterprise Policies, enhancement, P3)
Tracking
()
People
(Reporter: mkaply, Assigned: mkaply)
Details
Attachments
(4 files)
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
jcristau
:
approval-mozilla-esr68+
|
Details | Review |
3.81 KB,
patch
|
jcristau
:
approval-mozilla-esr68+
|
Details | Diff | Splinter Review |
47 bytes,
text/x-phabricator-request
|
Details | Review |
Currently we have a pref to turn off media.eme.enabled, but the locked state is ignored by Firefox.
We should implement a proper policy with locking/unlocking and add the check in MediaKeySystemAccessManager.cpp
- MediaKeySystemAccess::NotifyObservers(mWindow, aRequest->mKeySystem,
- MediaKeySystemStatus::Api_disabled);
+ if (!Preferences::IsLocked("media.eme.enabled")) {
+ MediaKeySystemAccess::NotifyObservers(mWindow, aRequest->mKeySystem,
+ MediaKeySystemStatus::Api_disabled);
+ }
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 1•4 years ago
|
||
Assignee | ||
Comment 2•4 years ago
|
||
Pushed by mozilla@kaply.com: https://hg.mozilla.org/integration/autoland/rev/3e560ed2592e Don't show DRM install message if eme pref is locked. r=bryce
Comment 4•4 years ago
|
||
bugherder |
Assignee | ||
Comment 5•4 years ago
|
||
Forgot to mark leave open.
Updated•4 years ago
|
Pushed by mozilla@kaply.com: https://hg.mozilla.org/integration/autoland/rev/74d737283c49 Add policy for Encrypted Media Extensions. r=mconley,fluent-reviewers,flod
Comment 7•4 years ago
|
||
bugherder |
Assignee | ||
Comment 8•4 years ago
|
||
[Approval Request Comment]
If this is not a sec:{high,crit} bug, please state case for ESR consideration: Fix to make policy work properly
User impact if declined: When EME pref is locked, UI shows but doesn't work.
Fix Landed on Version: 77
Risk to taking this patch (and alternatives if risky): Low, test and only kicks in if policy is set.
See https://wiki.mozilla.org/Release_Management/ESR_Landing_Process for more info.
Assignee | ||
Comment 9•4 years ago
|
||
Comment on attachment 9140267 [details]
Bug 1628871 - Add policy for Encrypted Media Extensions. r?mconley
ESR Uplift Approval Request
- If this is not a sec:{high,crit} bug, please state case for ESR consideration: New policy
- User impact if declined: Policy unavailable
- Fix Landed on Version: 77
- 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: New policy string.
Updated•4 years ago
|
Comment 10•4 years ago
|
||
Comment on attachment 9145944 [details] [diff] [review] ESR version of first patch new policy, approved for 68.9
Updated•4 years ago
|
Comment 11•4 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-esr68/rev/20de7a699084
https://hg.mozilla.org/releases/mozilla-esr68/rev/84c0d49497b0
Updated•4 years ago
|
Comment 12•4 years ago
|
||
Comment 13•4 years ago
|
||
bugherder uplift |
Description
•