Bug 1298217. Part 10 - Assert |mState == DECODER_STATE_DECODING| in StartBuffering() which is only called by MaybeStartBuffering() which also asserts that. r=kaku

MozReview-Commit-ID: BhcNiiQZX3I

--HG--
extra : rebase_source : c06aa95641a1f82451ca2182588fe28f760d4cb8

dom/media/MediaDecoderStateMachine.cpp

@@ -2569,15 +2569,7 @@ MediaDecoderStateMachine::GetStatistics()
 void MediaDecoderStateMachine::StartBuffering()
 {
   MOZ_ASSERT(OnTaskQueue());
-
-  if (mState != DECODER_STATE_DECODING) {
-    // We only move into BUFFERING state if we're actually decoding.
-    // If we're currently doing something else, we don't need to buffer,
-    // and more importantly, we shouldn't overwrite mState to interrupt
-    // the current operation, as that could leave us in an inconsistent
-    // state!
-    return;
-  }
+  MOZ_ASSERT(mState == DECODER_STATE_DECODING);
 
   if (IsPlaying()) {
     StopPlayback();