public/browser
1
0
Fork 0
mirror of https://github.com/lightpanda-io/browser.git synced 2026-02-04 14:33:47 +00:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

handle Node self replacement in insertBefore

Browse Source
This commit is contained in:
Pierre Tachoire
2026-01-06 09:49:08 +01:00
parent 0b221615b7
commit b164ffeb95
1 changed files with 20 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

36
src/browser/webapi/Node.zig
View File

@@ -457,6 +457,21 @@ pub fn insertBefore(self: *Node, new_node: *Node, ref_node_: ?*Node, page: *Page
return self.appendChild(new_node, page); return self.appendChild(new_node, page);
}; };
// special case: if nodes are the same, ignore the change.
if (new_node == ref_node_) {
page.domChanged();
if (page.hasMutationObservers()) {
const parent = new_node._parent.?;
const previous_sibling = new_node.previousSibling();
const next_sibling = new_node.nextSibling();
const replaced = [_]*Node{new_node};
page.childListChange(parent, &replaced, &replaced, previous_sibling, next_sibling);
}
return new_node;
}
if (ref_node._parent == null or ref_node._parent.? != self) { if (ref_node._parent == null or ref_node._parent.? != self) {
return error.NotFound; return error.NotFound;
} }
@@ -500,23 +515,14 @@ pub fn replaceChild(self: *Node, new_child: *Node, old_child: *Node, page: *Page
try validateNodeInsertion(self, new_child); try validateNodeInsertion(self, new_child);
// special case: we replace a node by itself
if (new_child == old_child) {
page.domChanged();
if (page.hasMutationObservers()) {
const parent = new_child._parent.?;
const previous_sibling = new_child.previousSibling();
const next_sibling = new_child.nextSibling();
const replaced = [_]*Node{new_child};
page.childListChange(parent, &replaced, &replaced, previous_sibling, next_sibling);
}
return old_child;
}
_ = try self.insertBefore(new_child, old_child, page); _ = try self.insertBefore(new_child, old_child, page);
// Special case: if we replace a node by itself, we don't remove it.
// insertBefore is an noop in this case.
if (new_child != old_child) {
page.removeNode(self, old_child, .{ .will_be_reconnected = false }); page.removeNode(self, old_child, .{ .will_be_reconnected = false });
}
return old_child; return old_child;
} }