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

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

Issue 2621733004: RTCPeerConnection.getStats whitelist updated due to recent spec changes. (Closed)
Patch Set: RTCPeerConnectionStats updated after spec changed Created 3 years, 11 months 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 300ae02b481174806daf68c18b276f5f91666b09..6054a5cea7c19057127c4ec5898391e570d5ff35 100644
--- a/chrome/test/data/webrtc/peerconnection_getstats.js
+++ b/chrome/test/data/webrtc/peerconnection_getstats.js
@@ -21,6 +21,9 @@ var kRTCRTPStreamStats = new RTCStats_(null, {
associateStatsId: 'string',
isRemote: 'boolean',
mediaType: 'string',
+ trackId: 'string',
+ // TODO(hbos): As soon as |mediaTrackId| has been renamed to |trackId|, remove
+ // this line. crbug.com/657854
mediaTrackId: 'string',
transportId: 'string',
codecId: 'string',
@@ -93,6 +96,8 @@ gStatsWhitelist.set('outbound-rtp', kRTCOutboundRTPStreamStats);
var kRTCPeerConnectionStats = new RTCStats_(null, {
dataChannelsOpened: 'number',
dataChannelsClosed: 'number',
+ dataChannelsRequested: 'number',
+ dataChannelsAccepted: 'number',
});
gStatsWhitelist.set('peer-connection', kRTCPeerConnectionStats);
@@ -117,6 +122,9 @@ var kRTCMediaStreamTrackStats = new RTCStats_(null, {
remoteSource: 'boolean',
ended: 'boolean',
detached: 'boolean',
+ kind: 'string',
+ // TODO(hbos): As soon as |ssrcIds| has been removed, remove this line.
+ // crbug.com/659137
ssrcIds: 'sequence_string',
frameWidth: 'number',
frameHeight: 'number',
@@ -160,6 +168,9 @@ var kRTCTransportStats = new RTCStats_(null, {
bytesSent: 'number',
bytesReceived: 'number',
rtcpTransportStatsId: 'string',
+ dtlsState: 'string',
+ // TODO(hbos): As soon as |activeConnection| has been replaced by |dtlsState|,
+ // remove this line. crbug.com/653873
activeConnection: 'boolean',
selectedCandidatePairId: 'string',
localCertificateId: 'string',
« 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