hls.js 1.6.0-beta.2.0.canary.10869 → 1.6.0-beta.2.0.canary.10873

package/package.json

@@ -134,5 +134,5 @@
     "url-toolkit": "2.2.5",
     "wrangler": "3.99.0"
   },
-  "version": "1.6.0-beta.2.0.canary.10869"
+  "version": "1.6.0-beta.2.0.canary.10873"
 }

package/src/demux/video/hevc-video-parser.ts

@@ -122,8 +122,10 @@ class HevcVideoParser extends BaseVideoParser {
         case 32:
           push = true;
           if (!track.vps) {
-            const config = this.readVPS(unit.data);
-            track.params = { ...config };
+            if (typeof track.params !== 'object') {
+              track.params = {};
+            }
+            track.params = Object.assign(track.params, this.readVPS(unit.data));
             this.initVPS = unit.data;
           }
           track.vps = [unit.data];
@@ -133,31 +135,35 @@ class HevcVideoParser extends BaseVideoParser {
         case 33:
           push = true;
           spsfound = true;
-          if (typeof track.params === 'object') {
-            if (
-              track.vps !== undefined &&
-              track.vps[0] !== this.initVPS &&
-              track.sps !== undefined &&
-              !this.matchSPS(track.sps[0], unit.data)
-            ) {
-              this.initVPS = track.vps[0];
-              track.sps = track.pps = undefined;
-            }
-            if (!track.sps) {
-              const config = this.readSPS(unit.data);
-              track.width = config.width;
-              track.height = config.height;
-              track.pixelRatio = config.pixelRatio;
-              track.codec = config.codecString;
-              track.sps = [];
-              for (const prop in config.params) {
-                track.params[prop] = config.params[prop];
-              }
+          if (
+            track.vps !== undefined &&
+            track.vps[0] !== this.initVPS &&
+            track.sps !== undefined &&
+            !this.matchSPS(track.sps[0], unit.data)
+          ) {
+            this.initVPS = track.vps[0];
+            track.sps = track.pps = undefined;
+          }
+          if (!track.sps) {
+            const config = this.readSPS(unit.data);
+            track.width = config.width;
+            track.height = config.height;
+            track.pixelRatio = config.pixelRatio;
+            track.codec = config.codecString;
+            track.sps = [];
+            if (typeof track.params !== 'object') {
+              track.params = {};
             }
-            if (track.vps !== undefined && track.vps[0] === this.initVPS) {
-              track.sps.push(unit.data);
+            for (const prop in config.params) {
+              track.params[prop] = config.params[prop];
             }
           }
+          if (
+            (!track.vps && !track.sps.length) ||
+            (track.vps && track.vps[0] === this.initVPS)
+          ) {
+            track.sps.push(unit.data);
+          }
           if (!VideoSample) {
             VideoSample = this.VideoSample = this.createVideoSample(
               true,
@@ -179,7 +185,10 @@ class HevcVideoParser extends BaseVideoParser {
                 track.params[prop] = config[prop];
               }
             }
-            if (track.vps !== undefined && track.vps[0] === this.initVPS) {
+            if (
+              (!track.vps && !track.pps.length) ||
+              (track.vps && track.vps[0] === this.initVPS)
+            ) {
               track.pps.push(unit.data);
             }
           }

package/src/utils/fetch-loader.ts

@@ -93,8 +93,6 @@ class FetchLoader implements Loader<LoaderContext> {
     stats.loading.start = self.performance.now();
 
     const initParams = getRequestParameters(context, this.controller.signal);
-    const onProgress: LoaderOnProgress<LoaderContext> | undefined =
-      callbacks.onProgress;
     const isArrayBuffer = context.responseType === 'arraybuffer';
     const LENGTH = isArrayBuffer ? 'byteLength' : 'length';
     const { maxTimeToFirstByteMs, maxLoadTimeMs } = config.loadPolicy;
@@ -109,8 +107,10 @@ class FetchLoader implements Loader<LoaderContext> {
         ? maxTimeToFirstByteMs
         : maxLoadTimeMs;
     this.requestTimeout = self.setTimeout(() => {
-      this.abortInternal();
-      callbacks.onTimeout(stats, context, this.response);
+      if (this.callbacks) {
+        this.abortInternal();
+        this.callbacks.onTimeout(stats, context, this.response);
+      }
     }, config.timeout);
 
     const fetchPromise = isPromise(this.request)
@@ -127,8 +127,10 @@ class FetchLoader implements Loader<LoaderContext> {
         config.timeout = maxLoadTimeMs;
         this.requestTimeout = self.setTimeout(
           () => {
-            this.abortInternal();
-            callbacks.onTimeout(stats, context, this.response);
+            if (this.callbacks) {
+              this.abortInternal();
+              this.callbacks.onTimeout(stats, context, this.response);
+            }
           },
           maxLoadTimeMs - (first - stats.loading.start),
         );
@@ -145,6 +147,7 @@ class FetchLoader implements Loader<LoaderContext> {
 
         stats.total = getContentLength(response.headers) || stats.total;
 
+        const onProgress = this.callbacks?.onProgress;
         if (onProgress && Number.isFinite(config.highWaterMark)) {
           return this.loadProgressively(
             response,
@@ -184,11 +187,12 @@ class FetchLoader implements Loader<LoaderContext> {
           code: response.status,
         };
 
+        const onProgress = this.callbacks?.onProgress;
         if (onProgress && !Number.isFinite(config.highWaterMark)) {
           onProgress(stats, context, responseData, response);
         }
 
-        callbacks.onSuccess(loaderResponse, stats, context, response);
+        this.callbacks?.onSuccess(loaderResponse, stats, context, response);
       })
       .catch((error) => {
         self.clearTimeout(this.requestTimeout);
@@ -199,7 +203,7 @@ class FetchLoader implements Loader<LoaderContext> {
         // when destroying, 'error' itself can be undefined
         const code: number = !error ? 0 : error.code || 0;
         const text: string = !error ? null : error.message;
-        callbacks.onError(
+        this.callbacks?.onError(
           { code, text },
           context,
           error ? error.details : null,

package/src/utils/xhr-loader.ts

@@ -113,7 +113,7 @@ class XhrLoader implements Loader<LoaderContext> {
         })
         .catch((error: Error) => {
           // IE11 throws an exception on xhr.open if attempting to access an HTTP resource over HTTPS
-          this.callbacks!.onError(
+          this.callbacks?.onError(
             { code: xhr.status, text: error.message },
             context,
             xhr,
@@ -220,23 +220,16 @@ class XhrLoader implements Loader<LoaderContext> {
             stats.loaded = stats.total = len;
             stats.bwEstimate =
               (stats.total * 8000) / (stats.loading.end - stats.loading.first);
-            if (!this.callbacks) {
-              return;
-            }
-            const onProgress = this.callbacks.onProgress;
+            const onProgress = this.callbacks?.onProgress;
             if (onProgress) {
               onProgress(stats, context, data, xhr);
             }
-            if (!this.callbacks) {
-              return;
-            }
             const response: LoaderResponse = {
               url: xhr.responseURL,
               data: data,
               code: status,
             };
-
-            this.callbacks.onSuccess(response, stats, context, xhr);
+            this.callbacks?.onSuccess(response, stats, context, xhr);
             return;
           }
         }
@@ -254,7 +247,7 @@ class XhrLoader implements Loader<LoaderContext> {
           this.retry(retryConfig);
         } else {
           logger.error(`${status} while loading ${context.url}`);
-          this.callbacks!.onError(
+          this.callbacks?.onError(
             { code: status, text: xhr.statusText },
             context,
             xhr,