1065235 - Apply eviction to all decoders owned by TrackBuffer and make EvictData threshold apply per-TrackBuffer rather than per-decoder

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Comment 1

[Matthew Gregan [:kinetik]]

Attachment: Apply eviction to all decoders owned by TrackBuffer and make EvictData threshold apply per-TrackBuffer rather than per-decoder.

Comment 2

[Matthew Gregan [:kinetik]]

Attachment: Apply eviction to all decoders owned by TrackBuffer and make EvictData threshold apply per-TrackBuffer rather than per-decoder.

Bail early if there's nothing to evict.

Comment 3

[Matthew Gregan [:kinetik]]

Attachment: Apply eviction to all decoders owned by TrackBuffer and make EvictData threshold apply per-TrackBuffer rather than per-decoder.

(wrong patch attached last time)

Comment 4

[Matthew Gregan [:kinetik]]

Attachment: Make the SourceBuffer eviction threshold a hidden pref.

kft mentioned doing this, but I don't think there was ever a bug/patch for it, so here's one.  A typical ~4 min YouTube video at 1080p is around the default eviction threshold, so being able to shrink the threshold easily for testing would be immensely useful.

Comment 5

[Matthew Gregan [:kinetik]]

https://tbpl.mozilla.org/?tree=Try&rev=0546aec2d701

Comment 6

[Matthew Gregan [:kinetik]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/1baee6918ab3
https://hg.mozilla.org/integration/mozilla-inbound/rev/7cb9cd119176

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/1baee6918ab3
https://hg.mozilla.org/mozilla-central/rev/7cb9cd119176