rx-player 3.30.0-dev.2023020100 → 3.30.0-dev.2023030200

package/src/core/stream/representation/representation_stream.ts

@@ -97,16 +97,16 @@ export default function RepresentationStream<TSegmentDataType>(
   const bufferType = adaptation.type;
 
   /** `TaskCanceller` stopping ALL operations performed by the `RepresentationStream` */
-  const globalCanceller = new TaskCanceller({ cancelOn: parentCancelSignal });
+  const globalCanceller = new TaskCanceller();
+  globalCanceller.linkToSignal(parentCancelSignal);
 
   /**
    * `TaskCanceller` allowing to only stop segment loading and checking operations.
    * This allows to stop only tasks linked to network resource usage, which is
    * often a limited resource, while still letting buffer operations to finish.
    */
-  const segmentsLoadingCanceller = new TaskCanceller({
-    cancelOn: globalCanceller.signal,
-  });
+  const segmentsLoadingCanceller = new TaskCanceller();
+  segmentsLoadingCanceller.linkToSignal(globalCanceller.signal);
 
   /** Saved initialization segment state for this representation. */
   const initSegmentState : IInitSegmentState<TSegmentDataType> = {
@@ -149,7 +149,7 @@ export default function RepresentationStream<TSegmentDataType>(
       callbacks.encryptionDataEncountered(
         encryptionData.map(d => objectAssign({ content }, d))
       );
-      if (globalCanceller.isUsed) {
+      if (globalCanceller.isUsed()) {
         return ; // previous callback has stopped everything by side-effect
       }
     }
@@ -161,7 +161,7 @@ export default function RepresentationStream<TSegmentDataType>(
                                                 segmentFetcher,
                                                 hasInitSegment);
   downloadingQueue.addEventListener("error", (err) => {
-    if (segmentsLoadingCanceller.signal.isCancelled) {
+    if (segmentsLoadingCanceller.signal.isCancelled()) {
       return; // ignore post requests-cancellation loading-related errors,
     }
     globalCanceller.cancel(); // Stop every operations
@@ -172,7 +172,7 @@ export default function RepresentationStream<TSegmentDataType>(
   downloadingQueue.addEventListener("emptyQueue", checkStatus);
   downloadingQueue.addEventListener("requestRetry", (payload) => {
     callbacks.warning(payload.error);
-    if (segmentsLoadingCanceller.signal.isCancelled) {
+    if (segmentsLoadingCanceller.signal.isCancelled()) {
       return; // If the previous callback led to loading operations being stopped, skip
     }
     const retriedSegment = payload.segment;
@@ -218,7 +218,7 @@ export default function RepresentationStream<TSegmentDataType>(
    * issues at the current time, calling the right callbacks if necessary.
    */
   function checkStatus() : void {
-    if (segmentsLoadingCanceller.isUsed) {
+    if (segmentsLoadingCanceller.isUsed()) {
       return ; // Stop all buffer status checking if load operations are stopped
     }
     const observation = playbackObserver.getReference().getValue();
@@ -305,7 +305,7 @@ export default function RepresentationStream<TSegmentDataType>(
                                    isEmptyStream: false,
                                    hasFinishedLoading: status.hasFinishedLoading,
                                    neededSegments: status.neededSegments });
-    if (segmentsLoadingCanceller.signal.isCancelled) {
+    if (segmentsLoadingCanceller.signal.isCancelled()) {
       return ; // previous callback has stopped loading operations by side-effect
     }
     const { UPTO_CURRENT_POSITION_CLEANUP } = config.getCurrent();
@@ -314,7 +314,7 @@ export default function RepresentationStream<TSegmentDataType>(
         0,
         initialWantedTime - UPTO_CURRENT_POSITION_CLEANUP);
       if (gcedPosition > 0) {
-        segmentBuffer.removeBuffer(0, gcedPosition, segmentsLoadingCanceller.signal)
+        segmentBuffer.removeBuffer(0, gcedPosition, globalCanceller.signal)
           .catch(onFatalBufferError);
       }
     }
@@ -332,7 +332,7 @@ export default function RepresentationStream<TSegmentDataType>(
     evt : IParsedInitSegmentPayload<TSegmentDataType> |
           IParsedSegmentPayload<TSegmentDataType>
   ) : void {
-    if (globalCanceller.isUsed) {
+    if (globalCanceller.isUsed()) {
       // We should not do anything with segments if the `RepresentationStream`
       // is not running anymore.
       return ;
@@ -382,7 +382,7 @@ export default function RepresentationStream<TSegmentDataType>(
           callbacks.encryptionDataEncountered(
             allEncryptionData.map(p => objectAssign({ content }, p))
           );
-          if (globalCanceller.isUsed) {
+          if (globalCanceller.isUsed()) {
             return ; // previous callback has stopped everything by side-effect
           }
         }
@@ -390,13 +390,13 @@ export default function RepresentationStream<TSegmentDataType>(
 
       if (needsManifestRefresh === true) {
         callbacks.needsManifestRefresh();
-        if (globalCanceller.isUsed) {
+        if (globalCanceller.isUsed()) {
           return ; // previous callback has stopped everything by side-effect
         }
       }
       if (inbandEvents !== undefined && inbandEvents.length > 0) {
         callbacks.inbandEvent(inbandEvents);
-        if (globalCanceller.isUsed) {
+        if (globalCanceller.isUsed()) {
           return ; // previous callback has stopped everything by side-effect
         }
       }
@@ -425,7 +425,7 @@ export default function RepresentationStream<TSegmentDataType>(
    * @param {*} err
    */
   function onFatalBufferError(err : unknown) : void {
-    if (globalCanceller.isUsed && err instanceof CancellationError) {
+    if (globalCanceller.isUsed() && err instanceof CancellationError) {
       // The error is linked to cancellation AND we explicitely cancelled buffer
       // operations.
       // We can thus ignore it, it is very unlikely to lead to true buffer issues.

package/src/core/stream/representation/utils/append_segment_to_buffer.ts

@@ -47,7 +47,7 @@ export default async function appendSegmentToBuffer<T>(
   try {
     await segmentBuffer.pushChunk(dataInfos, cancellationSignal);
   } catch (appendError : unknown) {
-    if (cancellationSignal.isCancelled && appendError instanceof CancellationError) {
+    if (cancellationSignal.isCancelled() && appendError instanceof CancellationError) {
       throw appendError;
     } else if (!(appendError instanceof Error) ||
                appendError.name !== "QuotaExceededError")

package/src/core/stream/representation/utils/downloading_queue.ts

@@ -28,6 +28,7 @@ import {
 } from "../../../../transports";
 import assert from "../../../../utils/assert";
 import EventEmitter from "../../../../utils/event_emitter";
+import noop from "../../../../utils/noop";
 import objectAssign from "../../../../utils/object_assign";
 import createSharedReference, {
   IReadOnlySharedReference,
@@ -231,6 +232,7 @@ export default class DownloadingQueue<T>
 
   public stop() {
     this._currentCanceller?.cancel();
+    this._currentCanceller = null;
   }
 
   /**
@@ -245,7 +247,7 @@ export default class DownloadingQueue<T>
     const recursivelyRequestSegments = (
       startingSegment : IQueuedSegment | undefined
     ) : void  => {
-      if (this._currentCanceller !== null && this._currentCanceller.isUsed) {
+      if (this._currentCanceller !== null && this._currentCanceller.isUsed()) {
         this._mediaSegmentRequest = null;
         return;
       }
@@ -254,7 +256,10 @@ export default class DownloadingQueue<T>
         this.trigger("emptyQueue", null);
         return;
       }
-      const canceller = new TaskCanceller({ cancelOn: this._currentCanceller?.signal });
+      const canceller = new TaskCanceller();
+      const unlinkCanceller = this._currentCanceller === null ?
+        noop :
+        canceller.linkToSignal(this._currentCanceller.signal);
 
       const { segment, priority } = startingSegment;
       const context = objectAssign({ segment }, this._content);
@@ -368,6 +373,7 @@ export default class DownloadingQueue<T>
          * requests are scheduled. It is used to schedule the next segment.
          */
         beforeEnded: () : void => {
+          unlinkCanceller();
           this._mediaSegmentRequest = null;
 
           if (isWaitingOnInitSegment) {
@@ -381,6 +387,7 @@ export default class DownloadingQueue<T>
       }, canceller.signal);
 
       request.catch((error : unknown) => {
+        unlinkCanceller();
         if (!isComplete) {
           isComplete = true;
           this.stop();
@@ -400,7 +407,7 @@ export default class DownloadingQueue<T>
   private _restartInitSegmentDownloadingQueue(
     queuedInitSegment : IQueuedSegment | null
   ) : void {
-    if (this._currentCanceller !== null && this._currentCanceller.isUsed) {
+    if (this._currentCanceller !== null && this._currentCanceller.isUsed()) {
       return;
     }
     if (this._initSegmentRequest !== null) {
@@ -410,7 +417,10 @@ export default class DownloadingQueue<T>
       return ;
     }
 
-    const canceller = new TaskCanceller({ cancelOn: this._currentCanceller?.signal });
+    const canceller = new TaskCanceller();
+    const unlinkCanceller = this._currentCanceller === null ?
+      noop :
+      canceller.linkToSignal(this._currentCanceller.signal);
     const { segment, priority } = queuedInitSegment;
     const context = objectAssign({ segment }, this._content);
 
@@ -428,6 +438,7 @@ export default class DownloadingQueue<T>
         log.info("Stream: init segment request interrupted temporarly.", segment.id);
       },
       beforeEnded: () => {
+        unlinkCanceller();
         this._initSegmentRequest = null;
         isComplete = true;
       },
@@ -446,6 +457,7 @@ export default class DownloadingQueue<T>
     }, canceller.signal);
 
     request.catch((error : unknown) => {
+      unlinkCanceller();
       if (!isComplete) {
         isComplete = true;
         this.stop();

package/src/default_config.ts

@@ -923,15 +923,36 @@ const DEFAULT_CONFIG = {
      */
   MIN_CANCELABLE_PRIORITY: 3, // priority number 3 onward can be cancelled
 
-    /**
-     * Robustnesses used in the {audio,video}Capabilities of the
-     * MediaKeySystemConfiguration (DRM).
-     *
-     * Only used for widevine keysystems.
-     *
-     * Defined in order of importance (first will be tested first etc.)
-     * @type {Array.<string>}
-     */
+  /**
+   * Codecs used in the videoCapabilities of the MediaKeySystemConfiguration
+   * (DRM).
+   *
+   * Defined in order of importance (first will be tested first etc.)
+   * @type {Array.<string>}
+   */
+  EME_DEFAULT_VIDEO_CODECS: [ "video/mp4;codecs=\"avc1.4d401e\"",
+                              "video/mp4;codecs=\"avc1.42e01e\"",
+                              "video/webm;codecs=\"vp8\"" ],
+
+  /**
+   * Codecs used in the audioCapabilities of the MediaKeySystemConfiguration
+   * (DRM).
+   *
+   * Defined in order of importance (first will be tested first etc.)
+   * @type {Array.<string>}
+   */
+  EME_DEFAULT_AUDIO_CODECS: [ "audio/mp4;codecs=\"mp4a.40.2\"",
+                              "audio/webm;codecs=opus" ],
+
+  /**
+   * Robustnesses used in the {audio,video}Capabilities of the
+   * MediaKeySystemConfiguration (DRM).
+   *
+   * Only used for widevine keysystems.
+   *
+   * Defined in order of importance (first will be tested first etc.)
+   * @type {Array.<string>}
+   */
   EME_DEFAULT_WIDEVINE_ROBUSTNESSES: [ "HW_SECURE_ALL",
                                        "HW_SECURE_DECODE",
                                        "HW_SECURE_CRYPTO",

package/src/experimental/features/__tests__/debug_element.test.ts

@@ -0,0 +1,26 @@
+/* eslint-disable @typescript-eslint/no-unsafe-assignment */
+/* eslint-disable @typescript-eslint/no-unsafe-member-access */
+/* eslint-disable @typescript-eslint/no-var-requires */
+/* eslint-disable @typescript-eslint/no-unsafe-call */
+/* eslint-disable @typescript-eslint/no-unsafe-assignment */
+/* eslint-disable @typescript-eslint/no-unsafe-return */
+
+describe("Features list - DEBUG_ELEMENT", () => {
+  beforeEach(() => {
+    jest.resetModules();
+  });
+
+  it("should add DEBUG_ELEMENT in the current features", () => {
+    const feat = {};
+    jest.mock("../../../core/api/debug", () => ({ __esModule: true as const,
+                                                  default: feat }));
+    const addFeature = jest.requireActual("../debug_element").default;
+
+    const featureObject : {
+      createDebugElement? : unknown;
+    } = {};
+    addFeature(featureObject);
+    expect(featureObject).toEqual({ createDebugElement: {} });
+    expect(featureObject.createDebugElement).toBe(feat);
+  });
+});

package/src/experimental/features/debug_element.ts

@@ -0,0 +1,13 @@
+import createDebugElement from "../../core/api/debug";
+import { IFeaturesObject } from "../../features/types";
+
+/**
+ * Add ability to parse SAMI text tracks in an HTML textrack mode.
+ * @param {Object} features
+ */
+function addDebugElementFeature(features : IFeaturesObject) : void {
+  features.createDebugElement = createDebugElement;
+}
+
+export { addDebugElementFeature as DEBUG_ELEMENT };
+export default addDebugElementFeature;

package/src/experimental/features/index.ts

@@ -15,5 +15,6 @@
  */
 
 export { DASH_WASM } from "./dash_wasm";
+export { DEBUG_ELEMENT } from "./debug_element";
 export { LOCAL_MANIFEST } from "./local";
 export { METAPLAYLIST } from "./metaplaylist";

package/src/experimental/tools/VideoThumbnailLoader/prepare_source_buffer.ts

@@ -18,6 +18,7 @@ import { MediaSource_ } from "../../../compat";
 import { resetMediaSource } from "../../../core/init/utils/create_media_source";
 import { AudioVideoSegmentBuffer } from "../../../core/segment_buffers/implementations";
 import log from "../../../log";
+import createCancellablePromise from "../../../utils/create_cancellable_promise";
 import isNonEmptyString from "../../../utils/is_non_empty_string";
 import { CancellationSignal } from "../../../utils/task_canceller";
 
@@ -33,7 +34,7 @@ export default function prepareSourceBuffer(
   codec: string,
   cleanUpSignal: CancellationSignal
 ): Promise<AudioVideoSegmentBuffer> {
-  return new Promise((resolve, reject) => {
+  return createCancellablePromise(cleanUpSignal, (resolve, reject) => {
     if (MediaSource_ == null) {
       throw new Error("No MediaSource Object was found in the current browser.");
     }
@@ -49,15 +50,17 @@ export default function prepareSourceBuffer(
 
     log.info("Init: Attaching MediaSource URL to the media element", objectURL);
     videoElement.src = objectURL;
+    cleanUpSignal.register(() => {
+      resetMediaSource(videoElement, mediaSource, objectURL);
+    });
 
     mediaSource.addEventListener("sourceopen", onSourceOpen);
     mediaSource.addEventListener("webkitsourceopen", onSourceOpen);
 
-    cleanUpSignal.register(() => {
+    return () => {
       mediaSource.removeEventListener("sourceopen", onSourceOpen);
       mediaSource.removeEventListener("webkitsourceopen", onSourceOpen);
-      resetMediaSource(videoElement, mediaSource, objectURL);
-    });
+    };
 
     function onSourceOpen() {
       try {

package/src/experimental/tools/VideoThumbnailLoader/video_thumbnail_loader.ts

@@ -243,15 +243,19 @@ export default class VideoThumbnailLoader {
           if (pending !== undefined) {
             promises.push(pending.promise);
           } else {
-            const requestCanceller = new TaskCanceller({
-              cancelOn: lastRepInfo.cleaner.signal,
-            });
+            const requestCanceller = new TaskCanceller();
+            const unlinkSignal = requestCanceller
+              .linkToSignal(lastRepInfo.cleaner.signal);
             const segmentInfo = objectAssign({ segment },
                                              content);
             const prom = loadAndPushSegment(segmentInfo,
                                             segmentBuffer,
                                             lastRepInfo.segmentFetcher,
-                                            requestCanceller.signal);
+                                            requestCanceller.signal)
+              .then(unlinkSignal, (err) => {
+                unlinkSignal();
+                throw err;
+              });
             const newReq = {
               segmentId: segment.id,
               canceller: requestCanceller,

package/src/features/__tests__/initialize_features.test.ts

@@ -48,6 +48,7 @@ describe("Features - initializeFeaturesObject", () => {
       HTML_VTT: 0,
       LOCAL_MANIFEST: 0,
       METAPLAYLIST: 0,
+      DEBUG_ELEMENT: 0,
       NATIVE_SAMI: 0,
       NATIVE_SRT: 0,
       NATIVE_TTML: 0,
@@ -78,6 +79,7 @@ describe("Features - initializeFeaturesObject", () => {
       HTML_VTT: 1,
       LOCAL_MANIFEST: 1,
       METAPLAYLIST: 1,
+      DEBUG_ELEMENT: 1,
       NATIVE_SAMI: 1,
       NATIVE_SRT: 1,
       NATIVE_TTML: 1,
@@ -114,6 +116,7 @@ describe("Features - initializeFeaturesObject", () => {
         wasm: null,
       },
       ContentDecryptor: jest.requireActual("../../core/decrypt/index").default,
+      createDebugElement: jest.requireActual("../../core/api/debug").default,
       directfile: {
         initDirectFile: jest.requireActual("../../core/init/directfile_content_initializer").default,
         mediaElementTrackChoiceManager:
@@ -161,6 +164,7 @@ describe("Features - initializeFeaturesObject", () => {
       HTML_VTT: 1,
       LOCAL_MANIFEST: 0,
       METAPLAYLIST: 0,
+      DEBUG_ELEMENT: 0,
       NATIVE_SAMI: 0,
       NATIVE_SRT: 0,
       NATIVE_TTML: 0,
@@ -203,6 +207,7 @@ describe("Features - initializeFeaturesObject", () => {
       HTML_VTT: 0,
       LOCAL_MANIFEST: 0,
       METAPLAYLIST: 0,
+      DEBUG_ELEMENT: 0,
       NATIVE_SAMI: 0,
       NATIVE_SRT: 0,
       NATIVE_TTML: 0,
@@ -245,6 +250,7 @@ describe("Features - initializeFeaturesObject", () => {
       HTML_VTT: 0,
       LOCAL_MANIFEST: 0,
       METAPLAYLIST: 0,
+      DEBUG_ELEMENT: 0,
       NATIVE_SAMI: 0,
       NATIVE_SRT: 0,
       NATIVE_TTML: 0,
@@ -287,6 +293,7 @@ describe("Features - initializeFeaturesObject", () => {
       HTML_VTT: 0,
       LOCAL_MANIFEST: 0,
       METAPLAYLIST: 0,
+      DEBUG_ELEMENT: 0,
       NATIVE_SAMI: 0,
       NATIVE_SRT: 0,
       NATIVE_TTML: 0,
@@ -329,6 +336,7 @@ describe("Features - initializeFeaturesObject", () => {
       HTML_VTT: 0,
       LOCAL_MANIFEST: 0,
       METAPLAYLIST: 0,
+      DEBUG_ELEMENT: 0,
       NATIVE_SAMI: 0,
       NATIVE_SRT: 0,
       NATIVE_TTML: 0,
@@ -371,6 +379,7 @@ describe("Features - initializeFeaturesObject", () => {
       HTML_VTT: 0,
       LOCAL_MANIFEST: 0,
       METAPLAYLIST: 0,
+      DEBUG_ELEMENT: 0,
       NATIVE_SAMI: 1,
       NATIVE_SRT: 0,
       NATIVE_TTML: 0,
@@ -413,6 +422,7 @@ describe("Features - initializeFeaturesObject", () => {
       HTML_VTT: 0,
       LOCAL_MANIFEST: 0,
       METAPLAYLIST: 0,
+      DEBUG_ELEMENT: 0,
       NATIVE_SAMI: 0,
       NATIVE_SRT: 0,
       NATIVE_TTML: 1,
@@ -455,6 +465,7 @@ describe("Features - initializeFeaturesObject", () => {
       HTML_VTT: 0,
       LOCAL_MANIFEST: 0,
       METAPLAYLIST: 0,
+      DEBUG_ELEMENT: 0,
       NATIVE_SAMI: 0,
       NATIVE_SRT: 1,
       NATIVE_TTML: 0,

package/src/features/features_object.ts

@@ -22,6 +22,7 @@ import { IFeaturesObject } from "./types";
  */
 const features : IFeaturesObject = { dashParsers: { wasm: null,
                                                     js: null },
+                                     createDebugElement: null,
                                      directfile: null,
                                      ContentDecryptor: null,
                                      htmlTextTracksBuffer: null,

package/src/features/initialize_features.ts

@@ -37,10 +37,11 @@ export default function initializeFeaturesObject() : void {
   }
 
   // Feature switching the Native TextTrack implementation
-  const HAS_NATIVE_MODE = __FEATURES__.NATIVE_VTT ||
-                          __FEATURES__.NATIVE_SAMI ||
-                          __FEATURES__.NATIVE_TTML ||
-                          __FEATURES__.NATIVE_SRT;
+  const HAS_NATIVE_MODE =
+    __FEATURES__.NATIVE_VTT === __FEATURES__.IS_ENABLED as number ||
+    __FEATURES__.NATIVE_SAMI === __FEATURES__.IS_ENABLED as number ||
+    __FEATURES__.NATIVE_TTML === __FEATURES__.IS_ENABLED as number ||
+    __FEATURES__.NATIVE_SRT === __FEATURES__.IS_ENABLED as number;
 
   if (__FEATURES__.SMOOTH === __FEATURES__.IS_ENABLED as number) {
     features.transports.smooth = require("../transports/smooth/index.ts").default;
@@ -57,8 +58,11 @@ export default function initializeFeaturesObject() : void {
     features.transports.metaplaylist =
       require("../transports/metaplaylist/index.ts").default;
   }
+  if (__FEATURES__.DEBUG_ELEMENT === __FEATURES__.IS_ENABLED as number) {
+    features.createDebugElement = require("../core/api/debug/index.ts").default;
+  }
 
-  if (HAS_NATIVE_MODE === __FEATURES__.IS_ENABLED as number) {
+  if (HAS_NATIVE_MODE) {
     features.nativeTextTracksBuffer =
       require("../core/segment_buffers/implementations/text/native/index.ts").default;
     if (__FEATURES__.NATIVE_VTT === __FEATURES__.IS_ENABLED as number) {
@@ -83,12 +87,13 @@ export default function initializeFeaturesObject() : void {
   }
 
   // Feature switching the HTML TextTrack implementation
-  const HAS_HTML_MODE = __FEATURES__.HTML_VTT ||
-                        __FEATURES__.HTML_SAMI ||
-                        __FEATURES__.HTML_TTML ||
-                        __FEATURES__.HTML_SRT;
+  const HAS_HTML_MODE =
+    __FEATURES__.HTML_VTT === __FEATURES__.IS_ENABLED as number ||
+    __FEATURES__.HTML_SAMI === __FEATURES__.IS_ENABLED as number ||
+    __FEATURES__.HTML_TTML === __FEATURES__.IS_ENABLED as number ||
+    __FEATURES__.HTML_SRT === __FEATURES__.IS_ENABLED as number;
 
-  if (HAS_HTML_MODE === __FEATURES__.IS_ENABLED as number) {
+  if (HAS_HTML_MODE) {
     features.htmlTextTracksBuffer =
       require("../core/segment_buffers/implementations/text/html/index.ts").default;
     if (__FEATURES__.HTML_SAMI === __FEATURES__.IS_ENABLED as number) {

package/src/features/types.ts

@@ -14,6 +14,7 @@
  * limitations under the License.
  */
 
+import RxPlayer from "../core/api";
 // eslint-disable-next-line max-len
 import MediaElementTrackChoiceManager from "../core/api/tracks_management/media_element_track_choice_manager";
 import type ContentDecryptor from "../core/decrypt";
@@ -29,6 +30,7 @@ import {
   INativeTextTracksParserFn,
 } from "../parsers/texttracks";
 import { ITransportFunction } from "../transports";
+import { CancellationSignal } from "../utils/task_canceller";
 
 export type IDirectFileInit = typeof DirectFileContentInitializer;
 
@@ -78,6 +80,13 @@ export interface IFeaturesObject {
                  mediaElementTrackChoiceManager : IMediaElementTrackChoiceManager; } |
                null;
   ContentDecryptor : IContentDecryptorClass|null;
+  createDebugElement : (
+    (
+      parentElt : HTMLElement,
+      instance : RxPlayer,
+      cancelSignal : CancellationSignal
+    ) => void
+  ) | null;
   htmlTextTracksBuffer : IHTMLTextTracksBuffer|null;
   htmlTextTracksParsers : Partial<Record<string, IHTMLTextTracksParserFn>>;
   imageBuffer : IImageBuffer|null;

package/src/manifest/adaptation.ts

@@ -14,6 +14,7 @@
  * limitations under the License.
  */
 
+import log from "../log";
 import { IParsedAdaptation } from "../parsers/manifest";
 import { IRepresentationFilter } from "../public_types";
 import arrayFind from "../utils/array_find";
@@ -164,6 +165,12 @@ export default class Adaptation {
         if (!isSupported && representation.isSupported) {
           isSupported = true;
         }
+      } else {
+        log.debug("Filtering Representation due to representationFilter",
+                  this.type,
+                  `Adaptation: ${this.id}`,
+                  `Representation: ${representation.id}`,
+                  `(${representation.bitrate})`);
       }
     }
     representations.sort((a, b) => a.bitrate - b.bitrate);

package/src/manifest/manifest.ts

@@ -15,6 +15,7 @@
  */
 
 import { MediaError } from "../errors";
+import log from "../log";
 import { IParsedManifest } from "../parsers/manifest";
 import {
   IPlayerError,
@@ -786,6 +787,9 @@ function updateDeciperability(
         const result = isDecipherable(representation);
         if (result !== representation.decipherable) {
           updates.push({ manifest, period, adaptation, representation });
+          log.debug(`Decipherability changed for "${representation.id}"`,
+                    `(${representation.bitrate})`,
+                    String(representation.decipherable));
           representation.decipherable = result;
         }
       }

package/src/manifest/representation.ts

@@ -145,10 +145,16 @@ class Representation {
     this.cdnMetadata = args.cdnMetadata;
 
     this.index = args.index;
-    this.isSupported = opts.type === "audio" ||
-                       opts.type === "video" ?
-      isCodecSupported(this.getMimeTypeString()) :
-      true; // TODO for other types
+    if (opts.type === "audio" || opts.type === "video") {
+      const mimeTypeStr = this.getMimeTypeString();
+      const isSupported = isCodecSupported(mimeTypeStr);
+      if (!isSupported) {
+        log.info("Unsupported Representation", mimeTypeStr, this.id, this.bitrate);
+      }
+      this.isSupported = isSupported;
+    } else {
+      this.isSupported = true; // TODO for other types
+    }
   }
 
   /**