Add deprecation warning for removing support for EME on insecure contexts

RESOLVED FIXED in Firefox 55

Status

()

enhancement
P3
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: cpearce, Assigned: cpearce)

Tracking

(Blocks 1 bug, {dev-doc-complete, site-compat})

unspecified
mozilla55
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

In bug 1322517 we will remove support for EME in insecure contexts. We should add a warning when EME is used in insecure contexts, so authors have time to transition.

We have 1 day of data on the telemetry probe I added to determine what proportion of contexts where EME is used are insecure, and it's showing 15% of contexts are insecure.

https://mzl.la/2oWGeWs
Comment hidden (mozreview-request)

Comment 2

2 years ago
mozreview-review
Comment on attachment 8863306 [details]
Bug 1361000 - Log a deprecation warning to the web console upon use of EME in an insecure context.

https://reviewboard.mozilla.org/r/135100/#review138128

Does the console show which url (of the page, that is) is involved?   If not, please include it in the warning.
Attachment #8863306 - Flags: review?(bzbarsky) → review+
Comment hidden (mozreview-request)

Comment 4

2 years ago
Pushed by cpearce@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c733f9dbec85
Log a deprecation warning to the web console upon use of EME in an insecure context. r=bz
Backed out in https://hg.mozilla.org/integration/autoland/rev/a812a96caba2 - built nicely on Windows, but everything else failed like https://treeherder.mozilla.org/logviewer.html#?job_id=95817339&repo=autoland, "Navigator.cpp:2032:31: error: ignoring return value of 'nsresult nsIDocument::GetDocumentURI(nsString&) const', declared with attribute warn_unused_result [-Werror=unused-result]"
Comment hidden (mozreview-request)

Comment 7

2 years ago
Pushed by cpearce@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1242d70ee6a2
Log a deprecation warning to the web console upon use of EME in an insecure context. r=bz

Comment 8

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/1242d70ee6a2
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.