Improved Immediate queue management

packages/dev/core/src/Engines/abstractEngine.ts

@@ -52,6 +52,7 @@ import { Observable } from "../Misc/observable";
 import { EngineFunctionContext, _loadFile } from "./abstractEngine.functions";
 import type { Material } from "core/Materials/material";
 import { _GetCompatibleTextureLoader } from "core/Materials/Textures/Loaders/textureLoaderManager";
+import { _RunImmediateQueue } from "core/Misc/timingTools";
 
 /**
  * Defines the interface used by objects working like Scene
@@ -216,6 +217,9 @@ export abstract class AbstractEngine {
 
     protected _activeRequests: IFileRequest[] = [];
 
+    /** @internal */
+    public _immediateQueue: Array<() => void> = [];
+
     /** @internal */
     public _badOS = false;
     /** @internal */
@@ -880,6 +884,9 @@ export abstract class AbstractEngine {
         this._frameId++;
 
         this.onEndFrameObservable.notifyObservers(this);
+
+        /** It is ok to call it even if several engines run */
+        _RunImmediateQueue();
     }
 
     /**

packages/dev/core/src/Misc/timingTools.ts

@@ -1,4 +1,4 @@
-import { IsWindowObjectExist } from "./domManagement";
+const ImmediateQueue: Array<() => void> = [];
 
 /**
  * Class used to provide helper for timing
@@ -9,12 +9,26 @@ export class TimingTools {
      * @param action defines the action to execute after the current execution block
      */
     public static SetImmediate(action: () => void) {
-        if (IsWindowObjectExist() && window.setImmediate) {
-            // Note - deprecated and should not be used directly. Not supported in any browser.
-            window.setImmediate(action);
-        } else {
-            setTimeout(action, 1);
-        }
+        ImmediateQueue.push(action);
+    }
+}
+
+/**
+ * @internal
+ */
+export function _RunImmediateQueue() {
+    if (ImmediateQueue.length) {
+        // Execute all immediate functions
+        // Doing a copy even though it should be fine but I do not want to deal with race conditions
+        // in the future. So, let's be safe.
+        const functionsToCall = ImmediateQueue.slice(0);
+        ImmediateQueue.length = 0;
+
+        setTimeout(() => {
+            for (let i = 0; i < functionsToCall.length; i++) {
+                functionsToCall[i]();
+            }
+        }, 1);
     }
 }