Bug 1295923 - remove the workaround since bug 1329403 is fixed and 'onload' won't be fired prematurely before the media element finishes loading. r=gerald
authorJW Wang <jwwang@mozilla.com>
Fri, 20 Jan 2017 11:52:52 +0800
changeset 375598 26db8aff32c8b9c2eda34b791b4f0e24f764db89
parent 375597 a00c204385df6dfc6806893a951f3452a359d18f
child 375599 6a8c89e67f9a4395be69ead4bfc0eb053aa75201
push id6996
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 20:48:21 +0000
treeherdermozilla-beta@d89512dab048 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersgerald
bugs1295923, 1329403
milestone53.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 1295923 - remove the workaround since bug 1329403 is fixed and 'onload' won't be fired prematurely before the media element finishes loading. r=gerald MozReview-Commit-ID: beXOjulT1E
dom/media/test/test_error_in_video_document.html
--- a/dom/media/test/test_error_in_video_document.html
+++ b/dom/media/test/test_error_in_video_document.html
@@ -44,25 +44,16 @@ function check() {
 var t = getPlayableVideo(gErrorTests);
 if (!t) {
   todo(false, "No types supported");
 } else {
   SimpleTest.waitForExplicitFinish();
 
   var f = document.createElement("iframe");
   f.src = t.name;
-  f.addEventListener("load", function() {
-    if (documentVideo().error) {
-      info("Error occured by the time we got |load| - checking directly.");
-      check();
-    } else {
-      //TODO: Fix this todo in Bug 1295923.
-      todo(false, "Error hasn't occurred yet - adding |error| event listener. This shouldn't happen, see bug 608634.");
-      documentVideo().addEventListener("error", check);
-    }
-  });
+  f.addEventListener("load", check);
   document.body.appendChild(f);
 }
 
 </script>
 </pre>
 </body>
 </html>