Closed Bug 1120075 Opened 10 years ago Closed 10 years ago

MP4: Use movie extend header box's duration if no track duration could be found

Categories

(Core :: Audio/Video, defect)

Product:
Core
Component:
Audio/Video
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla38
Tracking Flags:
Tracking Status
firefox36 --- fixed
firefox37 --- fixed
firefox38 --- fixed

People

(Reporter: jya, Assigned: jya)

Details

Attachments

(2 files, 1 obsolete file)

Use Movie Extend Header's duration as fallback when available
4.59 KB, patch
ajones
: review+
Sylvestre
: approval-mozilla-aurora+
Sylvestre
: approval-mozilla-beta+
Details | Diff | Splinter Review
Update w3c web reference tests
1.02 KB, patch
Details | Diff | Splinter Review
We currently only extract the duration from the trak's atom track header. Which may not always exit. Use the mvex's mehd (Movie Extend Header) if available as fallback.
Read Movie Extend Header's duration and use as fallback if no track header's duration is defined.
Attachment #8547035 - Flags: review?(ajones)
Assignee: nobody → jyavenard
Status: NEW → ASSIGNED
To test: http://people.mozilla.org/~jyavenard/tests/mse_mp4/gizmo.html?eos=0&eosat=-1&duration=-1&init=1 duration is properly set to 5.534s ... (same as what IE and Chrome are doing)
Attachment #8547035 - Flags: review?(ajones) → review+
Attached patch Update w3c web reference tests (obsolete) — — Splinter Review
This change was actually required by this mehd change
Comment on attachment 8550226 [details] [diff] [review] Update w3c web reference tests Carrying r+ from bug 1120086 by cajbir
Attachment #8550226 - Flags: review+
oops, wrong patch uploaded
Attachment #8550226 - Attachment is obsolete: true
https://hg.mozilla.org/mozilla-central/rev/0bf6cb3e40dc https://hg.mozilla.org/mozilla-central/rev/936880d652de
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
Comment on attachment 8547035 [details] [diff] [review] Use Movie Extend Header's duration as fallback when available Approval Request Comment [Feature/regressing bug #]: MSE [User impact if declined]: Less consistent testing, sites more likely to serve flash video. [Describe test coverage new/current, TBPL]: Landed on m-c. [Risks and why]: Risk is relatively low. We parse a new duration field, but only use it in fallback. [String/UUID change made/needed]: None This request applies to both patches on this bug.
Attachment #8547035 - Flags: approval-mozilla-beta?
Attachment #8547035 - Flags: approval-mozilla-aurora?
Attachment #8547035 - Flags: approval-mozilla-beta?
Attachment #8547035 - Flags: approval-mozilla-beta+
Attachment #8547035 - Flags: approval-mozilla-aurora?
Attachment #8547035 - Flags: approval-mozilla-aurora+
Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: