Closed Bug 1286722 Opened 4 years ago Closed 4 years ago

[MSE] Add support for event handlers

Categories

(Core :: Audio/Video: Playback, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla50
Tracking Status
firefox50 --- fixed

People

(Reporter: jya, Assigned: jya)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

There are various event handlers that where recently added to the spec (issue 66)

those are:
MediaSource.onsourceopen
MediaSource.onsourceended
MediaSource.onsourceclose

SourceBuffer.onupdatestart
SourceBuffer.onupdate
SourceBuffer.onupdateend
SourceBuffer.onerror
SourceBuffer.onabort

SourceBufferList.onaddsourcebuffer
SourceBufferList.onremovesourcebuffer
Assignee: nobody → jyavenard
Comment on attachment 8770860 [details]
Bug 1286722: [MSE] P2. Add mochitests for event handler attributes.

https://reviewboard.mozilla.org/r/64192/#review61228
Attachment #8770860 - Flags: review?(gsquelart) → review+
Comment on attachment 8770859 [details]
Bug 1286722: [MSE] P1. Add mediasource event handler attributes.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/64190/diff/1-2/
Comment on attachment 8770860 [details]
Bug 1286722: [MSE] P2. Add mochitests for event handler attributes.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/64192/diff/1-2/
Comment on attachment 8770859 [details]
Bug 1286722: [MSE] P1. Add mediasource event handler attributes.

https://reviewboard.mozilla.org/r/64190/#review61336

r=me
Attachment #8770859 - Flags: review?(bzbarsky) → review+
Pushed by jyavenard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/501b4b7d83ea
[MSE] P1. Add mediasource event handler attributes. r=bz
https://hg.mozilla.org/integration/autoland/rev/5b9850e2567a
[MSE] P2. Add mochitests for event handler attributes. r=gerald
https://hg.mozilla.org/mozilla-central/rev/501b4b7d83ea
https://hg.mozilla.org/mozilla-central/rev/5b9850e2567a
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.