fix: cleanup iframe first to prevent memory leaks, see niklasvh#1609

yuyuyzl · Sep 6, 2022 · 485a49e · 485a49e
1 parent 6020386
commit 485a49e
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/src/dom/document-cloner.ts b/src/dom/document-cloner.ts
@@ -479,6 +479,16 @@ export class DocumentCloner {
     }
 
     static destroy(container: HTMLIFrameElement): boolean {
+        // cleanup iframe first to prevent memory leaks, see #1609
+        try{
+            const iframe=container.contentWindow;
+            container.src='about:blank';
+            if(iframe) {
+                iframe.document.write('');
+                iframe.document.clear();
+                iframe.close();
+            }
+        }catch {}
         if (container.parentNode) {
             container.parentNode.removeChild(container);
             return true;