Bug 1156708: Part5. Don't ignore DrainComplete() on first demux. r=me
authorJean-Yves Avenard <jyavenard@mozilla.com>
Mon, 18 May 2015 16:58:55 +1000
changeset 265173 dc240e7fedeeee3c863264ebfc361880520eaa4e
parent 265136 24356909416d194ad8d7ecf550cc06bcddc4a858
child 265174 a9a7bb0afb7b3723fab198d2837ceccc9dca04ac
push id2102
push usermartin.thomson@gmail.com
push dateMon, 18 May 2015 17:35:37 +0000
reviewersme
bugs1156708
milestone41.0a1
Bug 1156708: Part5. Don't ignore DrainComplete() on first demux. r=me Followup on part4 (7ab507ec07cb) which introduce this regression.
dom/media/MediaFormatReader.cpp
--- a/dom/media/MediaFormatReader.cpp
+++ b/dom/media/MediaFormatReader.cpp
@@ -896,16 +896,17 @@ MediaFormatReader::Update(TrackType aTra
       decoder.RejectPromise(END_OF_STREAM, __func__);
       decoder.mDrainComplete = false;
     }
   }
 
   LOGV("Update(%s) ni=%d no=%d", TrackTypeToStr(aTrack), needInput, needOutput);
 
   if (decoder.mDemuxEOS && !decoder.mDemuxEOSServiced) {
+    decoder.mOutputRequested = true;
     decoder.mDecoder->Drain();
     decoder.mDemuxEOSServiced = true;
   }
 
   if (!needInput) {
     return;
   }