Closed Bug 1361000 Opened 7 years ago Closed 7 years ago

Add deprecation warning for removing support for EME on insecure contexts

Categories

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

enhancement

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: cpearce, Assigned: cpearce)

References

(Blocks 1 open bug)

Details

(Keywords: dev-doc-complete, site-compat)

Attachments

(1 file)

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 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+
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]"
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
https://hg.mozilla.org/mozilla-central/rev/1242d70ee6a2
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: