Bug 1306945 - remove the check of mPlayState in the MDSM::VisibilityChanged(); r?jwwang, kamidphish draft
authorKaku Kuo <kaku@mozilla.com>
Tue, 04 Oct 2016 16:10:13 +0800
changeset 420516 212ccaadf8328e8e9ff0247a30972c71a44186f5
parent 420379 c8a660c5f105e60ad536ddde0c3edd637ab5b7c1
child 532828 275285d226e80f302971df27c87902a844590fd6
push id31218
push userbmo:kaku@mozilla.com
push dateTue, 04 Oct 2016 08:10:44 +0000
reviewersjwwang, kamidphish
bugs1306945
milestone52.0a1
Bug 1306945 - remove the check of mPlayState in the MDSM::VisibilityChanged(); r?jwwang, kamidphish MozReview-Commit-ID: JBJJ7dPt64i
dom/media/MediaDecoderStateMachine.cpp
--- a/dom/media/MediaDecoderStateMachine.cpp
+++ b/dom/media/MediaDecoderStateMachine.cpp
@@ -1806,21 +1806,16 @@ void MediaDecoderStateMachine::Visibilit
   DECODER_LOG("VisibilityChanged: mIsVisible=%d, "
               "mVideoDecodeSuspended=%c, mIsReaderSuspended=%d",
               mIsVisible.Ref(), mVideoDecodeSuspended ? 'T' : 'F', mIsReaderSuspended.Ref());
 
   if (!HasVideo()) {
     return;
   }
 
-  // If not playing then there's nothing to do.
-  if (mPlayState != MediaDecoder::PLAY_STATE_PLAYING) {
-    return;
-  }
-
   // Start timer to trigger suspended decoding state when going invisible.
   if (!mIsVisible) {
     TimeStamp target = TimeStamp::Now() + SuspendBackgroundVideoDelay();
 
     RefPtr<MediaDecoderStateMachine> self = this;
     mVideoDecodeSuspendTimer.Ensure(target,
                                     [=]() { self->OnSuspendTimerResolved(); },
                                     [=]() { self->OnSuspendTimerRejected(); });