Closed Bug 1142433 Opened 10 years ago Closed 10 years ago

Duration not properly read from MEHD atom

Categories

(Core :: Audio/Video, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox38 --- fixed
firefox39 --- fixed

People

(Reporter: jya, Assigned: jya)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

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
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
Attached patch Part2. Update webref (deleted) — Splinter Review
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+
Status: ASSIGNED → RESOLVED
Closed: 10 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.

Attachment

General

Created:
Updated:
Size: