Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5129)

Unified Diff: chrome/test/data/webrtc/peerconnection_getstats.js

Issue 2590293004: RTCPeerConnection.getStats: Whitelist updated after spec changed. (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/webrtc/peerconnection_getstats.js
diff --git a/chrome/test/data/webrtc/peerconnection_getstats.js b/chrome/test/data/webrtc/peerconnection_getstats.js
index 53677f49e1577e881c2037d0ac8eaf5900d61790..300ae02b481174806daf68c18b276f5f91666b09 100644
--- a/chrome/test/data/webrtc/peerconnection_getstats.js
+++ b/chrome/test/data/webrtc/peerconnection_getstats.js
@@ -28,6 +28,7 @@ var kRTCRTPStreamStats = new RTCStats_(null, {
pliCount: 'number',
nackCount: 'number',
sliCount: 'number',
+ qpSum: 'number',
});
/*
@@ -66,6 +67,7 @@ var kRTCInboundRTPStreamStats = new RTCStats_(kRTCRTPStreamStats, {
burstDiscardRate: 'number',
gapLossRate: 'number',
gapDiscardRate: 'number',
+ framesDecoded: 'number',
});
gStatsWhitelist.set('inbound-rtp', kRTCInboundRTPStreamStats);
@@ -79,6 +81,7 @@ var kRTCOutboundRTPStreamStats = new RTCStats_(kRTCRTPStreamStats, {
bytesSent: 'number',
targetBitrate: 'number',
roundTripTime: 'number',
+ framesEncoded: 'number',
});
gStatsWhitelist.set('outbound-rtp', kRTCOutboundRTPStreamStats);
@@ -170,12 +173,15 @@ gStatsWhitelist.set('transport', kRTCTransportStats);
* @private
*/
var kRTCIceCandidateStats = new RTCStats_(null, {
+ transportId: 'string',
+ isRemote: 'boolean',
ip: 'string',
port: 'number',
protocol: 'string',
candidateType: 'string',
priority: 'number',
url: 'string',
+ deleted: 'boolean',
});
gStatsWhitelist.set('local-candidate', kRTCIceCandidateStats);
gStatsWhitelist.set('remote-candidate', kRTCIceCandidateStats);
@@ -198,11 +204,6 @@ var kRTCIceCandidatePairStats = new RTCStats_(null, {
bytesReceived: 'number',
totalRoundTripTime: 'number',
currentRoundTripTime: 'number',
- // TODO(hbos): *Rtt have been renamed to *RoundTripTime, both listed at the
- // moment. As soon as https://codereview.webrtc.org/2576383002/ has rolled in,
- // remove the *Rtt ones. crbug.com/627816
- totalRtt: 'number',
- currentRtt: 'number',
availableOutgoingBitrate: 'number',
availableIncomingBitrate: 'number',
requestsReceived: 'number',
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698