Bug 1289742. P2 - remove the e.canPlayType() test which is already done by MediaTestManager. r=kaku
authorJW Wang <jwwang@mozilla.com>
Mon, 06 Mar 2017 14:59:30 +0800
changeset 346065 f5c0df74ee7357ac251a9403f3da7e0465915d45
parent 346064 9a65dfeb3d7038bceccbaed89197a308faf8f2c2
child 346066 34ad60160536daa711cd574721a03538cd699d76
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. P2 - remove the e.canPlayType() test which is already done by MediaTestManager. r=kaku MozReview-Commit-ID: 7Pe8CyHgmzB
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
@@ -74,27 +74,25 @@ function tryClone(event) {
 // 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);
   e.preload = "auto";
-  if (e.canPlayType(test.type)) {
-    e.src = test.name;
-    if (test.duration) {
-      e._expectedDuration = test.duration;
-    }
-    ok(true, "Trying to load " + test.name);
-    e.addEventListener("loadeddata", tryClone);
-    e.load();
-    e.token = token;
-    manager.started(token);
+  e.src = test.name;
+  if (test.duration) {
+    e._expectedDuration = test.duration;
   }
+  ok(true, "Trying to load " + test.name);
+  e.addEventListener("loadeddata", tryClone);
+  e.load();
+  e.token = token;
+  manager.started(token);
 }
 
 manager.runTests(gCloneTests, initTest);
 
 </script>
 </pre>
 </body>
 </html>