Bug 1525209 - Revert spec-incompliant pause event from bug 1461877. r=cpearce a=pascalc
authorAndreas Pehrson <apehrson@mozilla.com>
Wed, 17 Apr 2019 09:57:59 +0000
changeset 523246 1694fdeb8a6ad706eb77001076bb6f0ba95a577a
parent 523245 a725a9315f5d84022d54bc2bcdc88190eb05abb0
child 523247 f350dc70a86486f45ee7e2414332d9c058445617
push id11115
push useraiakab@mozilla.com
push dateFri, 19 Apr 2019 16:14:22 +0000
treeherdermozilla-beta@86f6ec90b34b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerscpearce, pascalc
bugs1525209, 1461877
milestone67.0
Bug 1525209 - Revert spec-incompliant pause event from bug 1461877. r=cpearce a=pascalc Differential Revision: https://phabricator.services.mozilla.com/D27864
dom/html/HTMLMediaElement.cpp
--- a/dom/html/HTMLMediaElement.cpp
+++ b/dom/html/HTMLMediaElement.cpp
@@ -1843,17 +1843,16 @@ void HTMLMediaElement::AbortExistingLoad
 
   if (mNetworkState != NETWORK_EMPTY) {
     NS_ASSERTION(!mDecoder && !mSrcStream,
                  "How did someone setup a new stream/decoder already?");
     // ChangeNetworkState() will call UpdateAudioChannelPlayingState()
     // indirectly which depends on mPaused. So we need to update mPaused first.
     if (!mPaused) {
       mPaused = true;
-      DispatchAsyncEvent(NS_LITERAL_STRING("pause"));
       RejectPromises(TakePendingPlayPromises(), NS_ERROR_DOM_MEDIA_ABORT_ERR);
     }
     ChangeNetworkState(NETWORK_EMPTY);
     ChangeReadyState(HAVE_NOTHING);
 
     // TODO: Apply the rules for text track cue rendering Bug 865407
     if (mTextTrackManager) {
       mTextTrackManager->GetTextTracks()->SetCuesInactive();