Closed Bug 1656398 Opened 4 months ago Closed 4 months ago

Provide more methods/attribute/event handler on media controller webidl interface

Categories

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

task

Tracking

()

RESOLVED FIXED
81 Branch
Tracking Status
firefox81 --- fixed

People

(Reporter: alwu, Assigned: alwu)

References

Details

Attachments

(8 files, 3 obsolete files)

47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review

If we want to use this interface to construct media hub, we would definitely need more abilities on this interface.

In addition, we might also use this interface to implement media control on GeckoView, which would more fit their requirement. But we would implement that in C++ implementation first because that feature(media control on GV) is targeting on Fx81, which we might be not able to finish this interface to 100% usable on Fx81.

I've made some patches last week but didn't have time to finish them. Upload those WIP and will continue to work on them later.

Depends on: 1658075
Attachment #9168659 - Attachment is obsolete: true
Attachment #9168660 - Attachment is obsolete: true
Attachment #9168661 - Attachment is obsolete: true
Blocks: 1658937
Pushed by alwu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b81b6bf1b140
part1 : porting media controller's controlling methods r=chunmin
https://hg.mozilla.org/integration/autoland/rev/8f48374d8c41
part2 : add `getMetadata()` on media controller webidl method. r=chunmin
https://hg.mozilla.org/integration/autoland/rev/2c22b474652d
part3 : using 'getMetadata()' in tests. r=chunmin
https://hg.mozilla.org/integration/autoland/rev/48a86f2dd6bc
part4 : expose `playbackState` on MediaController webidl interface. r=chunmin
https://hg.mozilla.org/integration/autoland/rev/d0386487d649
part5 : add event handler `onplaybackstatechange`. r=chunmin
https://hg.mozilla.org/integration/autoland/rev/8ae6a224e19f
part6 : use 'controller.playbackState' in test. r=chunmin
https://hg.mozilla.org/integration/autoland/rev/f4c506b647fe
part7 : add event handler 'onmetadatachange'. r=chunmin
https://hg.mozilla.org/integration/autoland/rev/d50f03f8c92f
part8 : use 'onmetadatachange' in test. r=chunmin
You need to log in before you can comment on or make changes to this bug.