hls.js 1.6.0-beta.1.0.canary.10811 → 1.6.0-beta.1.0.canary.10813

package/src/controller/interstitials-controller.ts

@@ -1151,9 +1151,6 @@ MediaSource ${JSON.stringify(attachMediaSourceData)} from ${logFromSource}`,
     this.waitingItem = null;
 
     this.bufferedToItem(scheduledItem);
-    if (!fromItem) {
-      return;
-    }
 
     this.log(`resuming ${segmentToString(scheduledItem)}`);
 
@@ -1169,6 +1166,10 @@ MediaSource ${JSON.stringify(attachMediaSourceData)} from ${logFromSource}`,
       this.attachPrimary(timelinePos, scheduledItem);
     }
 
+    if (!fromItem) {
+      return;
+    }
+
     const scheduleItems = this.schedule.items;
     if (!scheduleItems) {
       return;
@@ -1434,14 +1435,6 @@ Schedule: ${scheduleItems.map((seg) => segmentToString(seg))}`,
     if (removedIds.length) {
       this.log(`Removed events ${removedIds}`);
     }
-    if (
-      this.isInterstitial(playingItem) &&
-      removedIds.includes(playingItem.event.identifier)
-    ) {
-      this.warn(
-        `Interstitial "${playingItem.event.identifier}" removed while playing`,
-      );
-    }
 
     this.playerQueue.forEach((player) => {
       if (player.interstitial.appendInPlace) {
@@ -1474,7 +1467,7 @@ Schedule: ${scheduleItems.map((seg) => segmentToString(seg))}`,
       } else if (!this.updateItem(bufferingItem)) {
         // Interstitial removed from schedule (Live -> VOD or other scenario where Start Date is outside the range of VOD Playlist)
         this.bufferingItem = null;
-        this.clearInterstitial(bufferingItem.event);
+        this.clearInterstitial(bufferingItem.event, null);
       }
     }
     // Clear waitingItem if it has been removed from the schedule
@@ -1482,7 +1475,7 @@ Schedule: ${scheduleItems.map((seg) => segmentToString(seg))}`,
 
     removedInterstitials.forEach((interstitial) => {
       interstitial.assetList.forEach((asset) => {
-        this.clearAssetPlayer(asset.identifier);
+        this.clearAssetPlayer(asset.identifier, null);
       });
     });
 
@@ -1494,6 +1487,17 @@ Schedule: ${scheduleItems.map((seg) => segmentToString(seg))}`,
         removedIds,
       });
 
+      if (
+        this.isInterstitial(playingItem) &&
+        removedIds.includes(playingItem.event.identifier)
+      ) {
+        this.warn(
+          `Interstitial "${playingItem.event.identifier}" removed while playing`,
+        );
+        this.primaryFallback(playingItem.event);
+        return;
+      }
+
       // Check is buffered to new Interstitial event boundary
       // (Live update publishes Interstitial with new segment)
       this.checkBuffer();
@@ -2065,7 +2069,7 @@ Schedule: ${scheduleItems.map((seg) => segmentToString(seg))}`,
 
   private clearInterstitial(
     interstitial: InterstitialEvent,
-    toSegment?: InterstitialScheduleItem | null,
+    toSegment: InterstitialScheduleItem | null,
   ) {
     interstitial.assetList.forEach((asset) => {
       this.clearAssetPlayer(asset.identifier, toSegment);
@@ -2075,7 +2079,7 @@ Schedule: ${scheduleItems.map((seg) => segmentToString(seg))}`,
 
   private clearAssetPlayer(
     assetId: InterstitialAssetId,
-    toSegment?: InterstitialScheduleItem | null,
+    toSegment: InterstitialScheduleItem | null,
   ) {
     if (toSegment === null) {
       return;
@@ -2228,7 +2232,7 @@ Schedule: ${scheduleItems.map((seg) => segmentToString(seg))}`,
     const error = new Error(errorMessage);
     if (assetItem) {
       if (this.playingAsset !== assetItem) {
-        this.clearAssetPlayer(assetItem.identifier);
+        this.clearAssetPlayer(assetItem.identifier, null);
       }
       assetItem.error = error;
     }
@@ -2251,6 +2255,13 @@ Schedule: ${scheduleItems.map((seg) => segmentToString(seg))}`,
     // Update schedule now that interstitial/assets are flagged with `error` for fallback
     this.updateSchedule();
     if (playingItem) {
+      this.log(
+        `Fallback to primary from event "${interstitial.identifier}" start: ${
+          flushStart
+        } pos: ${this.timelinePos} playing: ${
+          playingItem ? segmentToString(playingItem) : '<none>'
+        } error: ${interstitial.error}`,
+      );
       if (interstitial.appendInPlace) {
         interstitial.appendInPlace = false;
         this.attachPrimary(flushStart, null);

package/src/hls.ts

@@ -300,6 +300,12 @@ export default class Hls implements HlsEventEmitter {
     if (typeof onErrorOut === 'function') {
       this.on(Events.ERROR, onErrorOut, errorController);
     }
+    // Autostart load handler
+    this.on(
+      Events.MANIFEST_LOADED,
+      playListLoader.onManifestLoaded,
+      playListLoader,
+    );
   }
 
   createController(ControllerClass, components) {

package/src/loader/playlist-loader.ts

@@ -11,6 +11,7 @@ import { ErrorDetails, ErrorTypes } from '../errors';
 import { Events } from '../events';
 import { PlaylistContextType, PlaylistLevelType } from '../types/loader';
 import { AttrList } from '../utils/attr-list';
+import { computeReloadInterval } from '../utils/level-helper';
 import type { LevelDetails } from './level-details';
 import type { LoaderConfig, RetryConfig } from '../config';
 import type Hls from '../hls';
@@ -69,6 +70,7 @@ class PlaylistLoader implements NetworkComponentAPI {
     [key: string]: Loader<LoaderContext>;
   } = Object.create(null);
   private variableList: VariableMap | null = null;
+  public onManifestLoaded = this.checkAutostartLoad;
 
   constructor(hls: Hls) {
     this.hls = hls;
@@ -495,8 +497,6 @@ class PlaylistLoader implements NetworkComponentAPI {
       startTimeOffset,
       variableList,
     });
-
-    this.checkAutostartLoad();
   }
 
   private handleTrackOrLevelPlaylist(
@@ -537,6 +537,8 @@ class PlaylistLoader implements NetworkComponentAPI {
         name: '',
         url,
       };
+      levelDetails.requestScheduled =
+        stats.loading.start + computeReloadInterval(levelDetails, 0);
 
       hls.trigger(Events.MANIFEST_LOADED, {
         levels: [singleLevel],
@@ -566,14 +568,6 @@ class PlaylistLoader implements NetworkComponentAPI {
       networkDetails,
       loader,
     );
-
-    if (
-      type === PlaylistContextType.MANIFEST &&
-      (!levelDetails.playlistParsingError ||
-        (!levelDetails.fragments.length && levelDetails.live))
-    ) {
-      this.checkAutostartLoad();
-    }
   }
 
   private handleManifestParsingError(

package/src/types/events.ts

@@ -7,8 +7,6 @@ import type {
   SourceBufferName,
   SourceBufferTrackSet,
 } from './buffer';
-import type { ChunkMetadata } from './transmuxer';
-import type { ErrorDetails, ErrorTypes } from '../errors';
 import type { MetadataSample, UserdataSample } from './demuxer';
 import type {
   HdcpLevel,
@@ -27,6 +25,7 @@ import type {
   PlaylistLoaderContext,
 } from './loader';
 import type { MediaPlaylist, MediaPlaylistType } from './media-playlist';
+import type { ChunkMetadata } from './transmuxer';
 import type { SteeringManifest } from '../controller/content-steering-controller';
 import type { IErrorAction } from '../controller/error-controller';
 import type { HlsAssetPlayer } from '../controller/interstitial-player';
@@ -34,6 +33,7 @@ import type {
   InterstitialScheduleDurations,
   InterstitialScheduleItem,
 } from '../controller/interstitials-schedule';
+import type { ErrorDetails, ErrorTypes } from '../errors';
 import type { HlsListeners } from '../events';
 import type { Fragment, MediaFragment, Part } from '../loader/fragment';
 import type {