Closed Bug 1252766 Opened 5 years ago Closed 5 years ago
Decoder State Machine::m Decode To Seek Target
MozReview Request: Bug 1252766 - Remove MediaDecoderStateMachine::mDecodeToSeekTarget which is never read. r=kaku.
58 bytes, text/x-review-board-request
After fixing bug 1252762, mDecodeToSeekTarget is not used and can be removed.
Assignee: nobody → jwwang
Depends on: 1252762
AFAICT, it is currently used. mDecodeToSeekTarget is used to determine if when state == DECODER_STATE_SEEKING and if so if we need to continue decoding until we reach the target. It is set to false but MDSM::SeekComplete() may stay in DECODER_STATE_SEEKING if a new seek was started before the current one completing. Can probably modify the logic so we don't we need, but the title of this bug indicates that it's not used, and right now it is.
It won't after fixing bug 1252762. When writing patches for bug 1252762, it turned out mDecodeToSeekTarget is never read and can be removed safely.
Review commit: https://reviewboard.mozilla.org/r/38081/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/38081/
Attachment #8726553 - Flags: review?(tkuo)
Comment on attachment 8726553 [details] MozReview Request: Bug 1252766 - Remove MediaDecoderStateMachine::mDecodeToSeekTarget which is never read. r=kaku. https://reviewboard.mozilla.org/r/38081/#review34615
Attachment #8726553 - Flags: review?(tkuo) → review+
You need to log in before you can comment on or make changes to this bug.