Closed
Bug 1518754
Opened 7 years ago
Closed 6 years ago
[wpt-sync] Sync PR 14757 - RTCRtpReceiver.getSynchronizationSources() added.
Categories
(Core :: WebRTC, enhancement, P4)
Core
WebRTC
Tracking
()
RESOLVED
FIXED
mozilla67
| Tracking | Status | |
|---|---|---|
| firefox67 | --- | fixed |
People
(Reporter: wpt-sync, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 14757 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/14757
Details from upstream follow.
Henrik Boström <hbos@chromium.org> wrote:
RTCRtpReceiver.getSynchronizationSources() added.
This CL:
- Adds getSynchronizationSources() for both audio and video.
- Adds proper test coverage, including test coverage for unsupported
members audioLevel and voiceActivityFlag. It turns out that our
timestamps are not comparable to performance.now(), this should be
fixed separately (this must already have been the case for
getContributingSources() but not noticed until now).- Removes the WPTs for getContributingSources() since these tests were
not working (an SFU or the like or fakes/mocks are needed to test
this, this is not something that is testable in WPTs).Bug: 893158, 893160
Change-Id: Ic90344af7fa48c3cd0b94929cc9453deb9dc7f9f
Reviewed-on: https://chromium-review.googlesource.com/1401068
WPT-Export-Revision: d99630f15c38d2321de91e2c3789eb0167cf4117
| Assignee | ||
Updated•7 years ago
|
Component: web-platform-tests → WebRTC
Product: Testing → Core
| Assignee | ||
Comment 1•7 years ago
|
||
| Assignee | ||
Comment 2•7 years ago
|
||
| Assignee | ||
Comment 3•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=9784406e203249e6a273338a8cd19c14bc2dd751
| Assignee | ||
Comment 4•6 years ago
|
||
Ran 1 tests and 12 subtests
PASS : 5
TIMEOUT: 2
NOTRUN : 6
Existing tests that now have a worse result (e.g. they used to PASS and now FAIL):
/webrtc/RTCRtpReceiver-getSynchronizationSources.https.html: TIMEOUT
New tests that have failures or other problems:
/webrtc/RTCRtpReceiver-getSynchronizationSources.https.html
[audio-only] RTCRtpSynchronizationSource.audioLevel is a number [0, 1]: NOTRUN
[audio-only] RTCRtpSynchronizationSource.voiceActivityFlag is a boolean: NOTRUN
[video] RTCRtpSynchronizationSource.source is a number: NOTRUN
[video] RTCRtpSynchronizationSource.timestamp is a number: NOTRUN
[video] RTCRtpSynchronizationSource.timestamp is comparable to performance.timeOrigin + performance.now(): NOTRUN
[video] getSynchronizationSources() does not contain SSRCs older than 10 seconds: NOTRUN
[video] getSynchronizationSources() eventually returns a non-empty list: TIMEOUT
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9a353865db5a
[wpt PR 14757] - RTCRtpReceiver.getSynchronizationSources() added., a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/3e1fe68d4fa7
[wpt PR 14757] - Update wpt metadata, a=testonly
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2e1b2bb14876
[wpt PR 14757] - RTCRtpReceiver.getSynchronizationSources() added., a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/40bb7ca11e2b
[wpt PR 14757] - Update wpt metadata, a=testonly
Comment 7•6 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/2e1b2bb14876
https://hg.mozilla.org/mozilla-central/rev/40bb7ca11e2b
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in
before you can comment on or make changes to this bug.
Description
•