rx-player 4.3.0-dev.2025011000 → 4.3.0-dev.2025012900

package/src/core/main/worker/worker_main.ts

@@ -594,7 +594,7 @@ function loadOrReloadPreparedContent(
   );
 
   StreamOrchestrator(
-    { initialPeriod: manifest.periods[0], manifest },
+    { initialPeriod, manifest },
     playbackObserver,
     representationEstimator,
     segmentSinksStore,

package/src/main_thread/api/public_api.ts

@@ -411,7 +411,7 @@ class Player extends EventEmitter<IPublicAPIEvent> {
     // See: https://bugzilla.mozilla.org/show_bug.cgi?id=1194624
     videoElement.preload = "auto";
 
-    this.version = /* PLAYER_VERSION */ "4.3.0-dev.2025011000";
+    this.version = /* PLAYER_VERSION */ "4.3.0-dev.2025012900";
     this.log = log;
     this.state = "STOPPED";
     this.videoElement = videoElement;
@@ -3330,7 +3330,7 @@ class Player extends EventEmitter<IPublicAPIEvent> {
     }
   }
 }
-Player.version = /* PLAYER_VERSION */ "4.3.0-dev.2025011000";
+Player.version = /* PLAYER_VERSION */ "4.3.0-dev.2025012900";
 
 /** Every events sent by the RxPlayer's public API. */
 interface IPublicAPIEvent {

package/src/main_thread/init/utils/rebuffering_controller.ts

@@ -171,15 +171,16 @@ export default class RebufferingController extends EventEmitter<IRebufferingCont
         if (position.isAwaitingFuturePosition()) {
           playbackRateUpdater.stopRebuffering();
           log.debug("Init: let rebuffering happen as we're awaiting a future position");
-          this.trigger("stalled", stalledReason);
-          return;
+        } else {
+          playbackRateUpdater.startRebuffering();
         }
 
-        playbackRateUpdater.startRebuffering();
-
         if (
           this._manifest === null ||
           (isSeekingApproximate &&
+            // Don't handle discontinuities on devices with broken seeks before
+            // enough time have passed because seeking brings more risks to
+            // lead to a lengthy rebuffering-exiting process
             getMonotonicTimeStamp() - rebuffering.timestamp <= 1000)
         ) {
           this.trigger("stalled", stalledReason);
@@ -189,6 +190,16 @@ export default class RebufferingController extends EventEmitter<IRebufferingCont
         /** Position at which data is awaited. */
         const { position: stalledPosition } = rebuffering;
 
+        /**
+         * We may still be in the process of waiting for a position to be seeked
+         * to. When calculating a potential position to e.g. skip over
+         * discontinuities, we should compare it to that "target" position if
+         * one, not the one we're currently playing.
+         */
+        const targetTime = observation.position.isAwaitingFuturePosition()
+          ? observation.position.getWanted()
+          : this._playbackObserver.getCurrentTime();
+
         if (
           stalledPosition !== null &&
           stalledPosition !== undefined &&
@@ -201,10 +212,10 @@ export default class RebufferingController extends EventEmitter<IRebufferingCont
           );
           if (skippableDiscontinuity !== null) {
             const realSeekTime = skippableDiscontinuity + 0.001;
-            if (realSeekTime <= this._playbackObserver.getCurrentTime()) {
+            if (realSeekTime <= targetTime) {
               log.info(
                 "Init: position to seek already reached, no seeking",
-                this._playbackObserver.getCurrentTime(),
+                targetTime,
                 realSeekTime,
               );
             } else {
@@ -243,7 +254,7 @@ export default class RebufferingController extends EventEmitter<IRebufferingCont
           nextBufferRangeGap < BUFFER_DISCONTINUITY_THRESHOLD
         ) {
           const seekTo = positionBlockedAt + nextBufferRangeGap + EPSILON;
-          if (this._playbackObserver.getCurrentTime() < seekTo) {
+          if (targetTime < seekTo) {
             log.warn(
               "Init: discontinuity encountered inferior to the threshold",
               positionBlockedAt,
@@ -266,8 +277,7 @@ export default class RebufferingController extends EventEmitter<IRebufferingCont
           if (period.end !== undefined && period.end <= positionBlockedAt) {
             if (
               this._manifest.periods[i + 1].start > positionBlockedAt &&
-              this._manifest.periods[i + 1].start >
-                this._playbackObserver.getCurrentTime()
+              this._manifest.periods[i + 1].start > targetTime
             ) {
               const nextPeriod = this._manifest.periods[i + 1];
               this._playbackObserver.setCurrentTime(nextPeriod.start);

package/vitest.config.mjs

@@ -1,22 +1,4 @@
 import { defineConfig } from "vitest/config";
-import { promises as fs } from "fs";
-
-// https://github.com/tachibana-shin/vite-plugin-arraybuffer/blob/main/src/main.ts
-function vitePluginArraybuffer() {
-  return {
-    name: "array-buffer-loader",
-    async load(id) {
-      if (id.endsWith("?arraybuffer")) {
-        const filePath = id.replace(/\?arraybuffer$/, "");
-        const fileBuffer = await fs.readFile(filePath);
-        return {
-          code: `export default new Uint8Array([${new Uint8Array(fileBuffer).join(",")}]).buffer`,
-          map: { mappings: "" },
-        };
-      }
-    },
-  };
-}
 
 function getBrowserConfig(browser) {
   switch (browser) {
@@ -69,7 +51,6 @@ function getBrowserConfig(browser) {
 }
 
 export default defineConfig({
-  plugins: [vitePluginArraybuffer()],
   define: {
     // global variables
     __TEST_CONTENT_SERVER__: {