diff options
author | Henrik Boström <hbos@webrtc.org> | 2020-01-28 11:15:35 +0100 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-01-28 11:22:09 +0000 |
commit | 4a5dab00ae22c045828b4f23c70de8fda814ca29 (patch) | |
tree | 7dbd4e9266befb408708d0ee1003683b7c8d5a6b /pc | |
parent | 086055d0fd9b9b9efe8bcf85884324a019e9bd33 (diff) | |
download | webrtc-4a5dab00ae22c045828b4f23c70de8fda814ca29.tar.gz |
[Stats] Include fecPackets[Reeceived/Discarded] in Members()
This refers to modern getStats() only. The metrics has been implemented
for a while in C++ but was accidentally not included in the Members()
list, meaning they were not exposed in lists (including exposure in
Chrome/JavaScript).
The Chromium whitelist already include them.
TBR=hta@webrtc.org
Bug: webrtc:11317
Change-Id: I0c3ee9c552975fc37db2d87196c66e662c994aed
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/167530
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Commit-Queue: Henrik Boström <hbos@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#30391}
Diffstat (limited to 'pc')
-rw-r--r-- | pc/rtc_stats_integrationtest.cc | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/pc/rtc_stats_integrationtest.cc b/pc/rtc_stats_integrationtest.cc index 381d5bd8bf..e3f522bd47 100644 --- a/pc/rtc_stats_integrationtest.cc +++ b/pc/rtc_stats_integrationtest.cc @@ -795,6 +795,9 @@ class RTCStatsReportVerifier { inbound_stream.fec_packets_received); verifier.TestMemberIsNonNegative<uint64_t>( inbound_stream.fec_packets_discarded); + } else { + verifier.TestMemberIsUndefined(inbound_stream.fec_packets_received); + verifier.TestMemberIsUndefined(inbound_stream.fec_packets_discarded); } verifier.TestMemberIsNonNegative<uint64_t>(inbound_stream.bytes_received); verifier.TestMemberIsNonNegative<uint64_t>( |