Bug 1415028 - Remove unnecessary code which may lead to intermittent error. r=jwwang
authorKilik Kuo <kikuo@mozilla.com>
Tue, 07 Nov 2017 14:35:52 +0800
changeset 443746 ea03ba856101fd7d9c3fcaf642a8e0f7c3fd115c
parent 443745 3b928622368e4bc99502d37c9015565e517d3bd5
child 443747 75a973619e42c6d46d059a7ca36cba931f6d0b74
push id1618
push userCallek@gmail.com
push dateThu, 11 Jan 2018 17:45:48 +0000
treeherdermozilla-release@882ca853e05a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjwwang
bugs1415028
milestone58.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 1415028 - Remove unnecessary code which may lead to intermittent error. r=jwwang The session update triggered from the second LoadEME may come later than the end of playback. MozReview-Commit-ID: K1vOaztbx4v
dom/media/test/test_eme_sample_groups_playback.html
--- a/dom/media/test/test_eme_sample_groups_playback.html
+++ b/dom/media/test/test_eme_sample_groups_playback.html
@@ -108,24 +108,21 @@
           .then(() => {
             var ms = new MediaSource();
             video.src = URL.createObjectURL(ms);
 
             once(ms, "sourceopen", () => {
               Promise.all(test.track.fragments.map(fragment => DownloadMedia(fragment, test.track.type, ms)))
                 .then(() => {
                   ms.endOfStream();
-                  LoadEME();
-                })
-                .then(() => {
                   video.play();
                 });
             });
 
-            video.addEventListener("ended", SimpleTest.finish);
+            once(video, "ended", SimpleTest.finish);
           });
       }
 
       SetupEMEPref(LoadMSE);
 
   </script>
   </pre>
 </body>