Skip to content

Commit

Permalink
Bug 1343459. Part 2 - remove the 'updateend' handler which might fire…
Browse files Browse the repository at this point in the history
… before we register it. r=jya

Note the race is uncovered by P1 which kinda change the order of events.

MozReview-Commit-ID: 3INYvJVUhSG

--HG--
extra : rebase_source : e378c2a437a5a729008d39570be2a9087a7eb5f7
extra : intermediate-source : 02e2ecfea068dd9f487791287064e684a15dd599
extra : source : f2f40c70a2304e3e499422f3a7c46b59b54ad1ae
  • Loading branch information
jwwang committed Mar 1, 2017
1 parent 4f89ce3 commit b28f0b3
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions dom/media/mediasource/test/test_AudioChange_mp4.html
Original file line number Diff line number Diff line change
Expand Up @@ -53,11 +53,8 @@
})
.then(fetchAndLoad.bind(null, audiosb, 'aac51-48000-128000-', ['2'], '.m4s'))
.then(function() {
var promises = [];
ms.endOfStream();
promises.push(once(el, 'ended'));
promises.push(once(audiosb, 'updateend'));
return Promise.all(promises);
return once(el, 'ended');
})
.then(function() {
ok(el.currentTime >= 6, "played to the end");
Expand Down

0 comments on commit b28f0b3

Please sign in to comment.