Videos with negative time frames gets evicted.

RESOLVED FIXED in Firefox 41

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jya, Assigned: jya)

Tracking

Trunk
mozilla41
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox41 fixed)

Details

Attachments

(3 attachments)

Some MP4 videos starts with a keyframe that has a negative presentation time stamp.

Those videos do not play well. The new MSE code following the spec evicts them automatically as they are not within Window Append Start (default:0) / Window Append End (default:+oo).

And the MP4Reader / MediaFormatReader kick the skip to next keyframe logic.
Some videos have negative start time, as the default window size restriction
is [0, oo) those frames would be evicted (and as they are typically keyframes, rendering
the content unplayable from the start.
Attachment #8622189 - Flags: review?(cpearce)
MP4Reader.
Attachment #8622191 - Flags: review?(cpearce)
Assignee: nobody → jyavenard
Comment on attachment 8622189 [details] [diff] [review]
P1. Relax append window restrictions.

Review of attachment 8622189 [details] [diff] [review]:
-----------------------------------------------------------------

Passing review to Alfredo. Thanks for taking on this responsibility Alfredo!
Attachment #8622189 - Flags: review?(cpearce) → review?(ayang)
Comment on attachment 8622190 [details] [diff] [review]
P2. Do not skip keyframe with negative timestamp.

Review of attachment 8622190 [details] [diff] [review]:
-----------------------------------------------------------------

Passing review to Alfredo. Thanks for taking on this responsibility Alfredo!
Attachment #8622190 - Flags: review?(cpearce) → review?(ayang)
Comment on attachment 8622191 [details] [diff] [review]
P3. Do not skip keyframe with negative timestamp.

Review of attachment 8622191 [details] [diff] [review]:
-----------------------------------------------------------------

Passing review to Alfredo. Thanks for taking on this responsibility Alfredo!
Attachment #8622191 - Flags: review?(cpearce) → review?(ayang)
Could you please attach a video with "negative PTS" first key frame? Just curious with this kind of video and it probably has the same problem on normal playback.
Thank you.
Flags: needinfo?(jyavenard)
Those are Youtube videos, plenty of them.. let me see if I can grab some.
Flags: needinfo?(jyavenard)
I'm going to rpending those, because I don't want to wait any longer and it prevents some youtube videos to play.
Attachment #8622189 - Flags: review?(ayang) → review+
Attachment #8622190 - Flags: review?(ayang) → review+
Attachment #8622191 - Flags: review?(ayang) → review+
ayang: I was just provided such video which on anything but nightly will cause the first 2s of video to be non-decodable (as the first keyframe at -83377)
http://videojs.viewlorium.com/watch.php?brittany-murphy-story
Flags: needinfo?(ayang)
Thank you!
Flags: needinfo?(ayang)
You need to log in before you can comment on or make changes to this bug.