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

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

Issue 252703003: Wrote a test which exercises audio-only WebRTC calls. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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 | « content/browser/media/webrtc_browsertest.cc ('k') | content/test/data/media/webrtc_test_audio.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 caf61741faaba861a2662984d83663fb8b085c07..e2b5a32cd171ee7bcf9fe1bb66ebcb1068b26532 100644
--- a/content/test/data/media/peerconnection-call.html
+++ b/content/test/data/media/peerconnection-call.html
@@ -301,9 +301,9 @@
remoteAudioTrack.enabled = enabled;
}
- function callAndEnsureAudioIsPlaying(beLenient) {
+ function callAndEnsureAudioIsPlaying(beLenient, constraints) {
createConnections(null);
- navigator.webkitGetUserMedia({audio: true, video: true},
+ navigator.webkitGetUserMedia(constraints,
addStreamToBothConnectionsAndNegotiate, printGetUserMediaError);
// Wait until we have gathered samples and can conclude if audio is playing.
@@ -314,16 +314,13 @@
verifyAudioIsPlaying(samples, beLenient);
eventOccured();
});
-
- // (Also, ensure video muting doesn't affect audio).
- enableRemoteVideo(gSecondConnection, false);
};
- detectVideoPlaying('remote-view-2', onCallEstablished);
+ waitForConnectionToStabilize(gFirstConnection, onCallEstablished);
}
function callAndEnsureAudioTrackMutingWorks(beLenient) {
- callAndEnsureAudioIsPlaying(beLenient);
+ callAndEnsureAudioIsPlaying(beLenient, {audio: true, video: true});
setAllEventsOccuredHandler(function() {
// Call is up, now mute the track and check everything goes silent (give
// it a small delay though, we don't expect it to happen instantly).
@@ -339,10 +336,12 @@
}
function callAndEnsureAudioTrackUnmutingWorks(beLenient) {
- callAndEnsureAudioIsPlaying(beLenient);
+ callAndEnsureAudioIsPlaying(beLenient, {audio: true, video: true});
setAllEventsOccuredHandler(function() {
// Mute, wait a while, unmute, verify audio gets back up.
+ // (Also, ensure video muting doesn't affect audio).
enableRemoteAudio(gSecondConnection, false);
+ enableRemoteVideo(gSecondConnection, false);
setTimeout(function() {
enableRemoteAudio(gSecondConnection, true);
« no previous file with comments | « content/browser/media/webrtc_browsertest.cc ('k') | content/test/data/media/webrtc_test_audio.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698