Closed Bug 1616203 Opened 5 months ago Closed 5 months ago

[wpt-sync] Sync PR 21850 - Fix getStats test for RTPReceiver

Categories

(Core :: WebRTC, task, P4)

task

Tracking

()

RESOLVED FIXED
mozilla75
Tracking Status
firefox75 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 21850 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/21850
Details from upstream follow.

Harald Alvestrand <hta@chromium.org> wrote:

Fix getStats test for RTPReceiver

The test for getStats() on an RTP receiver was testing with the wrong
track (sent track not received track).
The change gives a reasonable error message (not all stats are present).

Bug: none
Change-Id: I7f35dc90a2e4f6ee4ed67e8d6ae2ecdf9246c2a3
Reviewed-on: https://chromium-review.googlesource.com/2060952
WPT-Export-Revision: d06e2a97f87ccc49b8e27f8b7737f25e49879635

Component: web-platform-tests → WebRTC
Product: Testing → Core

CI Results

Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests and 13 subtests

Status Summary

Firefox

OK : 1
PASS: 7
FAIL: 6

Chrome

OK : 1
PASS: 8
FAIL: 5

Safari

OK : 1
PASS: 5
FAIL: 8

Links

GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

/webrtc/RTCPeerConnection-getStats.https.html
getStats() with no argument should return stats report containing peer-connection stats on an empty PC: FAIL

New Tests That Don't Pass

/webrtc/RTCPeerConnection-getStats.https.html
getStats() with connected peer connections having tracks and data channel should return all mandatory to implement stats: FAIL (Chrome: FAIL, Safari: FAIL)
getStats() with no argument should return stats report containing peer-connection stats and outbound-track-stats: FAIL (Chrome: FAIL, Safari: FAIL)
getStats() on track associated with RtpReceiver should return stats report containing inbound-rtp stats: FAIL (Chrome: FAIL, Safari: FAIL)
getStats() with no argument should return stats for no-stream tracks: FAIL (Chrome: FAIL, Safari: FAIL)
getStats() on track associated with RtpSender should return stats report containing outbound-rtp stats: FAIL (Chrome: FAIL, Safari: FAIL)
getStats() with no argument should return stats report containing peer-connection stats on an empty PC: FAIL (Chrome: PASS, Safari: PASS)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5339583bf12d
[wpt PR 21850] - Fix getStats test for RTPReceiver, a=testonly
Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla75
You need to log in before you can comment on or make changes to this bug.