Bug 1208371 - Remove debug canvas from addTrackRemoveTrack test to help android perf. r?padenot draft
authorAndreas Pehrson <pehrsons@gmail.com>
Tue, 05 Jan 2016 10:16:31 +0800
changeset 342158 09e81c59240b19aa0fbe913cf60fbbff1dc1eb10
parent 342157 7fd72eb444871040356bc02801fa779e1d8ef775
child 342159 241934caf9ea136daa4926fa5b5b603f697bbeb9
push id13352
push userpehrsons@gmail.com
push dateFri, 18 Mar 2016 13:49:47 +0000
reviewerspadenot
bugs1208371
milestone47.0a1
Bug 1208371 - Remove debug canvas from addTrackRemoveTrack test to help android perf. r?padenot MozReview-Commit-ID: Ejyvk1Vc66k
dom/media/tests/mochitest/test_getUserMedia_addTrackRemoveTrack.html
--- a/dom/media/tests/mochitest/test_getUserMedia_addTrackRemoveTrack.html
+++ b/dom/media/tests/mochitest/test_getUserMedia_addTrackRemoveTrack.html
@@ -5,18 +5,17 @@
 </head>
 <body>
 <pre id="test">
 <script type="application/javascript">
   "use strict";
 
   createHTML({
     title: "MediaStream's addTrack() and removeTrack() with getUserMedia streams Test",
-    bug: "1103188",
-    visible: true
+    bug: "1103188"
   });
 
   runTest(() => Promise.resolve()
     .then(() => getUserMedia({audio: true})).then(stream =>
       getUserMedia({video: true}).then(otherStream => {
         info("Test addTrack()ing a video track to an audio-only gUM stream");
         var track = stream.getTracks()[0];
         var otherTrack = otherStream.getTracks()[0];
@@ -113,58 +112,50 @@
 
       var osc10k = createOscillatorStream(ac, 10000);
       var audioTrack10k = osc10k.getTracks()[0];
 
       var stream = osc1k;
       return Promise.resolve().then(() => {
         info("Analysing audio output with original 1k track");
         var analyser = new AudioStreamAnalyser(ac, stream);
-        analyser.enableDebugCanvas();
         return analyser.waitForAnalysisSuccess(array =>
                  array[analyser.binIndexForFrequency(50)] < 50 &&
                  array[analyser.binIndexForFrequency(1000)] > 200 &&
                  array[analyser.binIndexForFrequency(5000)] < 50 &&
-                 array[analyser.binIndexForFrequency(10000)] < 50)
-          .then(() => analyser.disableDebugCanvas());
+                 array[analyser.binIndexForFrequency(10000)] < 50);
       }).then(() => {
         info("Analysing audio output with removed original 1k track and added 5k track");
         stream.removeTrack(audioTrack1k);
         stream.addTrack(audioTrack5k);
         var analyser = new AudioStreamAnalyser(ac, stream);
-        analyser.enableDebugCanvas();
         return analyser.waitForAnalysisSuccess(array =>
                  array[analyser.binIndexForFrequency(1000)] < 50 &&
                  array[analyser.binIndexForFrequency(5000)] > 200 &&
-                 array[analyser.binIndexForFrequency(10000)] < 50)
-          .then(() => analyser.disableDebugCanvas());
+                 array[analyser.binIndexForFrequency(10000)] < 50);
       }).then(() => {
         info("Analysing audio output with removed 5k track and added 10k track");
         stream.removeTrack(audioTrack5k);
         stream.addTrack(audioTrack10k);
         var analyser = new AudioStreamAnalyser(ac, stream);
-        analyser.enableDebugCanvas();
         return analyser.waitForAnalysisSuccess(array =>
                  array[analyser.binIndexForFrequency(1000)] < 50 &&
                  array[analyser.binIndexForFrequency(5000)] < 50 &&
-                 array[analyser.binIndexForFrequency(10000)] > 200)
-          .then(() => analyser.disableDebugCanvas());
+                 array[analyser.binIndexForFrequency(10000)] > 200);
       }).then(() => {
         info("Analysing audio output with re-added 1k, 5k and added 10k tracks");
         stream.addTrack(audioTrack1k);
         stream.addTrack(audioTrack5k);
         var analyser = new AudioStreamAnalyser(ac, stream);
-        analyser.enableDebugCanvas();
         return analyser.waitForAnalysisSuccess(array =>
                  array[analyser.binIndexForFrequency(50)] < 50 &&
                  array[analyser.binIndexForFrequency(1000)] > 200 &&
                  array[analyser.binIndexForFrequency(2500)] < 50 &&
                  array[analyser.binIndexForFrequency(5000)] > 200 &&
                  array[analyser.binIndexForFrequency(7500)] < 50 &&
                  array[analyser.binIndexForFrequency(10000)] > 200 &&
-                 array[analyser.binIndexForFrequency(11000)] < 50)
-          .then(() => analyser.disableDebugCanvas());
+                 array[analyser.binIndexForFrequency(11000)] < 50);
       });
     }));
 </script>
 </pre>
 </body>
 </html>