Bug 1525209 - Revert spec-incompliant pause event from bug 1461877. r=cpearce
authorAndreas Pehrson <apehrson@mozilla.com>
Wed, 17 Apr 2019 09:57:59 +0000
changeset 469853 46235152b24e33c652c4e94430897369bbb2090b
parent 469852 58a63a9525b874c59da463c70a5c25ad846471a8
child 469854 2418f3bd2940a5d824ae841690dad8102a20ff39
push id35883
push userbtara@mozilla.com
push dateWed, 17 Apr 2019 21:47:29 +0000
treeherdermozilla-central@02b89c29412b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerscpearce
bugs1525209, 1461877
milestone68.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 1525209 - Revert spec-incompliant pause event from bug 1461877. r=cpearce Differential Revision: https://phabricator.services.mozilla.com/D27864
dom/html/HTMLMediaElement.cpp
--- a/dom/html/HTMLMediaElement.cpp
+++ b/dom/html/HTMLMediaElement.cpp
@@ -1842,17 +1842,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();