Fix alt-audio video-dev#2006 (video-dev#2067)

* Compute live start position for alt audio streams
Biswaranjanp · Jun 5, 2019 · cf801df · cf801df
1 parent 1ea14ad
commit cf801df
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 7 deletions.
diff --git a/src/controller/audio-stream-controller.js b/src/controller/audio-stream-controller.js
@@ -458,7 +458,12 @@ class AudioStreamController extends BaseStreamController {
           logger.log(`start time offset found in playlist, adjust startPosition to ${startTimeOffset}`);
           this.startPosition = startTimeOffset;
         } else {
-          this.startPosition = 0;
+          if (newDetails.live) {
+            this.startPosition = this.computeLivePosition(sliding, newDetails);
+            logger.log(`compute startPosition for audio-track to ${this.startPosition}`);
+          } else {
+            this.startPosition = 0;
+          }
         }
       }
       this.nextLoadPosition = this.startPosition;

diff --git a/src/controller/base-stream-controller.js b/src/controller/base-stream-controller.js
@@ -125,4 +125,9 @@ export default class BaseStreamController extends TaskLoop {
     this.state = State.STOPPED;
     this.fragmentTracker = null;
   }
+
+  computeLivePosition (sliding, levelDetails) {
+    let targetLatency = this.config.liveSyncDuration !== undefined ? this.config.liveSyncDuration : this.config.liveSyncDurationCount * levelDetails.targetduration;
+    return sliding + Math.max(0, levelDetails.totalduration - targetLatency);
+  }
 }
diff --git a/src/controller/stream-controller.js b/src/controller/stream-controller.js
@@ -1292,12 +1292,6 @@ class StreamController extends BaseStreamController {
   swapAudioCodec () {
     this.audioCodecSwap = !this.audioCodecSwap;
   }
-
-  computeLivePosition (sliding, levelDetails) {
-    let targetLatency = this.config.liveSyncDuration !== undefined ? this.config.liveSyncDuration : this.config.liveSyncDurationCount * levelDetails.targetduration;
-    return sliding + Math.max(0, levelDetails.totalduration - targetLatency);
-  }
-
   /**
    * Seeks to the set startPosition if not equal to the mediaElement's current time.
    * @private