fix: llHLS does not need forcedTimestampOffset (#1501)

* fix: llHLS does not need forcedTimestampOffset * more readable boolean
videojs · Apr 2, 2024 · f5d1209 · f5d1209
1 parent ddc9374
commit f5d1209
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/segment-loader.js b/src/segment-loader.js
@@ -1238,8 +1238,11 @@ bufferedEnd: ${lastBufferedEnd(this.buffered_())}
     this.syncPoint_ = null;
     this.isPendingTimestampOffset_ = false;
     // this is mainly to sync timing-info when switching between renditions with and without timestamp-rollover,
-    // so we don't want it for DASH
-    if (this.sourceType_ === 'hls') {
+    // so we don't want it for DASH or fragmented mp4 segments.
+    const isFmp4 = this.currentMediaInfo_ && this.currentMediaInfo_.isFmp4;
+    const isHlsTs = this.sourceType_ === 'hls' && !isFmp4;
+
+    if (isHlsTs) {
       this.shouldForceTimestampOffsetAfterResync_ = true;
     }
     this.callQueue_ = [];