Bug 1299074. Part 1 - fix the comment of mSentFirstFrameLoadedEvent because it is not necessarily related to dormant state.
MozReview-Commit-ID: 8YewoncY46P
--- a/dom/media/MediaDecoderStateMachine.h
+++ b/dom/media/MediaDecoderStateMachine.h
@@ -887,21 +887,21 @@ private:
// True if we need to call FinishDecodeFirstFrame() upon frame decoding
// succeeding.
bool mDecodingFirstFrame;
// True if we are back from DECODER_STATE_DORMANT state and
// LoadedMetadataEvent was already sent.
bool mSentLoadedMetadataEvent;
- // True if we are back from DECODER_STATE_DORMANT state and
- // FirstFrameLoadedEvent was already sent, then we can skip
- // SetStartTime because the mStartTime already set before. Also we don't need
- // to decode any audio/video since the MediaDecoder will trigger a seek
- // operation soon.
+
+ // True if we've decoded first frames (thus having the start time) and
+ // notified the FirstFrameLoaded event. Note we can't initiate seek until the
+ // start time is known which happens when the first frames are decoded or we
+ // are playing an MSE stream (the start time is always assumed 0).
bool mSentFirstFrameLoadedEvent;
bool mSentPlaybackEndedEvent;
// True if video decoding is suspended.
bool mVideoDecodeSuspended;
// Track enabling video decode suspension via timer