Bug 1319706. Part 3 - remove the assertion that is no longer valid. r?kaku draft
authorJW Wang <jwwang@mozilla.com>
Thu, 24 Nov 2016 14:51:00 +0800
changeset 445185 ec045344e92e0df172d98049a2b376a26cae560d
parent 445184 3a3074b1eb80c998c73d4476e7e621f94654edb1
child 445186 234a7ec9d28778046bd19f9e218a19618a2e9b21
push id37455
push userjwwang@mozilla.com
push dateTue, 29 Nov 2016 09:40:57 +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.