Bug 1453843 - Remove extraneous play promise reject. r=bryce
authorChris Pearce <cpearce@mozilla.com>
Fri, 20 Apr 2018 17:54:08 +1200
changeset 468535 6714516bb697bab6b0cc4514a0b12a705b184b28
parent 468534 37ac866c0cdc64f65cb5fb41967cecf826b84bcb
child 468536 68a1436f3e09e2e73cf4d13fc9f35813032d76cc
push id9165
push userasasaki@mozilla.com
push dateThu, 26 Apr 2018 21:04:54 +0000
treeherdermozilla-beta@064c3804de2e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbryce
bugs1453843
milestone61.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 1453843 - Remove extraneous play promise reject. r=bryce We already reject the play promises when we call Pause(), so this extra reject is unnecessary. MozReview-Commit-ID: 6LKw7hCwJPH
dom/html/HTMLMediaElement.cpp
--- a/dom/html/HTMLMediaElement.cpp
+++ b/dom/html/HTMLMediaElement.cpp
@@ -5542,17 +5542,16 @@ void HTMLMediaElement::PlaybackEnded()
     SetCurrentTime(0);
     return;
   }
 
   FireTimeUpdate(false);
 
   if (!mPaused) {
     Pause();
-    AsyncRejectPendingPlayPromises(NS_ERROR_DOM_MEDIA_ABORT_ERR);
   }
 
   if (mSrcStream) {
     // A MediaStream that goes from inactive to active shall be eligible for
     // autoplay again according to the mediacapture-main spec.
     mAutoplaying = true;
   }