Bug 1305136 - Add additional status messages to loadFile function; r=padenot
authorDan Minor <dminor@mozilla.com>
Wed, 22 Feb 2017 10:59:47 -0500
changeset 373617 7d15bc99dbf1f1c06e38379ca6bcd4289c9ea021
parent 373616 11a711ac2407fb856937c870f8d2a08311be3221
child 373618 d439fa74bf059fec9e32547d80b726a711d6f4ee
push id10863
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 23:02:23 +0000
treeherdermozilla-aurora@0931190cd725 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerspadenot
bugs1305136
milestone54.0a1
Bug 1305136 - Add additional status messages to loadFile function; r=padenot It seems like adding these extra "todos" decreases the frequency of this intermittent. I think it makes sense to land these temporarily while we look for a root cause. MozReview-Commit-ID: 4RiXtT3yBzG
dom/media/webaudio/test/test_audioContextSuspendResumeClose.html
--- a/dom/media/webaudio/test/test_audioContextSuspendResumeClose.html
+++ b/dom/media/webaudio/test/test_audioContextSuspendResumeClose.html
@@ -43,20 +43,22 @@ function tryToCreateNodeOnClosedContext(
 
       expectException(function() {
         ctx[e.name].apply(ctx, e.args);
       }, DOMException.INVALID_STATE_ERR);
     });
 }
 
 function loadFile(url, callback) {
+  todo(false, "loadFile: " + url);
   var xhr = new XMLHttpRequest();
   xhr.open("GET", url, true);
   xhr.responseType = "arraybuffer";
   xhr.onload = function() {
+    todo(false, "loadFile: " + url + " calling callback...");
     callback(xhr.response);
   };
   xhr.send();
 }
 
 // createBuffer, createPeriodicWave and decodeAudioData should work on a context
 // that has `state` == "closed"
 function tryLegalOpeerationsOnClosedContext(ctx) {