Bug 1289742. P8 - always create a video element which is capable of playing both audio and video. r=kaku
authorJW Wang <jwwang@mozilla.com>
Mon, 06 Mar 2017 17:14:29 +0800
changeset 346071 b73a2e4a93d738c20291a04cf8b5d0d35801c468
parent 346070 ca56682eb7c1388fb3cef8c4692b0f1cac2b909d
child 346072 deafdcdf32a52d0dfb81af6c34110fd894764d8f
push id31459
push usercbook@mozilla.com
push dateTue, 07 Mar 2017 14:05:14 +0000
treeherdermozilla-central@1fb56ba248d5 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerskaku
bugs1289742
milestone54.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1289742. P8 - always create a video element which is capable of playing both audio and video. r=kaku MozReview-Commit-ID: 6fSivL8rIsx
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
@@ -66,18 +66,17 @@ 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) {
-  var elemType = /^audio/.test(test.type) ? "audio" : "video";
-  var e = document.createElement(elemType);
+  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);
 }