Closed Bug 1324357 Opened 4 years ago Closed 4 years ago

Remove MediaDecoderReader::ForceZeroStartTime()

Categories

(Core :: Audio/Video: Playback, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: kaku, Assigned: kaku)

References

Details

Attachments

(1 file)

In bug 1309516, we made the MFR returns the start time without decoding the first frame and then JW did some refactoring to the MDSM (bug 1319701 and bug 1319706), the MediaDecoderReader::ForceZeroStartTime() is no longer used now. We can remove it.
Assignee: nobody → kaku
Comment on attachment 8820109 [details]
Bug 1324357 - Remove MediaDecoderReader::ForceZeroStartTime();

https://reviewboard.mozilla.org/r/99632/#review100062

will need rebasing following bug 1319992
Attachment #8820109 - Flags: review?(jyavenard) → review+
Try looks good, thanks for the review.
Keywords: checkin-needed
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3230d21e2a8b
Remove MediaDecoderReader::ForceZeroStartTime(); r=jya
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/3230d21e2a8b
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.