implement getSynchronizationSources for Video

RESOLVED FIXED in Firefox 68
(NeedInfo from)

Status

()

enhancement
P4
normal
Rank:
39
RESOLVED FIXED
5 months ago
2 months ago

People

(Reporter: ng, Assigned: ng, NeedInfo)

Tracking

({dev-doc-complete})

unspecified
mozilla68
Points:
---

Firefox Tracking Flags

(firefox68 fixed)

Details

Attachments

(1 attachment)

getSynchronizationSources is not implemented for video, only audio where it's primary purpose is to get the audio levels of packets. This probably isn't too hard, however there hasn't been anyone asking for it yet either.

Rank: 39
No longer blocks: 1525394
Duplicate of this bug: 1525394
Assignee: nobody → na-g

I also filed: https://github.com/web-platform-tests/wpt/issues/15828, which is a bug against the test suite requiring the optional field 'voiceActivityFlag'.

Bug 1534466 - P1 refactor RtpSourceObserver to accept RTPHeader

Bug 1534466 - P2 Add RtpSourceObserver to VideoConduit

Bug 1534466 - P3 Update WPT ini to reflect newly passing tests

Reminder to self, land Monday 3/18.

Flags: needinfo?(na-g)

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:ng, could you have a look please?

Flags: needinfo?(na-g)
Pushed by na-g@nostrum.com:
https://hg.mozilla.org/integration/autoland/rev/4f641f92edb2
implement getContributingSources and getSynchronizationSources for video r=jib
Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68

We've got these documented, but we should probably update the compat data.

Keywords: dev-doc-needed
Flags: needinfo?(na-g)
Flags: needinfo?(na-g)

Did we previously have these in IDL and implemented to do nothing? I see no WebIDL changes here, which you normally do when API support is added for methods... according to the docs we already have in place, we have had these since Firefox 59. Was that audio only?

Flags: needinfo?(na-g)

I've performed the following updates:

  • Submitted BCD PR 4340 to add data about audio and video track support to getContributingSources() and getSynchronizationSources()
  • Added a bullet point to the WebRTC section on Firefox 68 for developers, under APIs
You need to log in before you can comment on or make changes to this bug.