Bug 1325004. Part 3 - let DecodingFirstFrameState call Push{Audio,Video} instead. r?kikuo draft
authorJW Wang <jwwang@mozilla.com>
Wed, 21 Dec 2016 15:54:30 +0800
changeset 453986 ee4c68c34e4ab13145a0640ad1c97824d4b127f9
parent 453985 1941eaab3688a450cd6d8daf6a6df7af7d30e33a
child 453987 2efcae21b115fd37b596030d1ade5a3e98450888
push id39788
push userjwwang@mozilla.com
push dateTue, 27 Dec 2016 03:48:35 +0000
reviewerskikuo
bugs1325004
milestone53.0a1
Bug 1325004. Part 3 - let DecodingFirstFrameState call Push{Audio,Video} instead. r?kikuo DecodingFirstFrameState needs only at most one audio or video sample. MozReview-Commit-ID: 2pRrPlCUBSf
dom/media/MediaDecoderStateMachine.cpp
--- a/dom/media/MediaDecoderStateMachine.cpp
+++ b/dom/media/MediaDecoderStateMachine.cpp
@@ -481,23 +481,23 @@ public:
 
   State GetState() const override
   {
     return DECODER_STATE_DECODING_FIRSTFRAME;
   }
 
   void HandleAudioDecoded(MediaData* aAudio) override
   {
-    mMaster->Push(aAudio);
+    mMaster->PushAudio(aAudio);
     MaybeFinishDecodeFirstFrame();
   }
 
   void HandleVideoDecoded(MediaData* aVideo, TimeStamp aDecodeStart) override
   {
-    mMaster->Push(aVideo);
+    mMaster->PushVideo(aVideo);
     MaybeFinishDecodeFirstFrame();
   }
 
   void HandleEndOfStream() override
   {
     MaybeFinishDecodeFirstFrame();
   }