fix(FEC-11697): 2 captions are selected in the menu

karma.conf.js

@@ -18,7 +18,6 @@ const launchers = {
 module.exports = function (config) {
   let karmaConf = {
     logLevel: config.LOG_INFO,
-    browserNoActivityTimeout: 90000,
     browsers: ['Chrome', 'Firefox'],
     customLaunchers: launchers,
     concurrency: 1,
@@ -43,7 +42,8 @@ module.exports = function (config) {
     client: {
       mocha: {
         reporter: 'html',
-        timeout: 90000
+        timeout: 40000
+        // timeout: 2000 - this is the default, see https://mochajs.org/#-timeout-ms-t-ms
       }
     }
   };

package.json

@@ -34,6 +34,7 @@
     "pushTaggedRelease": "git push --follow-tags --no-verify origin master",
     "release": "standard-version",
     "test": "NODE_ENV=test karma start --color --mode development",
+    "test:watch": "NODE_ENV=test karma start --browsers=Chrome --single-run=false --auto-watch",
     "watch": "webpack --progress --colors --watch --mode development"
   },
   "husky": {

src/hls-adapter.js

@@ -98,6 +98,14 @@ export default class HlsAdapter extends BaseMediaSourceAdapter {
    */
   _loadPromise: ?Promise<Object>;
 
+  /**
+   * the _loadPromise handlers
+   * @member {{resolve: (result: Promise<R> | R) => void, reject: (error: any) => void}} - _loadPromiseHandlers
+   * @type {{resolve: (result: Promise<R> | R) => void, reject: (error: any) => void}}
+   * @private
+   */
+  _loadPromiseHandlers: {resolve: (result: Promise<*> | *) => void, reject: (error: any) => void} | null;
+
   /**
    * Reference to the player tracks.
    * @member {Array<Track>} - _playerTracks
@@ -123,7 +131,7 @@ export default class HlsAdapter extends BaseMediaSourceAdapter {
   _mediaAttachedPromise: Promise<*>;
   _requestFilterError: boolean = false;
   _responseFilterError: boolean = false;
-  _nativeTextTracksMap = [];
+  _nativeTextTracksMap = {};
   _lastLoadedFragSN: number = -1;
   _sameFragSNLoadedCount: number = 0;
   /**
@@ -525,8 +533,8 @@ export default class HlsAdapter extends BaseMediaSourceAdapter {
   load(startTime: ?number): Promise<Object> {
     if (!this._loadPromise) {
       this._startTime = startTime;
-      this._loadPromise = new Promise(resolve => {
-        this._resolveLoad = resolve;
+      this._loadPromise = new Promise((resolve, reject) => {
+        this._loadPromiseHandlers = {resolve, reject};
         this._loadInternal();
       });
     }
@@ -544,6 +552,8 @@ export default class HlsAdapter extends BaseMediaSourceAdapter {
       this._hls.loadSource(this._sourceObj.url);
       this._hls.attachMedia(this._videoElement);
       this._trigger(EventType.ABR_MODE_CHANGED, {mode: this.isAdaptiveBitrateEnabled() ? 'auto' : 'manual'});
+    } else {
+      this._loadPromiseHandlers?.reject(new window.Error('no url provided'));
     }
   }
 
@@ -576,15 +586,14 @@ export default class HlsAdapter extends BaseMediaSourceAdapter {
       super.destroy().then(
         () => {
           HlsAdapter._logger.debug('destroy');
-          this._loadPromise = null;
           this._playerTracks = [];
-          this._nativeTextTracksMap = [];
+          this._nativeTextTracksMap = {};
           this._sameFragSNLoadedCount = 0;
           this._lastLoadedFragSN = -1;
           this._reset();
           resolve();
         },
-        () => reject
+        () => reject()
       );
     });
   }
@@ -600,6 +609,11 @@ export default class HlsAdapter extends BaseMediaSourceAdapter {
     this._responseFilterError = false;
     this._hls.detachMedia();
     this._hls.destroy();
+    if (this._loadPromiseHandlers) {
+      this._loadPromiseHandlers.reject('The adapter has been reset/destoryed while loading');
+    }
+    this._loadPromiseHandlers = null;
+    this._loadPromise = null;
   }
 
   /**
@@ -667,15 +681,14 @@ export default class HlsAdapter extends BaseMediaSourceAdapter {
    */
   _parseTextTracks(hlsTextTracks: Array<Object>): Array<TextTrack> {
     let textTracks = [];
-    for (let i = 0; i < hlsTextTracks.length; i++) {
+    for (const hlsTextTrack of hlsTextTracks) {
       // Create text tracks
       let settings = {
-        id: hlsTextTracks[i].id,
-        active: hlsTextTracks[i].default,
-        label: hlsTextTracks[i].name,
-        kind: hlsTextTracks[i].type.toLowerCase(),
-        language: hlsTextTracks[i].lang,
-        index: i
+        id: hlsTextTrack.id,
+        active: hlsTextTrack.default,
+        label: hlsTextTrack.name,
+        kind: hlsTextTrack.type.toLowerCase(),
+        language: hlsTextTrack.lang
       };
       textTracks.push(new TextTrack(settings));
     }
@@ -696,11 +709,10 @@ export default class HlsAdapter extends BaseMediaSourceAdapter {
         active: CEATextTrack.mode === 'showing',
         label: CEATextTrack.label,
         kind: CEATextTrack.kind,
-        language: CEATextTrack.language,
-        index: this._playerTracks.filter(track => track instanceof TextTrack).length
+        language: CEATextTrack.language
       };
       textTrack = new TextTrack(settings);
-      this._nativeTextTracksMap[settings.index] = CEATextTrack;
+      this._nativeTextTracksMap[textTrack.index] = CEATextTrack;
     }
     return textTrack;
   }
@@ -928,7 +940,8 @@ export default class HlsAdapter extends BaseMediaSourceAdapter {
       this._hls.currentLevel = 0;
     }
     this._mediaAttachedPromise.then(() => {
-      this._resolveLoad({tracks: this._playerTracks});
+      this._loadPromiseHandlers?.resolve({tracks: this._playerTracks});
+      this._loadPromiseHandlers = null;
     });
     const {loading} = data.stats;
     const manifestDownloadTime = loading.end - loading.start;
@@ -1080,6 +1093,11 @@ export default class HlsAdapter extends BaseMediaSourceAdapter {
     const errorFatal = data.fatal;
     let errorDataObject = this._getErrorDataObject(data);
     if (errorFatal) {
+      if (this._loadPromiseHandlers) {
+        const {type, details, url} = data;
+        this._loadPromiseHandlers.reject({type, details, url});
+        this._loadPromiseHandlers = null;
+      }
       let error: typeof Error;
       switch (errorType) {
         case Hlsjs.ErrorTypes.NETWORK_ERROR: