1296964 - Use MediaEvents to communicate between AbstractMediaDecoder and HTMLMediaElement

Status: NEW

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

Description

[Bryce Seager van Dyk [:bryce] (he/him) - Not reading bugmail]

The MediaEvent mechanism from MediaEventSource.h can be used to reduce coupling between AbstractMediaDecoder and HTMLMediaElement. As it stands, we have the HTMLMediaElement owning its decoder, but GetOwner() is also used on the decoder to obtain references to the HTMLMediaElement to notify events.

This could be replaced by having the decoder expose events using the MediaEvent mechanism, and html media element listen to the producer. This has been done in Bug 1145011, and it would be good to use this pattern more, both for the intrinsic benefits, and for the sake of consistency.

Comment 1

[Tzuhao Kuo [:kaku]]

Hi SingingTree, 
Are you current working on this? If not, may I separate it into small parts and then solve gradually?

Comment 2

[Bryce Seager van Dyk [:bryce] (he/him) - Not reading bugmail]

I am not working on this at the moment. By all means you may take it.

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee is inactive on Bugzilla, so the assignee is being reset.