@xiboplayer/renderer 0.7.5 → 0.7.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xiboplayer/renderer",
-  "version": "0.7.5",
+  "version": "0.7.6",
   "description": "RendererLite - Fast, efficient XLF layout rendering engine",
   "type": "module",
   "main": "./src/index.js",
@@ -12,9 +12,9 @@
   },
   "dependencies": {
     "pdfjs-dist": "^4.10.38",
-    "@xiboplayer/cache": "0.7.5",
-    "@xiboplayer/schedule": "0.7.5",
-    "@xiboplayer/utils": "0.7.5"
+    "@xiboplayer/cache": "0.7.6",
+    "@xiboplayer/utils": "0.7.6",
+    "@xiboplayer/schedule": "0.7.6"
   },
   "devDependencies": {
     "jsdom": "^25.0.1",

package/src/renderer-lite.js

@@ -714,6 +714,7 @@ export class RendererLite {
     if (maxRegionDuration > 0 && maxRegionDuration !== this.currentLayout.duration) {
       const oldDuration = this.currentLayout.duration;
       this.currentLayout.duration = maxRegionDuration;
+      this.currentLayout._durationFromMetadata = true;
 
       this.log.info(`Layout duration updated: ${oldDuration}s → ${maxRegionDuration}s (based on video metadata)`);
       const final_ = !this._hasUnprobedVideos();
@@ -1639,7 +1640,9 @@ export class RendererLite {
     // Dynamic layouts (useDuration=0 videos): defer timer until video metadata
     // provides real durations. Safety timeout ensures corrupt/missing videos
     // don't freeze the display forever.
-    if (layout.isDynamic && this._hasUnprobedVideos()) {
+    // Skip deferral if updateLayoutDuration() already set the duration from
+    // video metadata (e.g. during preload or a previous play of this layout).
+    if (layout.isDynamic && !layout._durationFromMetadata && this._hasUnprobedVideos()) {
       this._deferredTimerLayoutId = layoutId;
       this._layoutTimerStartedAt = Date.now();
       this.log.info(`Layout ${layoutId} has unprobed videos — deferring timer until metadata loads`);
@@ -1662,14 +1665,30 @@ export class RendererLite {
   }
 
   /**
-   * Check if any video widget has useDuration=0 ("play to end") and hasn't
-   * been corrected by video metadata yet. The XLF always provides a non-zero
-   * duration attribute (typically 60s), so we check the _probed flag instead.
+   * Check if any region's longest-running video widget (useDuration=0) hasn't
+   * been probed yet. Used to decide whether to defer the layout timer.
+   *
+   * Only checks widgets that have had <video> elements created (during preload
+   * or show). Widgets that haven't been displayed yet can never be probed —
+   * checking them would always force a 30s timeout on layouts with multiple
+   * video widgets per region.
+   *
+   * Returns false if the layout duration has already been updated from video
+   * metadata (meaning at least one probe succeeded and updateLayoutDuration
+   * computed a real duration), since the timer can start with that value.
    */
   _hasUnprobedVideos() {
+    // If any video was probed and updateLayoutDuration ran, the layout duration
+    // is already based on real metadata — no need to defer further.
+    for (const [, region] of this.regions) {
+      for (const widget of region.widgets) {
+        if (widget.type === 'video' && widget.useDuration === 0 && widget._probed) return false;
+      }
+    }
+    // No videos probed at all — check if there are any that need probing
     for (const [, region] of this.regions) {
       for (const widget of region.widgets) {
-        if (widget.useDuration === 0 && !widget._probed) return true;
+        if (widget.type === 'video' && widget.useDuration === 0) return true;
       }
     }
     return false;