[MSE] Add support for event handlers

RESOLVED FIXED in Firefox 50

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: jya, Assigned: jya)

Tracking

(Blocks: 1 bug)

Trunk
mozilla50
Points:
---

Firefox Tracking Flags

(firefox50 fixed)

Details

Attachments

(2 attachments)

(Assignee)

Description

2 years ago
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)

Comment 1

2 years ago
Created attachment 8770859 [details]
Bug 1286722: [MSE] P1. Add mediasource event handler attributes.

See https://github.com/w3c/media-source/issues/66

Review commit: https://reviewboard.mozilla.org/r/64190/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/64190/
Attachment #8770859 - Flags: review?(bzbarsky)
Attachment #8770860 - Flags: review?(gsquelart)
(Assignee)

Comment 2

2 years ago
Created attachment 8770860 [details]
Bug 1286722: [MSE] P2. Add mochitests for event handler attributes.

Review commit: https://reviewboard.mozilla.org/r/64192/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/64192/
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+
(Assignee)

Comment 4

2 years ago
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/
(Assignee)

Comment 5

2 years ago
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+

Comment 7

2 years ago
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

Comment 8

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/501b4b7d83ea
https://hg.mozilla.org/mozilla-central/rev/5b9850e2567a
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox50: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.