Closed Bug 1244410 Opened 4 years ago Closed 4 years ago

Intermittent e10s test_DrainOnMissingData_mp4.html | Video has correct currentTime. - got 0.801677, expected 0.801666

Categories

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

Unspecified
Linux
defect

Tracking

()

RESOLVED FIXED
mozilla49
Tracking Status
e10s + ---
firefox47 --- affected
firefox49 --- fixed

People

(Reporter: philor, Assigned: jya)

References

(Blocks 1 open bug)

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

tracking-e10s: --- → ?
Priority: -- → P5
Component: Audio/Video → Audio/Video: Playback
OS: Unspecified → Linux
FFmpeg's AVFrame pkt_dts doesn't contain the dts of the frame used to decode the frame; but of the frame "that triggered returning this frame.". The last frame was returned when draining which is done by feeding the decoder with dummy frames ; all having a dts of 0.

Additionally, rename DurationMap argument name from aDts to aKey.

Review commit: https://reviewboard.mozilla.org/r/51025/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/51025/
Attachment #8749534 - Flags: review?(gsquelart)
Assignee: nobody → jyavenard
Comment on attachment 8749534 [details]
MozReview Request: Bug 1244410: [ffmpeg] Ensure the last drained frame has the proper duration set. r?gerald

https://reviewboard.mozilla.org/r/51025/#review47685
Attachment #8749534 - Flags: review?(gsquelart) → review+
https://hg.mozilla.org/mozilla-central/rev/7d9879db28af
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
Depends on: 1270323
The one showing in https://treeherder.mozilla.org/logviewer.html#?job_id=3916007&repo=mozilla-central

is actually the same as bug 1272964; Skip to next keyframe actually jumped over a gap, causing currentTime to be at 9s rather than 0.8s
You need to log in before you can comment on or make changes to this bug.