rx-player 3.26.2 → 3.27.0-dev.20220317

package/src/core/segment_buffers/implementations/text/html/html_text_segment_buffer.ts

@@ -48,8 +48,6 @@ const { onEnded$,
         onSeeked$,
         onSeeking$ } = events;
 
-const { MAXIMUM_HTML_TEXT_TRACK_UPDATE_INTERVAL,
-        TEXT_TRACK_SIZE_CHECKS_INTERVAL } = config;
 
 /**
  * Generate the interval at which TextTrack HTML Cues should be refreshed.
@@ -60,7 +58,7 @@ function generateRefreshInterval(videoElement : HTMLMediaElement) : Observable<b
   const seeking$ = onSeeking$(videoElement);
   const seeked$ = onSeeked$(videoElement);
   const ended$ = onEnded$(videoElement);
-
+  const { MAXIMUM_HTML_TEXT_TRACK_UPDATE_INTERVAL } = config.getCurrent();
   const manualRefresh$ = observableMerge(seeked$, ended$);
   const autoRefresh$ = observableInterval(MAXIMUM_HTML_TEXT_TRACK_UPDATE_INTERVAL)
     .pipe(startWith(null));
@@ -182,7 +180,7 @@ export default class HTMLTextSegmentBuffer extends SegmentBuffer {
           this._disableCurrentCues();
           return;
         }
-
+        const { MAXIMUM_HTML_TEXT_TRACK_UPDATE_INTERVAL } = config.getCurrent();
         // to spread the time error, we divide the regular chosen interval.
         const time = Math.max(this._videoElement.currentTime +
                               (MAXIMUM_HTML_TEXT_TRACK_UPDATE_INTERVAL / 1000) / 2,
@@ -420,6 +418,7 @@ export default class HTMLTextSegmentBuffer extends SegmentBuffer {
                                element : HTMLElement; } => cue.resolution !== null);
 
     if (proportionalCues.length > 0) {
+      const { TEXT_TRACK_SIZE_CHECKS_INTERVAL } = config.getCurrent();
       // update propertionally-sized elements periodically
       onHeightWidthChange(this._textTrackElement, TEXT_TRACK_SIZE_CHECKS_INTERVAL)
         .pipe(takeUntil(this._clearSizeUpdates$),
@@ -445,11 +444,11 @@ export interface INativeTextTracksBufferSegmentData {
    * This is mostly needed for "sami" subtitles, to know which cues can / should
    * be parsed.
    */
-  language? : string;
+  language? : string | undefined;
   /** start time from which the segment apply, in seconds. */
-  start? : number;
+  start? : number | undefined;
   /** end time until which the segment apply, in seconds. */
-  end? : number;
+  end? : number | undefined;
 }
 
 /**

package/src/core/segment_buffers/implementations/text/html/text_track_cues_store.ts

@@ -63,8 +63,8 @@ export default class TextTrackCuesStore {
 
     // begins at the end as most of the time the player will ask for the last
     // CuesGroup
-    for (let i = cuesBuffer.length - 1; i >= 0; i--) {
-      const segment = cuesBuffer[i];
+    for (let cueIdx = cuesBuffer.length - 1; cueIdx >= 0; cueIdx--) {
+      const segment = cuesBuffer[cueIdx];
       if (time < segment.end && time >= segment.start) {
         const cues = segment.cues;
         for (let j = 0; j < cues.length; j++) {
@@ -92,10 +92,10 @@ export default class TextTrackCuesStore {
 
     const to = Math.max(from, _to);
     const cuesBuffer = this._cuesBuffer;
-    for (let i = 0; i < cuesBuffer.length; i++) {
-      if (cuesBuffer[i].end > from) {
+    for (let cueIdx = 0; cueIdx < cuesBuffer.length; cueIdx++) {
+      if (cuesBuffer[cueIdx].end > from) {
         // this cuesInfos is concerned by the remove
-        const startCuesInfos = cuesBuffer[i];
+        const startCuesInfos = cuesBuffer[cueIdx];
         if (startCuesInfos.start >= to) {
           // our cuesInfos is strictly after this interval, we have nothing to do
           return;
@@ -112,8 +112,8 @@ export default class TextTrackCuesStore {
                     cuesInfos2 ] = removeCuesInfosBetween(startCuesInfos,
                                                           from,
                                                           to);
-            this._cuesBuffer[i] = cuesInfos1;
-            cuesBuffer.splice(i + 1, 0, cuesInfos2);
+            this._cuesBuffer[cueIdx] = cuesInfos1;
+            cuesBuffer.splice(cueIdx + 1, 0, cuesInfos2);
           }
           // No cuesInfos can be concerned after this one, we can quit
           return;
@@ -122,8 +122,8 @@ export default class TextTrackCuesStore {
         // Else remove all part after `from`
         if (startCuesInfos.start >= from) {
           // all the segment is concerned
-          cuesBuffer.splice(i, 1);
-          i--; // one less element, we have to decrement the loop
+          cuesBuffer.splice(cueIdx, 1);
+          cueIdx--; // one less element, we have to decrement the loop
         } else {
           // only the end is concerned
           startCuesInfos.cues = getCuesBefore(startCuesInfos.cues, from);
@@ -207,8 +207,8 @@ export default class TextTrackCuesStore {
       }
     }
 
-    for (let i = 0; i < cuesBuffer.length; i++) {
-      let cuesInfos = cuesBuffer[i];
+    for (let cueIdx = 0; cueIdx < cuesBuffer.length; cueIdx++) {
+      let cuesInfos = cuesBuffer[cueIdx];
       if (start < cuesInfos.end) {
         if (areNearlyEqual(start, cuesInfos.start)) {
           if (areNearlyEqual(end, cuesInfos.end)) {
@@ -218,7 +218,7 @@ export default class TextTrackCuesStore {
             //   Result:          |AAAAA|
             // Which means:
             //   1. replace the current cue with ours
-            cuesBuffer[i] = cuesInfosToInsert;
+            cuesBuffer[cueIdx] = cuesInfosToInsert;
             return;
           } else if (end < cuesInfos.end) {
             // our cue overlaps with the current one:
@@ -231,7 +231,7 @@ export default class TextTrackCuesStore {
             //   3. add ours before the current one
             cuesInfos.cues = getCuesAfter(cuesInfos.cues, end);
             cuesInfos.start = end;
-            cuesBuffer.splice(i, 0, cuesInfosToInsert);
+            cuesBuffer.splice(cueIdx, 0, cuesInfosToInsert);
             return;
           }
 
@@ -242,10 +242,10 @@ export default class TextTrackCuesStore {
           // Here we have to delete any cuesInfos which end before ours end,
           // and see about the following one.
           do {
-            cuesBuffer.splice(i, 1);
-            cuesInfos = cuesBuffer[i];
+            cuesBuffer.splice(cueIdx, 1);
+            cuesInfos = cuesBuffer[cueIdx];
           } while (cuesInfos !== undefined && end > cuesInfos.end);
-          onIndexOfNextCueFound(i);
+          onIndexOfNextCueFound(cueIdx);
           return;
         } else if (start < cuesInfos.start) {
           if (end < cuesInfos.start) {
@@ -255,7 +255,7 @@ export default class TextTrackCuesStore {
             //   Result:          |AAAAAAA| |BBBB|
             // Which means:
             //   - add ours before the current one
-            cuesBuffer.splice(i, 0, cuesInfosToInsert);
+            cuesBuffer.splice(cueIdx, 0, cuesInfosToInsert);
             return;
           } else if (areNearlyEqual(end, cuesInfos.start)) {
             // our cue goes just before the current one:
@@ -266,14 +266,14 @@ export default class TextTrackCuesStore {
             //   - update start time of the current one to be sure
             //   - add ours before the current one
             cuesInfos.start = end;
-            cuesBuffer.splice(i, 0, cuesInfosToInsert);
+            cuesBuffer.splice(cueIdx, 0, cuesInfosToInsert);
             return;
           } else if (areNearlyEqual(end, cuesInfos.end)) {
             //   ours:            |AAAAAAA|
             //   the current one:    |BBBB|
             //   Result:          |AAAAAAA|
             // Replace
-            cuesBuffer.splice(i, 1, cuesInfosToInsert);
+            cuesBuffer.splice(cueIdx, 1, cuesInfosToInsert);
             return;
           } else if (end < cuesInfos.end) {
             //   ours:            |AAAAAAA|
@@ -281,7 +281,7 @@ export default class TextTrackCuesStore {
             //   Result:          |AAAAAAABB|
             cuesInfos.cues = getCuesAfter(cuesInfos.cues, end);
             cuesInfos.start = end;
-            cuesBuffer.splice(i, 0, cuesInfosToInsert);
+            cuesBuffer.splice(cueIdx, 0, cuesInfosToInsert);
             return;
           }
 
@@ -289,10 +289,10 @@ export default class TextTrackCuesStore {
           //   the current one:   |BBB|...
           //   Result:          |AAAAAAA|...
           do {
-            cuesBuffer.splice(i, 1);
-            cuesInfos = cuesBuffer[i];
+            cuesBuffer.splice(cueIdx, 1);
+            cuesInfos = cuesBuffer[cueIdx];
           } while (cuesInfos !== undefined && end > cuesInfos.end);
-          onIndexOfNextCueFound(i);
+          onIndexOfNextCueFound(cueIdx);
           return;
         }
         // else -> start > cuesInfos.start
@@ -303,7 +303,7 @@ export default class TextTrackCuesStore {
           //   Result:          |BBAAAAAA|
           cuesInfos.cues = getCuesBefore(cuesInfos.cues, start);
           cuesInfos.end = start;
-          cuesBuffer.splice(i + 1, 0, cuesInfosToInsert);
+          cuesBuffer.splice(cueIdx + 1, 0, cuesInfosToInsert);
           return;
         } else if (cuesInfos.end > end) {
           //   ours:              |AAAAAA|
@@ -311,9 +311,9 @@ export default class TextTrackCuesStore {
           //   Result:          |BBAAAAAABBB|
           const [ cuesInfos1,
                   cuesInfos2 ] = removeCuesInfosBetween(cuesInfos, start, end);
-          this._cuesBuffer[i] = cuesInfos1;
-          cuesBuffer.splice(i + 1, 0, cuesInfosToInsert);
-          cuesBuffer.splice(i + 2, 0, cuesInfos2);
+          this._cuesBuffer[cueIdx] = cuesInfos1;
+          cuesBuffer.splice(cueIdx + 1, 0, cuesInfosToInsert);
+          cuesBuffer.splice(cueIdx + 2, 0, cuesInfos2);
           return;
         } else {
           //   ours:              |AAAAAA|
@@ -322,12 +322,13 @@ export default class TextTrackCuesStore {
           cuesInfos.cues = getCuesBefore(cuesInfos.cues, start);
           cuesInfos.end = start;
 
-          cuesInfos = cuesBuffer[i + 1];
+          const nextCueIdx = cueIdx + 1;
+          cuesInfos = cuesBuffer[nextCueIdx];
           while (cuesInfos !== undefined && end > cuesInfos.end) {
-            cuesBuffer.splice(i, 1);
-            cuesInfos = cuesBuffer[i];
+            cuesBuffer.splice(nextCueIdx, 1);
+            cuesInfos = cuesBuffer[nextCueIdx];
           }
-          onIndexOfNextCueFound(i);
+          onIndexOfNextCueFound(nextCueIdx);
           return;
         }
       }

package/src/core/segment_buffers/implementations/text/native/native_text_segment_buffer.ts

@@ -45,7 +45,7 @@ export default class NativeTextSegmentBuffer extends SegmentBuffer {
 
   private readonly _videoElement : HTMLMediaElement;
   private readonly _track : ICompatTextTrack;
-  private readonly _trackElement? : HTMLTrackElement;
+  private readonly _trackElement : HTMLTrackElement | undefined;
 
   private _buffered : ManualTimeRanges;
 
@@ -268,11 +268,11 @@ export interface INativeTextTracksBufferSegmentData {
    * This is mostly needed for "sami" subtitles, to know which cues can / should
    * be parsed.
    */
-  language? : string;
+  language? : string | undefined;
   /** start time from which the segment apply, in seconds. */
-  start? : number;
+  start? : number | undefined;
   /** end time until which the segment apply, in seconds. */
-  end? : number;
+  end? : number | undefined;
 }
 
 /**

package/src/core/segment_buffers/inventory/segment_inventory.ts

@@ -29,14 +29,23 @@ import BufferedHistory, {
 } from "./buffered_history";
 import { IChunkContext } from "./types";
 
-const { BUFFERED_HISTORY_RETENTION_TIME,
-        BUFFERED_HISTORY_MAXIMUM_ENTRIES,
-        MAX_MANIFEST_BUFFERED_START_END_DIFFERENCE,
-        MAX_MANIFEST_BUFFERED_DURATION_DIFFERENCE,
-        MINIMUM_SEGMENT_SIZE } = config;
 
 /** Information stored on a single chunk by the SegmentInventory. */
 export interface IBufferedChunk {
+  /**
+   * Complete size of the pushed chunk, in bytes.
+   * Note that this does not always reflect the memory imprint of the segment in
+   * memory:
+   *
+   *   1. It's the size of the original container file. A browser receiving that
+   *      segment might then transform it under another form that may be more or
+   *      less voluminous.
+   *
+   *   2. It's the size of the full chunk. In some scenarios only a sub-part of
+   *      that chunk is actually considered (examples: when using append
+   *      windows, when another chunk overlap that one etc.).
+   */
+  chunkSize : number | undefined;
   /**
    * Last inferred end in the media buffer this chunk ends at, in seconds.
    *
@@ -114,6 +123,8 @@ export interface IInsertedChunkInfos {
   representation : Representation;
   /** The Segment that chunk is linked to. */
   segment : ISegment;
+  /** Estimated size of the full pushed chunk, in bytes. */
+  chunkSize : number | undefined;
   /**
    * Start time, in seconds, this chunk most probably begins from after being
    * pushed.
@@ -152,6 +163,8 @@ export default class SegmentInventory {
   private _bufferedHistory : BufferedHistory;
 
   constructor() {
+    const { BUFFERED_HISTORY_RETENTION_TIME,
+            BUFFERED_HISTORY_MAXIMUM_ENTRIES } = config.getCurrent();
     this._inventory = [];
     this._bufferedHistory = new BufferedHistory(BUFFERED_HISTORY_RETENTION_TIME,
                                                 BUFFERED_HISTORY_MAXIMUM_ENTRIES);
@@ -180,7 +193,7 @@ export default class SegmentInventory {
     const inventory = this._inventory;
     let inventoryIndex = 0; // Current index considered.
     let thisSegment = inventory[0]; // Current segmentInfos considered
-
+    const { MINIMUM_SEGMENT_SIZE } = config.getCurrent();
     /** Type of buffer considered, used for logs */
     const bufferType : string | undefined = thisSegment?.infos.adaptation.type;
 
@@ -325,6 +338,7 @@ export default class SegmentInventory {
       adaptation,
       representation,
       segment,
+      chunkSize,
       start,
       end } : IInsertedChunkInfos
   ) : void {
@@ -341,6 +355,7 @@ export default class SegmentInventory {
 
     const inventory = this._inventory;
     const newSegment = { partiallyPushed: true,
+                         chunkSize,
                          splitted: false,
                          start,
                          end,
@@ -545,9 +560,16 @@ export default class SegmentInventory {
               //  prevSegment  : |---------|
               //  newSegment   :    |====|
               //  ===>         : |--|====|-|
-              log.debug("SI: Segment pushed is contained in a previous one",
-                        bufferType, start, end, segmentI.start, segmentI.end);
+              log.warn("SI: Segment pushed is contained in a previous one",
+                       bufferType, start, end, segmentI.start, segmentI.end);
               const nextSegment = { partiallyPushed: segmentI.partiallyPushed,
+                                    /**
+                                     * Note: this sadly means we're doing as if
+                                     * that chunk is present two times.
+                                     * Thankfully, this scenario should be
+                                     * fairly rare.
+                                     */
+                                    chunkSize: segmentI.chunkSize,
                                     splitted: true,
                                     start: newSegment.end,
                                     end: segmentI.end,
@@ -695,10 +717,15 @@ export default class SegmentInventory {
         }
 
         const firstI = i;
+        let segmentSize = inventory[i].chunkSize;
         i += 1;
         while (i < inventory.length &&
                areSameContent(inventory[i].infos, content))
         {
+          const chunkSize = inventory[i].chunkSize;
+          if (segmentSize !== undefined && chunkSize !== undefined) {
+            segmentSize += chunkSize;
+          }
           i++;
         }
 
@@ -711,6 +738,7 @@ export default class SegmentInventory {
           i -= length;
         }
         this._inventory[firstI].partiallyPushed = false;
+        this._inventory[firstI].chunkSize = segmentSize;
         this._inventory[firstI].end = lastEnd;
         this._inventory[firstI].bufferedEnd = lastBufferedEnd;
         this._inventory[firstI].splitted = splitted;
@@ -780,6 +808,8 @@ function bufferedStartLooksCoherent(
   }
   const { start, end } = thisSegment;
   const duration = end - start;
+  const { MAX_MANIFEST_BUFFERED_START_END_DIFFERENCE,
+          MAX_MANIFEST_BUFFERED_DURATION_DIFFERENCE } = config.getCurrent();
   return Math.abs(start - thisSegment.bufferedStart) <=
            MAX_MANIFEST_BUFFERED_START_END_DIFFERENCE &&
          (thisSegment.bufferedEnd === undefined ||
@@ -805,6 +835,8 @@ function bufferedEndLooksCoherent(
   }
   const { start, end } = thisSegment;
   const duration = end - start;
+  const { MAX_MANIFEST_BUFFERED_START_END_DIFFERENCE,
+          MAX_MANIFEST_BUFFERED_DURATION_DIFFERENCE } = config.getCurrent();
   return Math.abs(end - thisSegment.bufferedEnd) <=
            MAX_MANIFEST_BUFFERED_START_END_DIFFERENCE &&
          thisSegment.bufferedStart != null &&
@@ -828,6 +860,7 @@ function guessBufferedStartFromRangeStart(
                             null,
   bufferType : string
 ) : void {
+  const { MAX_MANIFEST_BUFFERED_START_END_DIFFERENCE } = config.getCurrent();
   if (firstSegmentInRange.bufferedStart !== undefined) {
     if (firstSegmentInRange.bufferedStart < rangeStart) {
       log.debug("SI: Segment partially GCed at the start",
@@ -892,6 +925,7 @@ function guessBufferedEndFromRangeEnd(
   rangeEnd : number,
   bufferType? : string
 ) : void {
+  const { MAX_MANIFEST_BUFFERED_START_END_DIFFERENCE } = config.getCurrent();
   if (lastSegmentInRange.bufferedEnd !== undefined) {
     if (lastSegmentInRange.bufferedEnd > rangeEnd) {
       log.debug("SI: Segment partially GCed at the end",

package/src/core/stream/adaptation/adaptation_stream.ts

@@ -76,7 +76,6 @@ import {
 } from "../types";
 import createRepresentationEstimator from "./create_representation_estimator";
 
-const { DELTA_POSITION_AFTER_RELOAD } = config;
 
 /** Regular playback information needed by the AdaptationStream. */
 export interface IAdaptationStreamPlaybackObservation extends
@@ -123,6 +122,7 @@ export interface IAdaptationStreamArguments {
    * this AdaptationStream won't try to download new segments.
    */
   wantedBufferAhead : IReadOnlySharedReference<number>;
+  maxVideoBufferSize : IReadOnlySharedReference<number>;
 }
 
 /**
@@ -189,6 +189,7 @@ export default function AdaptationStream({
   segmentBuffer,
   segmentFetcherCreator,
   wantedBufferAhead,
+  maxVideoBufferSize,
 } : IAdaptationStreamArguments) : Observable<IAdaptationStreamEvent> {
   const directManualBitrateSwitching = options.manualBitrateSwitchingMode === "direct";
   const { manifest, period, adaptation } = content;
@@ -280,6 +281,7 @@ export default function AdaptationStream({
         fromEstimate.manual &&
         !isFirstEstimate)
     {
+      const { DELTA_POSITION_AFTER_RELOAD } = config.getCurrent();
       return reloadAfterSwitch(period,
                                adaptation.type,
                                playbackObserver,
@@ -375,6 +377,9 @@ export default function AdaptationStream({
       const bufferGoal$ = wantedBufferAhead.asObservable().pipe(
         map((wba) => wba * bufferGoalRatio)
       );
+      // eslint-disable-next-line max-len
+      const maxBufferSize$ = adaptation.type === "video" ? maxVideoBufferSize.asObservable() :
+                                                           observableOf(Infinity);
 
       log.info("Stream: changing representation",
                adaptation.type,
@@ -389,6 +394,7 @@ export default function AdaptationStream({
                                     segmentFetcher,
                                     terminate$: terminateCurrentStream$,
                                     options: { bufferGoal$,
+                                               maxBufferSize$,
                                                drmSystemId: options.drmSystemId,
                                                fastSwitchThreshold$ } })
         .pipe(catchError((err : unknown) => {

package/src/core/stream/events_generators.ts

@@ -16,13 +16,14 @@
 
 import { Subject } from "rxjs";
 import { ICustomError } from "../../errors";
-import {
+import Manifest, {
   Adaptation,
   ISegment,
   Period,
   Representation,
 } from "../../manifest";
-import { IContentProtection } from "../eme";
+import { IRepresentationProtectionData } from "../../manifest/representation";
+import objectAssign from "../../utils/object_assign";
 import { IBufferType } from "../segment_buffers";
 import {
   IActivePeriodChangedEvent,
@@ -169,10 +170,14 @@ const EVENTS = {
   },
 
   encryptionDataEncountered(
-    initDataInfo : IContentProtection
+    reprProtData : IRepresentationProtectionData,
+    content : { manifest : Manifest;
+                period : Period;
+                adaptation : Adaptation;
+                representation : Representation; }
   ) : IEncryptionDataEncounteredEvent {
     return { type: "encryption-data-encountered",
-             value: initDataInfo };
+             value: objectAssign({ content }, reprProtData) };
   },
 
   representationChange(

package/src/core/stream/orchestrator/stream_orchestrator.ts

@@ -78,13 +78,12 @@ import getBlacklistedRanges from "./get_blacklisted_ranges";
 
 export type IStreamOrchestratorPlaybackObservation = IPeriodStreamPlaybackObservation;
 
-const { MAXIMUM_MAX_BUFFER_AHEAD,
-        MAXIMUM_MAX_BUFFER_BEHIND } = config;
 
 /** Options tweaking the behavior of the StreamOrchestrator. */
 export type IStreamOrchestratorOptions =
   IPeriodStreamOptions &
   { wantedBufferAhead : IReadOnlySharedReference<number>;
+    maxVideoBufferSize : IReadOnlySharedReference<number>;
     maxBufferAhead : IReadOnlySharedReference<number>;
     maxBufferBehind : IReadOnlySharedReference<number>; };
 
@@ -123,8 +122,13 @@ export default function StreamOrchestrator(
   options: IStreamOrchestratorOptions
 ) : Observable<IStreamOrchestratorEvent> {
   const { manifest, initialPeriod } = content;
-  const { maxBufferAhead, maxBufferBehind, wantedBufferAhead } = options;
+  const { maxBufferAhead,
+          maxBufferBehind,
+          wantedBufferAhead,
+          maxVideoBufferSize } = options;
 
+  const { MAXIMUM_MAX_BUFFER_AHEAD,
+          MAXIMUM_MAX_BUFFER_BEHIND } = config.getCurrent();
   // Keep track of a unique BufferGarbageCollector created per
   // SegmentBuffer.
   const garbageCollectors =
@@ -452,7 +456,8 @@ export default function StreamOrchestrator(
                                          segmentBuffersStore,
                                          options,
                                          playbackObserver,
-                                         wantedBufferAhead }
+                                         wantedBufferAhead,
+                                         maxVideoBufferSize }
     ).pipe(
       mergeMap((evt : IPeriodStreamEvent) : Observable<IMultiplePeriodStreamsEvent> => {
         if (evt.type === "stream-status") {

package/src/core/stream/period/create_empty_adaptation_stream.ts

@@ -48,7 +48,8 @@ export default function createEmptyAdaptationStream(
   let hasFinishedLoading = false;
   const wantedBufferAhead$ = wantedBufferAhead.asObservable();
   const observation$ = playbackObserver.observe(true);
-  return observableCombineLatest([observation$, wantedBufferAhead$]).pipe(
+  return observableCombineLatest([observation$,
+                                  wantedBufferAhead$]).pipe(
     mergeMap(([observation, wba]) => {
       const { position } = observation;
       if (period.end !== undefined && position + wba >= period.end) {

package/src/core/stream/period/get_adaptation_switch_strategy.ts

@@ -33,7 +33,6 @@ import {
   SegmentBuffer,
 } from "../../segment_buffers";
 
-const { ADAPTATION_SWITCH_BUFFER_PADDINGS } = config;
 
 export type IAdaptationSwitchStrategy =
   { type: "continue"; value: undefined } |
@@ -157,6 +156,7 @@ export default function getAdaptationSwitchStrategy(
 
   // Next, exclude data around current position to avoid decoding issues
   const bufferType = adaptation.type;
+  const { ADAPTATION_SWITCH_BUFFER_PADDINGS } = config.getCurrent();
 
   /** Ranges that won't be cleaned from the current buffer. */
   let paddingBefore = ADAPTATION_SWITCH_BUFFER_PADDINGS[bufferType].before;

package/src/core/stream/period/period_stream.ts

@@ -61,7 +61,6 @@ import {
 import createEmptyStream from "./create_empty_adaptation_stream";
 import getAdaptationSwitchStrategy from "./get_adaptation_switch_strategy";
 
-const { DELTA_POSITION_AFTER_RELOAD } = config;
 
 /** Playback observation required by the `PeriodStream`. */
 export interface IPeriodStreamPlaybackObservation {
@@ -99,6 +98,7 @@ export interface IPeriodStreamArguments {
   playbackObserver : IReadOnlyPlaybackObserver<IPeriodStreamPlaybackObservation>;
   options: IPeriodStreamOptions;
   wantedBufferAhead : IReadOnlySharedReference<number>;
+  maxVideoBufferSize : IReadOnlySharedReference<number>;
 }
 
 /** Options tweaking the behavior of the PeriodStream. */
@@ -143,6 +143,7 @@ export default function PeriodStream({
   segmentBuffersStore,
   options,
   wantedBufferAhead,
+  maxVideoBufferSize,
 } : IPeriodStreamArguments) : Observable<IPeriodStreamEvent> {
   const { period } = content;
 
@@ -161,6 +162,7 @@ export default function PeriodStream({
        * This value contains this relative position, in seconds.
        * @see reloadAfterSwitch
        */
+      const { DELTA_POSITION_AFTER_RELOAD } = config.getCurrent();
       const relativePosAfterSwitch =
         switchNb === 0 ? 0 :
         bufferType === "audio" ? DELTA_POSITION_AFTER_RELOAD.trackSwitch.audio :
@@ -279,7 +281,8 @@ export default function PeriodStream({
                               playbackObserver: adaptationPlaybackObserver,
                               segmentBuffer,
                               segmentFetcherCreator,
-                              wantedBufferAhead }).pipe(
+                              wantedBufferAhead,
+                              maxVideoBufferSize }).pipe(
       catchError((error : unknown) => {
         // Stream linked to a non-native media buffer should not impact the
         // stability of the player. ie: if a text buffer sends an error, we want

package/src/core/stream/representation/force_garbage_collection.ts

@@ -25,8 +25,6 @@ import log from "../../../log";
 import { getInnerAndOuterTimeRanges } from "../../../utils/ranges";
 import { SegmentBuffer } from "../../segment_buffers";
 
-const GC_GAP_CALM = config.BUFFER_GC_GAPS.CALM;
-const GC_GAP_BEEFY = config.BUFFER_GC_GAPS.BEEFY;
 
 /**
  * Run the garbage collector.
@@ -43,6 +41,8 @@ export default function forceGarbageCollection(
   bufferingQueue : SegmentBuffer
 ) : Observable<unknown> {
   return observableDefer(() => {
+    const GC_GAP_CALM = config.getCurrent().BUFFER_GC_GAPS.CALM;
+    const GC_GAP_BEEFY = config.getCurrent().BUFFER_GC_GAPS.BEEFY;
     log.warn("Stream: Running garbage collector");
     const buffered = bufferingQueue.getBufferedRanges();
     let cleanedupRanges = selectGCedRanges(currentPosition, buffered, GC_GAP_CALM);
@@ -84,19 +84,17 @@ function selectGCedRanges(
 
   // start by trying to remove all ranges that do not contain the
   // current time and respect the gcGap
-  // respect the gcGap? FIXME?
   for (let i = 0; i < outerRanges.length; i++) {
     const outerRange = outerRanges[i];
-    if (position - gcGap < outerRange.end) {
+    if (position - gcGap > outerRange.end) {
       cleanedupRanges.push(outerRange);
-    }
-    else if (position + gcGap > outerRange.start) {
+    } else if (position + gcGap < outerRange.start) {
       cleanedupRanges.push(outerRange);
     }
   }
 
   // try to clean up some space in the current range
-  if (innerRange != null) {
+  if (innerRange !== null) {
     log.debug("Stream: GC removing part of inner range", cleanedupRanges);
     if (position - gcGap > innerRange.start) {
       cleanedupRanges.push({ start: innerRange.start,