Bug 1317576. Part 7 - remove the assertion that doesn't make sense anymore. draft
authorJW Wang <jwwang@mozilla.com>
Tue, 15 Nov 2016 15:13:21 +0800
changeset 439474 78bf3c347bf0228f6a28ca5b088f61dc47b476dd
parent 439473 6f9f800e338bc09179f1b7818d93395fff92993a
child 537172 253b7b4ef31469f4bf4788be94cdb9ac8e765613
push id36009
push userjwwang@mozilla.com
push dateWed, 16 Nov 2016 02:24:06 +0000
bugs1317576
milestone53.0a1
Bug 1317576. Part 7 - remove the assertion that doesn't make sense anymore. It is possible to call AudioEndTime() in CompletedState::Step() with MedisSink not started at all. This happens when seeking to the end on a video only file. MozReview-Commit-ID: 78P72DAPpWI
dom/media/MediaDecoderStateMachine.cpp
--- a/dom/media/MediaDecoderStateMachine.cpp
+++ b/dom/media/MediaDecoderStateMachine.cpp
@@ -3071,17 +3071,16 @@ MediaDecoderStateMachine::IsShutdown() c
 
 int64_t
 MediaDecoderStateMachine::AudioEndTime() const
 {
   MOZ_ASSERT(OnTaskQueue());
   if (mMediaSink->IsStarted()) {
     return mMediaSink->GetEndTime(TrackInfo::kAudioTrack);
   }
-  MOZ_ASSERT(!HasAudio());
   return -1;
 }
 
 int64_t
 MediaDecoderStateMachine::VideoEndTime() const
 {
   MOZ_ASSERT(OnTaskQueue());
   if (mMediaSink->IsStarted()) {