Remove MediaDecoderReader::ForceZeroStartTime()

RESOLVED FIXED in Firefox 53

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: kaku, Assigned: kaku)

Tracking

unspecified
mozilla53
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox53 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
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)

Updated

2 years ago
Assignee: nobody → kaku
Comment hidden (mozreview-request)

Comment 3

2 years ago
mozreview-review
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+
Comment hidden (mozreview-request)
(Assignee)

Comment 6

2 years ago
Try looks good, thanks for the review.
Keywords: checkin-needed

Comment 7

2 years ago
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3230d21e2a8b
Remove MediaDecoderReader::ForceZeroStartTime(); r=jya
Keywords: checkin-needed

Comment 8

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/3230d21e2a8b
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox53: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.