1324357 - Remove MediaDecoderReader::ForceZeroStartTime()

Status: RESOLVED FIXED

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

Description

[Tzuhao Kuo [:kaku]]

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.

Comment 1

[Tzuhao Kuo [:kaku]]

Attachment: Bug 1324357 - Remove MediaDecoderReader::ForceZeroStartTime();

Review commit: https://reviewboard.mozilla.org/r/99632/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/99632/

Comment 2

[Tzuhao Kuo [:kaku]]

Try:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=588ac49b2ddf52f7ff682edc5d0717e238b38491
https://treeherder.mozilla.org/#/jobs?repo=try&revision=05e64dfe8219a79852e9e02b87d007aefe8625d8

Comment 3

[Jean-Yves Avenard [:jya]]

Comment on attachment 8820109 [details]
Bug 1324357 - Remove MediaDecoderReader::ForceZeroStartTime();

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

will need rebasing following bug 1319992

Comment 4

[Tzuhao Kuo [:kaku]]

Comment on attachment 8820109 [details]
Bug 1324357 - Remove MediaDecoderReader::ForceZeroStartTime();

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/99632/diff/1-2/

Comment 5

[Tzuhao Kuo [:kaku]]

Rebase onto bug 1319992.
Try:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=dcd64d952a685bc2c3792434ee76c0967dda7c24
https://treeherder.mozilla.org/#/jobs?repo=try&revision=a4db840d68119ebe5d607cc77842ea8993790264

Comment 6

[Tzuhao Kuo [:kaku]]

Try looks good, thanks for the review.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3230d21e2a8b
Remove MediaDecoderReader::ForceZeroStartTime(); r=jya

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/3230d21e2a8b