Bug 1229742: P1. Only clear EOS flag if we have new data. r?gerald draft
authorJean-Yves Avenard <jyavenard@mozilla.com>
Tue, 09 Feb 2016 00:35:20 +1100
changeset 329542 38111f5654d5813d5413766c70bbb417af30fbea
parent 329541 89670b4ab9f3e2edb1779bd154c7d115c1471b96
child 329543 7f7b70b3ff296e80eb14b04aecf0a80bf3e7090a
push id10542
push userbmo:jyavenard@mozilla.com
push dateMon, 08 Feb 2016 13:46:40 +0000
reviewersgerald
bugs1229742
milestone47.0a1
Bug 1229742: P1. Only clear EOS flag if we have new data. r?gerald MozReview-Commit-ID: ttmWsAIPxQ
dom/media/MediaFormatReader.cpp
--- a/dom/media/MediaFormatReader.cpp
+++ b/dom/media/MediaFormatReader.cpp
@@ -789,17 +789,17 @@ MediaFormatReader::UpdateReceivedNewData
     // We do not want to clear mWaitingForData or mDemuxEOS while
     // a drain is in progress in order to properly complete the operation.
     return false;
   }
 
   bool hasLastEnd;
   media::TimeUnit lastEnd = decoder.mTimeRanges.GetEnd(&hasLastEnd);
   if (hasLastEnd) {
-    if (decoder.mLastTimeRangesEnd && decoder.mLastTimeRangesEnd.ref() > lastEnd) {
+    if (decoder.mLastTimeRangesEnd && decoder.mLastTimeRangesEnd.ref() < lastEnd) {
       // New data was added after our previous end, we can clear the EOS flag.
       decoder.mDemuxEOS = false;
     }
     decoder.mLastTimeRangesEnd = Some(lastEnd);
   }
 
   decoder.mReceivedNewData = false;
   if (decoder.mTimeThreshold) {