Bug 1310140. Part 5 - add DecodingFirstFrameState::HandleVideoSuspendTimeout(). draft
authorJW Wang <jwwang@mozilla.com>
Tue, 18 Oct 2016 10:40:52 +0800
changeset 426830 89d7c1df5e50705646f5d72be13b906e61894594
parent 426829 92c5f64d197f68c1e868e8017be21a9e6e1de996
child 426831 9ef8b894bafcb683c8cd9b8acc5bed000cd6efdf
push id32814
push userjwwang@mozilla.com
push dateWed, 19 Oct 2016 06:32:41 +0000
bugs1310140
milestone52.0a1
Bug 1310140. Part 5 - add DecodingFirstFrameState::HandleVideoSuspendTimeout(). MozReview-Commit-ID: 9CAE5XOQFtQ
dom/media/MediaDecoderStateMachine.cpp
--- a/dom/media/MediaDecoderStateMachine.cpp
+++ b/dom/media/MediaDecoderStateMachine.cpp
@@ -442,16 +442,21 @@ public:
   bool HandleEndOfStream() override
   {
     MaybeFinishDecodeFirstFrame();
     return true;
   }
 
   RefPtr<MediaDecoder::SeekPromise> HandleSeek(SeekTarget aTarget) override;
 
+  void HandleVideoSuspendTimeout() override
+  {
+    // Do nothing for we need to decode the 1st video frame to get the dimensions.
+  }
+
 private:
   // Notify FirstFrameLoaded if having decoded first frames and
   // transition to SEEKING if there is any pending seek, or DECODING otherwise.
   void MaybeFinishDecodeFirstFrame();
 };
 
 class MediaDecoderStateMachine::DecodingState
   : public MediaDecoderStateMachine::StateObject