Bug 1306566 - Remove the check for mIsVisible from MDSM::DonePrerollingVideo(). r=kamidphish
authorJW Wang <jwwang@mozilla.com>
Fri, 30 Sep 2016 16:11:58 +0800
changeset 316355 02e7753c526100e7041d94dbc4ae2514d4c093fd
parent 316296 935121aed27e167e7b1c9d0d4f67768fc753bdf7
child 316356 22d95521160beb0c26e6dcee7c979e0cb617bb56
push id30770
push userkwierso@gmail.com
push dateWed, 05 Oct 2016 00:00:48 +0000
treeherdermozilla-central@3470e326025c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerskamidphish
bugs1306566
milestone52.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 1306566 - Remove the check for mIsVisible from MDSM::DonePrerollingVideo(). r=kamidphish MozReview-Commit-ID: GIEibT6YKmC
dom/media/MediaDecoderStateMachine.h
--- a/dom/media/MediaDecoderStateMachine.h
+++ b/dom/media/MediaDecoderStateMachine.h
@@ -719,18 +719,17 @@ private:
     MOZ_ASSERT(OnTaskQueue());
     return !IsAudioDecoding() ||
         GetDecodedAudioDuration() >= AudioPrerollUsecs() * mPlaybackRate;
   }
 
   bool DonePrerollingVideo()
   {
     MOZ_ASSERT(OnTaskQueue());
-    return !mIsVisible ||
-        !IsVideoDecoding() ||
+    return !IsVideoDecoding() ||
         static_cast<uint32_t>(VideoQueue().GetSize()) >=
             VideoPrerollFrames() * mPlaybackRate + 1;
   }
 
   void MaybeStopPrerolling();
 
   // When we start decoding (either for the first time, or after a pause)
   // we may be low on decoded data. We don't want our "low data" logic to