Bug 1283370. Part 7 - remove setting mNeedToStopPrerollingVideo per discussion in https://reviewboard.mozilla.org/r/43689/#comment54421. r=kaku draft
authorJW Wang <jwwang@mozilla.com>
Fri, 01 Jul 2016 11:47:44 +0800
changeset 384379 d8d2b60be9a17f35ed65dbb8e0defe14f7584f4b
parent 384378 79f092e64ac84983262100bc50eaec2b9471cd90
child 524690 78b5d1cff5a675a099baec07392874d2d130863a
push id22256
push userjwwang@mozilla.com
push dateWed, 06 Jul 2016 06:43:32 +0000
reviewerskaku
bugs1283370, 43689
milestone50.0a1
Bug 1283370. Part 7 - remove setting mNeedToStopPrerollingVideo per discussion in https://reviewboard.mozilla.org/r/43689/#comment54421. r=kaku MozReview-Commit-ID: 4yj3EBH84OF
dom/media/NextFrameSeekTask.cpp
--- a/dom/media/NextFrameSeekTask.cpp
+++ b/dom/media/NextFrameSeekTask.cpp
@@ -348,21 +348,16 @@ NextFrameSeekTask::OnVideoNotDecoded(Med
     RejectIfExist(__func__);
     return;
   }
 
   // If the decoder is waiting for data, we tell it to call us back when the
   // data arrives.
   if (aReason == MediaDecoderReader::WAITING_FOR_DATA) {
     mReader->WaitForData(MediaData::VIDEO_DATA);
-
-    // We are out of data to decode and will enter buffering mode soon.
-    // We want to play the frames we have already decoded, so we stop pre-rolling
-    // and ensure that loadeddata is fired as required.
-    mNeedToStopPrerollingVideo = true;
     return;
   }
 
   if (aReason == MediaDecoderReader::CANCELED) {
     EnsureVideoDecodeTaskQueued();
     return;
   }