Bug 1058422: Don't immediately request new data r=cajbir
authorJean-Yves Avenard <jyavenard@mozilla.com>
Fri, 29 Aug 2014 12:15:40 +1200
changeset 223947 1d074a9ef902b3c71bcabcd34b31143157a7793e
parent 223946 8669b13c8567a30c42854662ff8de9d6f1d77b3b
child 223948 5ffa67d71687c42275cbb231e7e96a33d1606f6d
push id3979
push userraliiev@mozilla.com
push dateMon, 13 Oct 2014 16:35:44 +0000
treeherdermozilla-beta@30f2cc610691 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerscajbir
bugs1058422
milestone34.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1058422: Don't immediately request new data r=cajbir
content/media/mediasource/MediaSourceReader.cpp
--- a/content/media/mediasource/MediaSourceReader.cpp
+++ b/content/media/mediasource/MediaSourceReader.cpp
@@ -134,21 +134,16 @@ MediaSourceReader::OnVideoEOS()
   if (SwitchReaders(SWITCH_FORCED)) {
     // Success! Resume decoding with next video decoder.
     RequestVideoData(false, mTimeThreshold);
   } else if (IsEnded()) {
     // End of stream.
     MSE_DEBUG("MediaSourceReader(%p)::OnVideoEOS reader=%p EOS (readers=%u)",
               this, mVideoReader.get(), mDecoders.Length());
     GetCallback()->OnVideoEOS();
-  } else {
-    // If a new decoder isn't ready to respond with frames yet, we're going to
-    // keep hitting this path at 1/frame_duration Hz. Bug 1058422 is raised to
-    // address this issue.
-    RequestVideoData(false, mTimeThreshold);
   }
 }
 
 void
 MediaSourceReader::OnDecodeError()
 {
   GetCallback()->OnDecodeError();
 }