1571493 - Decouple the media control from AudioChannelService

Status: RESOLVED FIXED

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

Description

[Alastor Wu [:alwu]]

In bug 1565689, we implement a centralized media control service in the chrome process which can control all media running in different content processes. For now, we use the implementation of the AudioChannelService to achieve controlling media, but I think it would be good to decouple them in the future, because AudioChannelSerivce itself has been too complicated from its original purpose, which is used to mute/unmute media.

Comment 2

[Alastor Wu [:alwu]]

Attachment: Bug 1571493 - part1 : implement 'ContentMediaController' which is used to notify 'MediaControlKeysEvent' for controlled media.

Comment 3

[Alastor Wu [:alwu]]

Attachment: Bug 1571493 - part2 : use 'MediaControlAgent' in media element.

Comment 4

[Alastor Wu [:alwu]]

Attachment: Bug 1571493 - part3 : notify media state and audible state changes.

Comment 5

[Alastor Wu [:alwu]]

Attachment: Bug 1571493 - part4 : remove the usage of MediaControlUtils's helper functions.

Comment 6

[Alastor Wu [:alwu]]

Attachment: Bug 1571493 - part4 : support 'ControlledMediaState' in media controller

Comment 7

[Alastor Wu [:alwu]]

Attachment: Bug 1571493 - part5 : use 'MediaControlKeysEventHandler' to deliver control keys event.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/006160f3a043
part1 : implement 'ContentMediaController' which is used to notify 'MediaControlKeysEvent' for controlled media. r=chunmin
https://hg.mozilla.org/integration/autoland/rev/ff27240d2c1d
part2 : use 'MediaControlAgent' in media element. r=chunmin
https://hg.mozilla.org/integration/autoland/rev/e45cf42b2bc2
part3 : notify media state and audible state changes. r=chunmin
https://hg.mozilla.org/integration/autoland/rev/d5c728d8c206
part4 : support 'ControlledMediaState' in media controller r=chunmin
https://hg.mozilla.org/integration/autoland/rev/3ef1b8838276
part5 : use 'MediaControlKeysEventHandler' to deliver control keys event. r=chunmin

Comment 10

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/006160f3a043
https://hg.mozilla.org/mozilla-central/rev/ff27240d2c1d
https://hg.mozilla.org/mozilla-central/rev/e45cf42b2bc2
https://hg.mozilla.org/mozilla-central/rev/d5c728d8c206
https://hg.mozilla.org/mozilla-central/rev/3ef1b8838276