Use MediaEvents to communicate between AbstractMediaDecoder and HTMLMediaElement

ASSIGNED
Assigned to

Status

()

P3
normal
ASSIGNED
2 years ago
a year ago

People

(Reporter: bryce, Assigned: kaku)

Tracking

(Blocks: 1 bug)

50 Branch
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

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

Updated

2 years ago
Assignee: nobody → bvandyk
Priority: -- → P3
(Assignee)

Comment 1

2 years ago
Hi SingingTree, 
Are you current working on this? If not, may I separate it into small parts and then solve gradually?
Flags: needinfo?(bvandyk)
(Reporter)

Comment 2

2 years ago
I am not working on this at the moment. By all means you may take it.
Assignee: bvandyk → nobody
Flags: needinfo?(bvandyk)
(Assignee)

Updated

a year ago
Assignee: nobody → kaku
Blocks: 1347406
Status: NEW → ASSIGNED
You need to log in before you can comment on or make changes to this bug.