@xiboplayer/renderer 0.7.7 → 0.7.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xiboplayer/renderer",
-  "version": "0.7.7",
+  "version": "0.7.9",
   "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/schedule": "0.7.7",
-    "@xiboplayer/utils": "0.7.7",
-    "@xiboplayer/cache": "0.7.7"
+    "@xiboplayer/utils": "0.7.9",
+    "@xiboplayer/cache": "0.7.9",
+    "@xiboplayer/schedule": "0.7.9"
   },
   "devDependencies": {
     "jsdom": "^25.0.1",

package/src/index.d.ts

@@ -74,6 +74,7 @@ export class RendererLite {
 
   updateLayoutDuration(): void;
   checkLayoutComplete(): void;
+  hasActiveLayoutTimer(): boolean;
 
   cleanup(): void;
 }

package/src/layout-pool.js

@@ -167,6 +167,15 @@ export class LayoutPool {
    * @param {HTMLElement} container
    */
   static releaseMediaElements(container) {
+    // Defer the actual release by one animation frame to give the GPU compositor
+    // time to stop referencing textures from the old layout. Without this delay,
+    // the compositor may still hold stale mailbox references when we destroy the
+    // video backing, causing SharedImageManager::ProduceSkia "non-existent mailbox"
+    // errors (Chrome bug: race in shared_image_manager.cc acknowledged in a TODO).
+    requestAnimationFrame(() => LayoutPool._releaseMediaElementsSync(container));
+  }
+
+  static _releaseMediaElementsSync(container) {
     let videoCount = 0;
     let hlsCount = 0;
 
@@ -195,13 +204,43 @@ export class LayoutPool {
       a.load();
     });
 
+    // Release media inside iframes (embedded widgets with HLS streams, webcams, etc.)
+    // We can't querySelectorAll('video') across iframe boundaries, but we can:
+    // 1. Try to access same-origin iframe contentDocument
+    // 2. Force-remove the iframe src to stop all network activity
+    let iframeCount = 0;
+    container.querySelectorAll('iframe').forEach(iframe => {
+      try {
+        // Same-origin iframes: reach inside and release videos
+        const doc = iframe.contentDocument || iframe.contentWindow?.document;
+        if (doc) {
+          doc.querySelectorAll('video').forEach(v => {
+            v.pause();
+            v.removeAttribute('src');
+            v.load();
+            videoCount++;
+          });
+          doc.querySelectorAll('audio').forEach(a => {
+            a.pause();
+            a.removeAttribute('src');
+            a.load();
+          });
+        }
+      } catch (_) {
+        // Cross-origin: can't access contentDocument
+      }
+      // Force stop all iframe network activity (HLS segments, SSE, WebSocket, etc.)
+      iframe.src = 'about:blank';
+      iframeCount++;
+    });
+
     // Destroy PDF documents and release GPU canvas backing stores
     container.querySelectorAll('.pdf-widget').forEach(el => {
       if (el._pdfDestroy) el._pdfDestroy();
     });
 
-    if (videoCount > 0) {
-      log.info(`Released ${videoCount} video(s)${hlsCount ? ` (${hlsCount} HLS)` : ''}`);
+    if (videoCount > 0 || iframeCount > 0) {
+      log.info(`Released ${videoCount} video(s)${hlsCount ? ` (${hlsCount} HLS)` : ''}${iframeCount ? `, ${iframeCount} iframe(s)` : ''}`);
     }
   }
 

package/src/renderer-lite.js

@@ -211,6 +211,7 @@ export class RendererLite {
     this.currentLayout = null;
     this.currentLayoutId = null;
     this._preloadingLayoutId = null; // Set during preload for blob URL tracking
+    this._preloadingPromise = null;  // Promise for in-flight preload (await instead of skip)
     this.regions = new Map(); // regionId => { element, widgets, currentIndex, timer }
     this.layoutTimer = null;
     this.layoutEndEmitted = false; // Prevents double layoutEnd on stop after timer
@@ -1911,6 +1912,21 @@ export class RendererLite {
       widgetElement._pdfCleanup();
     }
 
+    // Stop embedded widget iframes (HLS live streams, webcams, etc.)
+    // Setting src=about:blank kills all network activity (HLS segment fetches,
+    // WebSocket connections, SSE streams) and releases video decode buffers.
+    const iframes = widgetElement.querySelectorAll('iframe');
+    for (const iframe of iframes) {
+      try {
+        const doc = iframe.contentDocument || iframe.contentWindow?.document;
+        if (doc) {
+          doc.querySelectorAll('video').forEach(v => { v.pause(); v.removeAttribute('src'); v.load(); });
+          doc.querySelectorAll('audio').forEach(a => { a.pause(); a.removeAttribute('src'); a.load(); });
+        }
+      } catch (_) {}
+      iframe.src = 'about:blank';
+    }
+
     return { widget, animPromise };
   }
 
@@ -2872,13 +2888,20 @@ export class RendererLite {
       return true;
     }
 
-    // Don't preload if already in-flight (prevents triple preload when
-    // 75% and 90% timers both fire before the async preload completes)
-    if (this._preloadingLayoutId === layoutId) {
-      this.log.info(`Layout ${layoutId} preload already in-flight, skipping`);
-      return true;
+    // If already in-flight, wait for it instead of skipping (prevents the race
+    // where showLayout is called before the background preload finishes adding
+    // the layout to the pool).
+    if (this._preloadingLayoutId === layoutId && this._preloadingPromise) {
+      this.log.info(`Layout ${layoutId} preload in-flight, waiting for it...`);
+      return this._preloadingPromise;
     }
 
+    // Store the preload promise so concurrent callers can await it
+    this._preloadingPromise = this._doPreloadLayout(xlfXml, layoutId);
+    return this._preloadingPromise;
+  }
+
+  async _doPreloadLayout(xlfXml, layoutId) {
     try {
       this.log.info(`Preloading layout ${layoutId} into pool...`);
 
@@ -2981,6 +3004,11 @@ export class RendererLite {
     } catch (error) {
       this.log.error(`Preload failed for layout ${layoutId}:`, error);
       return false;
+    } finally {
+      if (this._preloadingLayoutId === layoutId) {
+        this._preloadingLayoutId = null;
+        this._preloadingPromise = null;
+      }
     }
   }
 
@@ -3192,6 +3220,15 @@ export class RendererLite {
     this._swapToPreloadedLayout(layoutId);
   }
 
+  /**
+   * Check if the layout timer is active (running or deferred waiting for metadata).
+   * Used to detect stalled layouts that need timer restart.
+   * @returns {boolean}
+   */
+  hasActiveLayoutTimer() {
+    return this.layoutTimer !== null || this._deferredTimerLayoutId !== null;
+  }
+
   /**
    * Check if all regions have completed one full cycle
    * This is informational only - layout timer is authoritative

package/src/renderer-lite.test.js

@@ -817,12 +817,9 @@ describe('RendererLite', () => {
       const region = { id: 'r1', widgets: [widget1, widget2], isDrawer: false };
       renderer.regions = new Map([['r1', region]]);
 
-      // One still unprobed
-      expect(renderer._hasUnprobedVideos()).toBe(true);
-
-      // Probe the second
-      widget2.duration = 30;
-      widget2._probed = true;
+      // Once any video is probed, updateLayoutDuration has already run with
+      // real metadata, so _hasUnprobedVideos() returns false — no further
+      // deferral needed.
       expect(renderer._hasUnprobedVideos()).toBe(false);
     });
   });