Bug 1330889. Part 4 - remove the mState checks from MaybeStartPlayback(). r=kikuo
authorJW Wang <jwwang@mozilla.com>
Fri, 13 Jan 2017 15:14:27 +0800
changeset 374568 9584a95515b391be336e6316d8355e3fce2fb0a6
parent 374567 4971ca440d80abc6a260e7f7af2f053c611881d5
child 374569 e4de8d9250ace18ddf418ec43a168803e327e76b
push id6996
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 20:48:21 +0000
treeherdermozilla-beta@d89512dab048 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerskikuo
bugs1330889
milestone53.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 1330889. Part 4 - remove the mState checks from MaybeStartPlayback(). r=kikuo The same reason as P2. MozReview-Commit-ID: GqWgiXjlTrb
dom/media/MediaDecoderStateMachine.cpp
--- a/dom/media/MediaDecoderStateMachine.cpp
+++ b/dom/media/MediaDecoderStateMachine.cpp
@@ -2763,18 +2763,16 @@ MediaDecoderStateMachine::StopPlayback()
   }
 }
 
 void MediaDecoderStateMachine::MaybeStartPlayback()
 {
   MOZ_ASSERT(OnTaskQueue());
   // Should try to start playback only after decoding first frames.
   MOZ_ASSERT(mSentFirstFrameLoadedEvent);
-  MOZ_ASSERT(mState == DECODER_STATE_DECODING ||
-             mState == DECODER_STATE_COMPLETED);
 
   if (IsPlaying()) {
     // Logging this case is really spammy - don't do it.
     return;
   }
 
   if (mPlayState != MediaDecoder::PLAY_STATE_PLAYING) {
     DECODER_LOG("Not starting playback [mPlayState=%d]", mPlayState.Ref());