[EME] Clean up HTMLMediaElement.onencrypted

RESOLVED FIXED in Firefox 52

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: cpearce, Unassigned)

Tracking

(Blocks 1 bug)

unspecified
mozilla52
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox52 fixed)

Details

Attachments

(1 attachment)

The comments made in reviews of bug 1304250 and friends also apply to the implementation of HTMLMediaElement.onencrypted.

That is, we can just use EventTarget::{Get,Set}EventHandler() in HTMLMediaElement::{Get,Set}Onencrypted, and its parameter should be aCallback.
Comment on attachment 8793610 [details]
Bug 1304604 - Use EventTarget to simplyfy HTMLMediaElement.onencrypted implementation.

https://reviewboard.mozilla.org/r/80322/#review79024
Attachment #8793610 - Flags: review?(bugs) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/e6014b4e1bb361ad117525d26ea6577dd9c09789
Bug 1304604 - Use EventTarget to simplyfy HTMLMediaElement.onencrypted implementation. r=smaug
https://hg.mozilla.org/mozilla-central/rev/e6014b4e1bb3
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.