Bug 1441509 - Start performing the analysis in test_seamless_looping.html after the second loop, to let things stabilize. r?karlt draft
authorPaul Adenot <paul@paul.cx>
Fri, 09 Mar 2018 15:24:03 +0100
changeset 773269 f74b85b1a4b5aded39e01f2ad5b4cbd2de1d6efa
parent 773144 b906009d875d1f5d29b0d1252cdb43a9b1a5889c
push id104194
push userpaul@paul.cx
push dateTue, 27 Mar 2018 18:54:44 +0000
reviewerskarlt
bugs1441509
milestone61.0a1
Bug 1441509 - Start performing the analysis in test_seamless_looping.html after the second loop, to let things stabilize. r?karlt MozReview-Commit-ID: 5FK30GNk4s5
dom/media/test/test_seamless_looping.html
--- a/dom/media/test/test_seamless_looping.html
+++ b/dom/media/test/test_seamless_looping.html
@@ -118,18 +118,18 @@
       }
     }
 
     function analysisCallback() {
       if (!again) {
         return;
       }
       analyser.getFloatFrequencyData(frequencyBuf);
-      // There is an unrelated click at start, skip it.
-      if (loopCount != 0) {
+      // Let things stabilize at the beginning. See bug bug 1441509.
+      if (loopCount > 1) {
         doAnalysis(frequencyBuf, ac.sampleRate);
       }
 
       if (DEBUG) {
         c.clearRect(0, 0, w, h);
         analyser.getFloatTimeDomainData(timeDomainBuf);
         for (var i = 0; i < frequencyBuf.length; i++) {
           c.fillRect(i, h, 1, -frequencyBuf[i] + analyser.minDecibels);