Closed
Bug 1304604
Opened 9 years ago
Closed 9 years ago
[EME] Clean up HTMLMediaElement.onencrypted
Categories
(Core :: Audio/Video: Playback, defect)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
FIXED
mozilla52
| Tracking | Status | |
|---|---|---|
| firefox52 | --- | fixed |
People
(Reporter: cpearce, Unassigned)
References
Details
Attachments
(1 file)
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 hidden (mozreview-request) |
|
||
Comment 2•9 years ago
|
||
| mozreview-review | ||
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+
|
Reporter | |
Comment 3•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/e6014b4e1bb361ad117525d26ea6577dd9c09789
Bug 1304604 - Use EventTarget to simplyfy HTMLMediaElement.onencrypted implementation. r=smaug
|
||
Comment 4•9 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox52:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
|
|
||
Comment 5•9 years ago
|
||
| bugherder | ||
You need to log in
before you can comment on or make changes to this bug.
Description
•