wip

src/core/api/public_api.ts

@@ -1992,7 +1992,7 @@ class Player extends EventEmitter<IPublicAPIEvent> {
     const segmentBufferStatus = this._priv_contentInfos
       .segmentBuffersStore.getStatus(bufferType);
     return segmentBufferStatus.type === "initialized" ?
-      segmentBufferStatus.value.getInventory() :
+      segmentBufferStatus.value.getLastKnownInventory() :
       null;
   }
 

src/core/init/utils/rebuffering_controller.ts

@@ -439,7 +439,7 @@ export default class RebufferingController
       let isClear = true;
       for (const status of [statusAudio, statusVideo]) {
         if (status.type === "initialized") {
-          for (const segment of status.value.getInventory()) {
+          for (const segment of status.value.getLastKnownInventory()) {
             const { representation } = segment.infos;
             if (representation.decipherable === false) {
               log.warn(

src/core/segment_buffers/implementations/types.ts

@@ -157,14 +157,18 @@ export abstract class SegmentBuffer {
   ) : Promise<void>;
 
   /**
-   * Returns the currently buffered data for which the content is known with
-   * the corresponding content information.
-   * /!\ This data can fall out of sync with the real buffered ranges. Please
-   * call `synchronizeInventory` before to make sure it is correctly
-   * synchronized.
+   * Returns an inventory of the last known segments to be currently contained in
+   * the SegmentBuffer.
+   *
+   * /!\ Note that this data may not be up-to-date with the real current content
+   * of the SegmentBuffer.
+   * Generally speaking, pushed segments are added right away to it but segments
+   * may have been since removed, which might not be known right away.
+   * Please consider this when using this method, by considering that it does
+   * not reflect the full reality of the underlying buffer.
    * @returns {Array.<Object>}
    */
-  public getInventory() : IBufferedChunk[] {
+  public getLastKnownInventory() : IBufferedChunk[] {
     // The default implementation just use the SegmentInventory
     return this._segmentInventory.getInventory();
   }

src/core/stream/adaptation/get_representations_switch_strategy.ts

@@ -60,7 +60,7 @@ export default function getRepresentationsSwitchingStrategy(
     return { type: "continue", value: undefined };
   }
 
-  const inventory = segmentBuffer.getInventory();
+  const inventory = segmentBuffer.getLastKnownInventory();
 
   /** Data already in the right Adaptation */
   const rangesWithReps =

src/core/stream/orchestrator/get_time_ranges_for_content.ts

@@ -39,9 +39,8 @@ export default function getTimeRangesForContent(
   if (contents.length === 0) {
     return [];
   }
-  // segmentBuffer.synchronizeInventory();
   const accumulator : IRange[] = [];
-  const inventory = segmentBuffer.getInventory();
+  const inventory = segmentBuffer.getLastKnownInventory();
 
   for (let i = 0; i < inventory.length; i++) {
     const chunk = inventory[i];

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

@@ -84,7 +84,7 @@ export default function getAdaptationSwitchStrategy(
     return { type: "continue", value: undefined };
   }
 
-  const inventory = segmentBuffer.getInventory();
+  const inventory = segmentBuffer.getLastKnownInventory();
 
   // Continue if we have no other Adaptation buffered in the current Period
   if (!inventory.some(buf => buf.infos.period.id === period.id &&

src/core/stream/representation/utils/get_buffer_status.ts

@@ -129,7 +129,7 @@ export default function getBufferStatus(
   const bufferedSegments =
     getPlayableBufferedSegments({ start: Math.max(neededRange.start - 0.5, 0),
                                   end: neededRange.end + 0.5 },
-                                segmentBuffer.getInventory());
+                                segmentBuffer.getLastKnownInventory());
   let currentPlaybackTime = playbackObserver.getCurrentTime();
   if (currentPlaybackTime === undefined) {
     // We're in a WebWorker, just consider the last known position