Bug 1397793 - Don't use the AEC on a 440Hz tone when testing that audio is flowing. r?pehrsons draft
authorPaul Adenot <paul@paul.cx>
Mon, 04 Dec 2017 18:16:22 +0100
changeset 707573 f19c191747f2e63303406a43a79454e4ea1ed428
parent 707572 265d30d8bf83103c50bb3cb84b263bdb87b8494e
child 707586 a940b255ffdcc23c5c46c798b3706c9359c7c2aa
push id92158
push userpaul@paul.cx
push dateTue, 05 Dec 2017 14:38:23 +0000
reviewerspehrsons
bugs1397793
milestone59.0a1
Bug 1397793 - Don't use the AEC on a 440Hz tone when testing that audio is flowing. r?pehrsons This brings in a lot of noise and makes the test fail. MozReview-Commit-ID: 70EGM1q1J24
dom/media/tests/mochitest/test_getUserMedia_mediaElementCapture_audio.html
--- a/dom/media/tests/mochitest/test_getUserMedia_mediaElementCapture_audio.html
+++ b/dom/media/tests/mochitest/test_getUserMedia_mediaElementCapture_audio.html
@@ -11,17 +11,17 @@ createHTML({
   bug: "1259788",
   title: "Test CaptureStream audio content on HTMLMediaElement playing a gUM MediaStream",
   visible: true
 });
 
 var audioContext;
 var gUMAudioElement;
 var analyser;
-runTest(() => getUserMedia({audio: true})
+runTest(() => getUserMedia({audio: { echoCancellation: false }})
   .then(stream => {
     gUMAudioElement = createMediaElement("audio", "gUMAudio");
     gUMAudioElement.srcObject = stream;
 
     audioContext = new AudioContext();
     info("Capturing");
 
     analyser = new AudioStreamAnalyser(audioContext,