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

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

Issue 216523003: Tighten up webrtc audio tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Further tweaks to combat flakes on Android. 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
« no previous file with comments | « no previous file | 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 2327647f0225373f6d94f9279de2ce31403c1f28..57658fa6b603b7ce796f1cfacbd50004415f6e05 100644
--- a/content/test/data/media/peerconnection-call.html
+++ b/content/test/data/media/peerconnection-call.html
@@ -63,7 +63,7 @@
waitForVideo('remote-view-2');
}
- // Test that we can setup call with an audio and video track and check that
+ // Test that we can setup call with an audio and video track and check that
// the video resolution is as expected.
function callAndExpectResolution(constraints,
expected_width,
@@ -308,8 +308,8 @@
// Wait until we have gathered samples and can conclude if audio is playing.
addExpectedEvent();
var onCallEstablished = function() {
- gatherAudioLevelSamples(gSecondConnection, 300, 100,
- function(samples) {
+ // Gather 50 samples per second for 2 seconds.
+ gatherAudioLevelSamples(gSecondConnection, 100, 50, function(samples) {
verifyAudioIsPlaying(samples);
eventOccured();
});
@@ -329,7 +329,7 @@
enableRemoteAudio(gSecondConnection, false);
setTimeout(function() {
- gatherAudioLevelSamples(gSecondConnection, 200, 100, function(samples) {
+ gatherAudioLevelSamples(gSecondConnection, 100, 50, function(samples) {
verifyIsSilent(samples);
reportTestSuccess();
});
@@ -348,11 +348,11 @@
}, 500);
setTimeout(function() {
- gatherAudioLevelSamples(gSecondConnection, 200, 100, function(samples) {
+ gatherAudioLevelSamples(gSecondConnection, 100, 50, function(samples) {
verifyAudioIsPlaying(samples);
reportTestSuccess();
});
- }, 1000);
+ }, 1500);
});
}
« no previous file with comments | « no previous file | content/test/data/media/webrtc_test_audio.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698