Duration not properly read from MEHD atom

RESOLVED FIXED in Firefox 38

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jya, Assigned: jya)

Tracking

(Blocks 1 bug)

Trunk
mozilla39
x86
macOS
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox38 fixed, firefox39 fixed)

Details

Attachments

(2 attachments)

Introduced in bug 1120075

It appears that the duration found in the movie extend header is defined according to the timescale in the movie header (mvhd) atom.

And not in milliseconds as first assumed (all the samples I had use a timescale = 1000)
properly adjust duration according to movie header timescale
Attachment #8576546 - Flags: review?(ajones)
Assignee: nobody → jyavenard
Status: NEW → ASSIGNED
Blocks: 1141785
Attachment #8576546 - Flags: review?(ajones) → review+
This cause a lot of webref test to fail. IT appears that was cause we were incorrectly calculating durations before. So few tests now passes.
Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/aa48e92f7e57 - looks like the best two places to catch the failures are OS X 10.10 opt, in media-source/mediasource-append-buffer.html like https://treeherder.mozilla.org/logviewer.html#?job_id=7560429&repo=mozilla-inbound and Windows 8 opt in media-source/mediasource-remove.html like https://treeherder.mozilla.org/logviewer.html#?job_id=7566449&repo=mozilla-inbound
Update webref. We still timeout due to bug 1128332 that now occurs on all platforms.
Attachment #8577992 - Flags: review?(karlt)
Attachment #8577992 - Flags: review?(karlt) → review+
https://hg.mozilla.org/mozilla-central/rev/a7425df3572a
https://hg.mozilla.org/mozilla-central/rev/864a62ab3500
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
This bugs needs to be uplifted as otherwise webref tests will now fail (now that they have been re-activated due to bug 1144509)
Flags: needinfo?(giles)
Comment on attachment 8576546 [details] [diff] [review]
Properly scale duration found in movie extend header box

Approval Request Comment
[Feature/regressing bug #]:1142433
[User impact if declined]:Tests recently re-enabled will fail.
[Describe test coverage new/current, TreeHerder]:Been on m-c for over a week. 
[Risks and why]: Low. The tests were re-enabled, previously disabled. Problem was always there
[String/UUID change made/needed]:none
Attachment #8576546 - Flags: approval-mozilla-aurora?
the approval request is for both patches.
Attachment #8576546 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Flags: needinfo?(giles)
You need to log in before you can comment on or make changes to this bug.