Bug 1208371 - Remove debug canvas from peerConnection_trackDisabling test to help android perf. r?padenot
MozReview-Commit-ID: 7EWJ5GaO6fK
--- a/dom/media/tests/mochitest/test_peerConnection_trackDisabling.html
+++ b/dom/media/tests/mochitest/test_peerConnection_trackDisabling.html
@@ -4,18 +4,17 @@
<script type="application/javascript" src="pc.js"></script>
<script type="application/javascript" src="/tests/dom/canvas/test/captureStream_common.js"></script>
</head>
<body>
<pre id="test">
<script type="application/javascript;version=1.8">
createHTML({
bug: "1219711",
- title: "Disabling locally should be reflected remotely",
- visible: true
+ title: "Disabling locally should be reflected remotely"
});
runNetworkTest(() => {
var test = new PeerConnectionTest();
// Always use fake tracks since we depend on video to be somewhat green and
// audio to have a large 1000Hz component.
test.setMediaConstraints([{audio: true, video: true, fake: true}], []);
@@ -62,21 +61,17 @@ runNetworkTest(() => {
.then(() => info("Checking remote video disabled"))
.then(() => checkVideoDisabled(remoteVideo))
},
function CHECK_AUDIO() {
var ac = new AudioContext();
var localAnalyser = new AudioStreamAnalyser(ac, test.pcLocal.streams[0]);
var remoteAnalyser = new AudioStreamAnalyser(ac, test.pcRemote.streams[0]);
- var checkAudio = (analyser, fun) => {
- analyser.enableDebugCanvas();
- return analyser.waitForAnalysisSuccess(fun)
- .then(() => analyser.disableDebugCanvas());
- };
+ var checkAudio = (analyser, fun) => analyser.waitForAnalysisSuccess(fun);
var freq1k = localAnalyser.binIndexForFrequency(1000);
var checkAudioEnabled = analyser =>
checkAudio(analyser, array => array[freq1k] > 200);
var checkAudioDisabled = analyser =>
checkAudio(analyser, array => array[freq1k] < 50);
return Promise.resolve()