1304254 - Parse MovieExtendsBox to get duration

Status: RESOLVED FIXED

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

Description

[Alfredo Yang (:alfredo)]

In MSE mochitest test_SeekNoData_mp4.html, rust parser fails to retrieve duration in mvex box.

mp4parse-rust needs to be able to parse this box.

Comment 1

[Alfredo Yang (:alfredo)]

Actually, it is fragment_duration in mehd.

Comment 2

[Alfredo Yang (:alfredo)]

Attachment: Bug 1304254 - Use fragment_duration as track duration.

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

Comment 3

[Alfredo Yang (:alfredo)]

https://github.com/mozilla/mp4parse-rust/pull/27

Comment 4

[Ralph Giles (:rillian)]

Comment on attachment 8793744 [details]
Bug 1304254 - Use fragment_duration as track duration.

https://reviewboard.mozilla.org/r/80420/#review79574

Comment 5

[Alfredo Yang (:alfredo)]

Comment on attachment 8793744 [details]
Bug 1304254 - Use fragment_duration as track duration.

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

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/431dacf6e977
Use fragment_duration as track duration. r=rillian

Comment 7

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/431dacf6e977