Bug 1323931. Part 4 - no need to update mDecoded{Audio,Video}EndTime in OnSeekTaskResolved(). r?kaku draft
authorJW Wang <jwwang@mozilla.com>
Fri, 16 Dec 2016 15:40:05 +0800
changeset 450845 3d4da26171b70a406252edfd522a79ee2d7c9d3f
parent 450844 f65ea76ae5b908a522b66aebdf29da61d700b871
child 450846 795f197e5d522b96d22001dfab42c6306a520b5e
push id38961
push userjwwang@mozilla.com
push dateMon, 19 Dec 2016 03:21:27 +0000
reviewerskaku
bugs1323931
milestone53.0a1
Bug 1323931. Part 4 - no need to update mDecoded{Audio,Video}EndTime in OnSeekTaskResolved(). r?kaku For they are updated in MDSM::On{Audio,Video}Decoded. MozReview-Commit-ID: 55Od3V9vIf2
dom/media/MediaDecoderStateMachine.cpp
--- a/dom/media/MediaDecoderStateMachine.cpp
+++ b/dom/media/MediaDecoderStateMachine.cpp
@@ -1223,24 +1223,20 @@ private:
       OnSeekTaskResolved();
     }
   }
 
   void OnSeekTaskResolved()
   {
     if (mSeekedAudioData) {
       mMaster->Push(mSeekedAudioData);
-      mMaster->mDecodedAudioEndTime = std::max(
-        mSeekedAudioData->GetEndTime(), mMaster->mDecodedAudioEndTime);
     }
 
     if (mSeekedVideoData) {
       mMaster->Push(mSeekedVideoData);
-      mMaster->mDecodedVideoEndTime = std::max(
-        mSeekedVideoData->GetEndTime(), mMaster->mDecodedVideoEndTime);
     }
 
     SeekCompleted();
   }
 
   void OnSeekTaskRejected(const MediaResult& aError)
   {
     mMaster->DecodeError(aError);