hls.js 1.6.0-beta.1.0.canary.10826 → 1.6.0-beta.2

package/package.json

@@ -130,5 +130,5 @@
     "url-toolkit": "2.2.5",
     "wrangler": "3.92.0"
   },
-  "version": "1.6.0-beta.1.0.canary.10826"
+  "version": "1.6.0-beta.2"
 }

package/src/controller/buffer-controller.ts

@@ -340,11 +340,8 @@ export default class BufferController extends Logger implements ComponentAPI {
       : null;
     const trackCount = trackNames ? trackNames.length : 0;
     const mediaSourceOpenCallback = () => {
-      if (this.media) {
-        const readyState = this.mediaSource?.readyState;
-        if (readyState === 'open' || readyState === 'ended') {
-          this._onMediaSourceOpen();
-        }
+      if (this.media && this.mediaSourceOpenOrEnded) {
+        this._onMediaSourceOpen();
       }
     };
     if (transferredTracks && trackNames && trackCount) {
@@ -429,6 +426,11 @@ transfer tracks: ${JSON.stringify(transferredTracks, (key, value) => (key === 'i
     }
   }
 
+  private get mediaSourceOpenOrEnded(): boolean {
+    const readyState = this.mediaSource?.readyState;
+    return readyState === 'open' || readyState === 'ended';
+  }
+
   private _onEndStreaming = (event) => {
     if (!this.hls) {
       return;
@@ -466,18 +468,23 @@ transfer tracks: ${JSON.stringify(transferredTracks, (key, value) => (key === 'i
         });
         this.resetQueue();
       } else {
-        if (mediaSource.readyState === 'open') {
+        if (this.mediaSourceOpenOrEnded) {
+          const open = mediaSource.readyState === 'open';
           try {
             const sourceBuffers = mediaSource.sourceBuffers;
             for (let i = sourceBuffers.length; i--; ) {
-              sourceBuffers[i].abort();
+              if (open) {
+                sourceBuffers[i].abort();
+              }
               mediaSource.removeSourceBuffer(sourceBuffers[i]);
             }
-            // endOfStream could trigger exception if any sourcebuffer is in updating state
-            // we don't really care about checking sourcebuffer state here,
-            // as we are anyway detaching the MediaSource
-            // let's just avoid this exception to propagate
-            mediaSource.endOfStream();
+            if (open) {
+              // endOfStream could trigger exception if any sourcebuffer is in updating state
+              // we don't really care about checking sourcebuffer state here,
+              // as we are anyway detaching the MediaSource
+              // let's just avoid this exception to propagate
+              mediaSource.endOfStream();
+            }
           } catch (err) {
             this.warn(
               `onMediaDetaching: ${err.message} while calling endOfStream`,
@@ -651,7 +658,7 @@ transfer tracks: ${JSON.stringify(transferredTracks, (key, value) => (key === 'i
     if (this.sourceBufferCount) {
       return;
     }
-    if (this.mediaSource !== null && this.mediaSource.readyState === 'open') {
+    if (this.mediaSourceOpenOrEnded) {
       this.checkPendingTracks();
     }
   }
@@ -892,7 +899,7 @@ transfer tracks: ${JSON.stringify(transferredTracks, (key, value) => (key === 'i
           event.details = ErrorDetails.BUFFER_FULL_ERROR;
         } else if (
           (error as DOMException).code === DOMException.INVALID_STATE_ERR &&
-          this.mediaSource?.readyState === 'open' &&
+          this.mediaSourceOpenOrEnded &&
           !this.media?.error
         ) {
           // Allow retry for "Failed to execute 'appendBuffer' on 'SourceBuffer': This SourceBuffer is still processing" errors

package/src/controller/interstitials-controller.ts

@@ -218,7 +218,7 @@ export default class InterstitialsController
     // @ts-ignore
     this.assetListLoader = null;
     // @ts-ignore
-    this.onPlay = this.onSeeking = this.onTimeupdate = null;
+    this.onPlay = this.onPause = this.onSeeking = this.onTimeupdate = null;
     // @ts-ignore
     this.onScheduleUpdate = null;
   }
@@ -232,6 +232,7 @@ export default class InterstitialsController
 
   private removeMediaListeners(media: HTMLMediaElement) {
     media.removeEventListener('play', this.onPlay);
+    media.removeEventListener('pause', this.onPause);
     media.removeEventListener('seeking', this.onSeeking);
     media.removeEventListener('timeupdate', this.onTimeupdate);
   }
@@ -245,6 +246,7 @@ export default class InterstitialsController
     media.addEventListener('seeking', this.onSeeking);
     media.addEventListener('timeupdate', this.onTimeupdate);
     media.addEventListener('play', this.onPlay);
+    media.addEventListener('pause', this.onPause);
   }
 
   private onMediaAttached(
@@ -763,6 +765,10 @@ MediaSource ${JSON.stringify(attachMediaSourceData)} from ${logFromSource}`,
     this.shouldPlay = true;
   };
 
+  private onPause = () => {
+    this.shouldPlay = false;
+  };
+
   private onSeeking = () => {
     const currentTime = this.currentTime;
     if (currentTime === undefined || this.playbackDisabled) {