Bug 1348634 - wait for the 'suspend' event to ensure resource cloning work as expected. draft
authorJW Wang <jwwang@mozilla.com>
Mon, 04 Sep 2017 13:20:05 +0800
changeset 658430 fa2f2f5ca6041ec614f93ff58424567c2496347a
parent 658429 bdcfea5fc786f3c5c3c865856a024f6c220602a4
child 729654 b84058c9dba84d29387a300e691af31ced70853e
push id77770
push userjwwang@mozilla.com
push dateMon, 04 Sep 2017 05:58:59 +0000
bugs1348634
milestone57.0a1
Bug 1348634 - wait for the 'suspend' event to ensure resource cloning work as expected. MozReview-Commit-ID: BeGGnUfFDcN
dom/media/test/test_load_same_resource.html
--- a/dom/media/test/test_load_same_resource.html
+++ b/dom/media/test/test_load_same_resource.html
@@ -22,18 +22,17 @@ function checkDuration(actual, expected,
 function cloneLoaded(event) {
   var e = event.target;
   ok(true, `${e.token} loaded OK`);
   checkDuration(e.duration, e._expectedDuration, e.token);
   removeNodeAndSource(e);
   manager.finished(e.token);
 }
 
-function tryClone(event) {
-  var e = event.target;
+function tryClone(e) {
   var clone = e.cloneNode(false);
   clone.token = `${e.token}(cloned)`;
   manager.started(clone.token);
   manager.finished(e.token);
 
   // Log events for debugging.
   var events = ["suspend", "play", "canplay", "canplaythrough", "loadstart", "loadedmetadata",
                 "loadeddata", "playing", "ended", "error", "stalled", "emptied", "abort",
@@ -65,25 +64,36 @@ function tryClone(event) {
 
 // This test checks that loading the same URI twice in different elements at the same time
 // uses the same resource without doing another network fetch. One of the gCloneTests
 // uses dynamic_resource.sjs to return one resource on the first fetch and a different resource
 // on the second fetch. These resources have different lengths, so if the cloned element
 // does a network fetch it will get a resource with the wrong length and we get a test
 // failure.
 
-function initTest(test, token) {
+async function initTest(test, token) {
   var e = document.createElement("video");
   e.preload = "auto";
   e.src = test.name;
   e._expectedDuration = test.duration;
   ok(true, `Trying to load ${test.name}, duration=${test.duration}`);
-  e.addEventListener("loadeddata", tryClone, {once: true});
   e.token = token;
   manager.started(token);
+
+  // Since 320x240.ogv is less than 32KB, we need to wait for the
+  // 'suspend' event to ensure the partial block is flushed to the cache
+  // otherwise the cloned resource will create a new channel when it
+  // has no data to read at position 0. The new channel will download
+  // a different file than the original resource and fail the duration
+  // test.
+  let p1 = once(e, "loadeddata");
+  let p2 = once(e, "suspend");
+  await p1;
+  await p2;
+  tryClone(e);
 }
 
 SimpleTest.waitForExplicitFinish();
 SpecialPowers.pushPrefEnv(
   {"set": [
     ["logging.MediaDecoder", "Debug"],
     ["logging.nsMediaElement", "Debug"],
     ["logging.MediaCache", "Debug"],