Merge pull request #923 from lightpanda-io/doc-owner-next

node: don't call owner twice in _insertBefore

src/browser/dom/node.zig

@@ -307,6 +307,10 @@ pub const Node = struct {
     }
 
     pub fn _insertBefore(self: *parser.Node, new_node: *parser.Node, ref_node_: ?*parser.Node) !Union {
+        if (ref_node_ == null) {
+            return _appendChild(self, new_node);
+        }
+
         const self_owner = try parser.nodeOwnerDocument(self);
         const new_node_owner = try parser.nodeOwnerDocument(new_node);
 
@@ -318,16 +322,13 @@ pub const Node = struct {
         if (new_node_owner == null or (new_node_owner.? != self_owner.?)) {
             const w = Walker{};
             var current = new_node;
-            while(true) {
+            while (true) {
                 current.owner = self_owner;
                 current = try w.get_next(new_node, current) orelse break;
             }
         }
 
-        if (ref_node_) |ref_node| {
-            return Node.toInterface(try parser.nodeInsertBefore(self, new_node, ref_node));
-        }
-        return _appendChild(self, new_node);
+        return Node.toInterface(try parser.nodeInsertBefore(self, new_node, ref_node_.?));
     }
 
     pub fn _isDefaultNamespace(self: *parser.Node, namespace: ?[]const u8) !bool {
@@ -756,7 +757,7 @@ test "Browser.DOM.node" {
 }
 
 test "Browser.DOM.node.owner" {
-    var runner = try testing.jsRunner(testing.tracking_allocator, .{ .html= 
+    var runner = try testing.jsRunner(testing.tracking_allocator, .{ .html = 
         \\  <div id="target-container">
         \\     <p id="reference-node">
         \\         I am the original reference node.
@@ -770,12 +771,9 @@ test "Browser.DOM.node.owner" {
         .{
             \\ const parser = new DOMParser();
             \\ const newDoc = parser.parseFromString('<div id="new-node"><p>Hey</p><span>Marked</span></div>', 'text/html');
-         
             \\ const newNode = newDoc.getElementById('new-node');
-
             \\ const parent = document.getElementById('target-container');
             \\ const referenceNode = document.getElementById('reference-node');
-            
             \\ parent.insertBefore(newNode, referenceNode);
             \\ const k = document.getElementById('new-node');
             \\ const ptag = k.querySelector('p');
@@ -788,14 +786,14 @@ test "Browser.DOM.node.owner" {
             "[object HTMLDivElement]",
         },
 
-        .{"parent.ownerDocument === newNode.ownerDocument", "true" },
-        .{"parent.ownerDocument === anotherNewNode.ownerDocument", "true"},
-        .{"newNode.firstChild.nodeName", "P"},
-        .{"ptag.ownerDocument === parent.ownerDocument", "true"},
-        .{"spanTag.ownerDocument === parent.ownerDocument", "true"},
-        .{"parent.contains(newNode)", "true"},
-        .{"parent.contains(anotherNewNode)", "true"},
-        .{"anotherDoc.contains(anotherNewNode)", "false"},
-        .{"newDoc.contains(newNode)", "false"},
+        .{ "parent.ownerDocument === newNode.ownerDocument", "true" },
+        .{ "parent.ownerDocument === anotherNewNode.ownerDocument", "true" },
+        .{ "newNode.firstChild.nodeName", "P" },
+        .{ "ptag.ownerDocument === parent.ownerDocument", "true" },
+        .{ "spanTag.ownerDocument === parent.ownerDocument", "true" },
+        .{ "parent.contains(newNode)", "true" },
+        .{ "parent.contains(anotherNewNode)", "true" },
+        .{ "anotherDoc.contains(anotherNewNode)", "false" },
+        .{ "newDoc.contains(newNode)", "false" },
     }, .{});
 }