Bug 1274445: P2. Don't unnecessarily reset the decoder context. r?kamidphish draft
authorJean-Yves Avenard <jyavenard@mozilla.com>
Mon, 30 May 2016 15:46:55 +1000
changeset 372850 2d6e61b29fe68fcd5416bf807ca907968e7af76d
parent 372849 2453298ed43598bbab10286bbebad01073041be1
child 372874 98bf4649bf58e9a3f593ab1174d55bf85e443831
push id19607
push userbmo:jyavenard@mozilla.com
push dateMon, 30 May 2016 11:39:19 +0000
reviewerskamidphish
bugs1274445
milestone49.0a1
Bug 1274445: P2. Don't unnecessarily reset the decoder context. r?kamidphish MediaFormatReader::Reset is already resetting the context. MozReview-Commit-ID: AWIh9aTFA6S
dom/media/MediaFormatReader.cpp
--- a/dom/media/MediaFormatReader.cpp
+++ b/dom/media/MediaFormatReader.cpp
@@ -1334,26 +1334,24 @@ MediaFormatReader::ResetDecode(TargetQue
   }
   mVideo.mWaitingPromise.RejectIfExists(WaitForDataRejectValue(MediaData::VIDEO_DATA, WaitForDataRejectValue::CANCELED), __func__);
 
   // Reset miscellaneous seeking state.
   mPendingSeekTime.reset();
 
   if (HasVideo()) {
     mVideo.ResetDemuxer();
-    mVideo.ResetState();
     Reset(TrackInfo::kVideoTrack);
     if (mVideo.HasPromise()) {
       mVideo.RejectPromise(CANCELED, __func__);
     }
   }
 
   if (HasAudio() && aQueues == AUDIO_VIDEO) {
     mAudio.ResetDemuxer();
-    mAudio.ResetState();
     Reset(TrackInfo::kAudioTrack);
     if (mAudio.HasPromise()) {
       mAudio.RejectPromise(CANCELED, __func__);
     }
   }
   return MediaDecoderReader::ResetDecode(aQueues);
 }