Bug 1319706. Part 3 - remove the assertion that is no longer valid. r=kaku
authorJW Wang <jwwang@mozilla.com>
Thu, 24 Nov 2016 14:51:00 +0800
changeset 324619 3dc995228de419ee09061b826f21e0ad547abc97
parent 324618 03144421b4a60da0fa4cbaf0548252bb2af3dc5a
child 324620 e355836470f90868b9bba2d15e2acc10f8b6ff4b
push id24
push usermaklebus@msu.edu
push dateTue, 20 Dec 2016 03:11:33 +0000
reviewerskaku
bugs1319706
milestone53.0a1
Bug 1319706. Part 3 - remove the assertion that is no longer valid. r=kaku MozReview-Commit-ID: FgmMTw7cy1w
dom/media/MediaDecoderStateMachine.cpp
--- a/dom/media/MediaDecoderStateMachine.cpp
+++ b/dom/media/MediaDecoderStateMachine.cpp
@@ -1435,18 +1435,16 @@ SeekingState::SeekCompleted()
 
   // We want to resolve the seek request prior finishing the first frame
   // to ensure that the seeked event is fired prior loadeded.
   mSeekJob.Resolve(__func__);
 
   // Notify FirstFrameLoaded now if we haven't since we've decoded some data
   // for readyState to transition to HAVE_CURRENT_DATA and fire 'loadeddata'.
   if (!mMaster->mSentFirstFrameLoadedEvent) {
-    // Only MSE can start seeking before finishing decoding first frames.
-    MOZ_ASSERT(Reader()->ForceZeroStartTime());
     mMaster->FinishDecodeFirstFrame();
   }
 
   // Ensure timestamps are up to date.
   if (!mSeekJob.mTarget.IsVideoOnly()) {
     // Don't update playback position for video-only seek.
     // Otherwise we might have |newCurrentTime > mMediaSink->GetPosition()|
     // and fail the assertion in GetClock() since we didn't stop MediaSink.