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

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

Issue 2641263003: Performance measures of old and new RTCPeerConnection.getStats added. (Closed)
Patch Set: Fix windows compile error (change name of enum keys) 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
Index: chrome/test/data/webrtc/peerconnection.js
diff --git a/chrome/test/data/webrtc/peerconnection.js b/chrome/test/data/webrtc/peerconnection.js
index 8d0f976963353ddf0b254d52c905ad6451325664..d2b5c298a7eca4041961aeb7aa944bb2ed5d1ed0 100644
--- a/chrome/test/data/webrtc/peerconnection.js
+++ b/chrome/test/data/webrtc/peerconnection.js
@@ -376,7 +376,8 @@ function hasSeenCryptoInSdp() {
}
/**
- * Verifies that |RTCPeerConnection.getStats| returns stats.
+ * Verifies that the legacy |RTCPeerConnection.getStats| returns stats and
+ * verifies that each stats member is a string.
*
* Returns ok-got-stats on success.
*/
@@ -400,6 +401,20 @@ function verifyStatsGenerated() {
});
}
+/**
+ * Invokes the legacy (callback-based) |RTCPeerConnection.getStats| and returns
+ * to test when done, not doing anything with the results. For testing
+ * performance.
+ *
+ * Returns done on success.
+ */
+function getStatsCallbackAndReturn() {
+ peerConnection_().getStats(
+ function(response) {
+ returnToTest('done');
+ });
+}
+
// Internals.
/** @private */

Powered by Google App Engine
This is Rietveld 408576698