Bug 1317576. Part 7 - remove the assertion that doesn't make sense anymore. r=cpearce
authorJW Wang <jwwang@mozilla.com>
Tue, 15 Nov 2016 15:13:21 +0800
changeset 322678 1f9d289e8272f27659bf980528b6d89f61f7ac87
parent 322677 6f39dfa71498b1398c4a6573dafc0ae3283c0409
child 322679 d282d9e8232adadbf9664f2aa2cc18ceeb0b35af
push id21
push usermaklebus@msu.edu
push dateThu, 01 Dec 2016 06:22:08 +0000
reviewerscpearce
bugs1317576
milestone53.0a1
Bug 1317576. Part 7 - remove the assertion that doesn't make sense anymore. r=cpearce 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()) {