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

Unified Diff: content/test/data/media/peerconnection-call.html

Issue 214663004: Add test for sending square resolution video on a PeerConnection. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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: content/test/data/media/peerconnection-call.html
diff --git a/content/test/data/media/peerconnection-call.html b/content/test/data/media/peerconnection-call.html
index 29a23b8f9151bde9b034f3b21a6c4eb347144d21..0ae84b2c9a2342497491750e0beeafd969c75eaf 100644
--- a/content/test/data/media/peerconnection-call.html
+++ b/content/test/data/media/peerconnection-call.html
@@ -55,12 +55,16 @@
setAllEventsOccuredHandler(reportTestSuccess);
// Test that we can setup call with an audio and video track.
- function call(constraints) {
+ function call(constraints, expected_width, expected_height) {
createConnections(null);
navigator.webkitGetUserMedia(constraints,
addStreamToBothConnectionsAndNegotiate, printGetUserMediaError);
- waitForVideo('remote-view-1');
- waitForVideo('remote-view-2');
+ waitForVideoAndExpectResolution('remote-view-1',
phoglund_chromium 2014/03/28 08:33:30 You can shorten this to waitForVideoWithResolution
perkj_chrome 2014/03/28 09:14:23 Done.
+ expected_width,
+ expected_height);
+ waitForVideoAndExpectResolution('remote-view-2',
+ expected_width,
+ expected_height);
}
// First calls without streams on any connections, and then adds a stream

Powered by Google App Engine
This is Rietveld 408576698