From 5389f62aa3f7493ce72d0549689f65f2c1f31f14 Mon Sep 17 00:00:00 2001
From: Eoghan Murray <eoghan@getthere.ie>
Date: Mon, 25 Mar 2024 11:05:21 +0000
Subject: [PATCH] Delay call until all attributes are present on the new node,
 in case the AssetManager needs to interrogate other attributes to determine
 how to manage, e.g. <link with href depending on rel="stylesheet"

---
 packages/rrweb-snapshot/src/rebuild.ts | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/packages/rrweb-snapshot/src/rebuild.ts b/packages/rrweb-snapshot/src/rebuild.ts
index 29dc2a6a7d..e0dabbe75c 100644
--- a/packages/rrweb-snapshot/src/rebuild.ts
+++ b/packages/rrweb-snapshot/src/rebuild.ts
@@ -162,6 +162,7 @@ function buildNode(
        * We need to parse them last so they can overwrite conflicting attributes.
        */
       const specialAttributes: { [key: string]: string | number } = {};
+      const managedAttributes: [Element, number, string][] = [];
       for (const name in n.attributes) {
         if (!Object.prototype.hasOwnProperty.call(n.attributes, name)) {
           continue;
@@ -258,13 +259,15 @@ function buildNode(
             continue;
           } else {
             node.setAttribute(name, value.toString());
-
-            options.assetManager?.manageAttribute(node, n.id, name);
+            managedAttributes.push([node, n.id, name]);
           }
         } catch (error) {
           // skip invalid attribute
         }
       }
+      for (const ma of managedAttributes) {
+        options.assetManager?.manageAttribute(...ma);
+      }
 
       for (const name in specialAttributes) {
         const value = specialAttributes[name];