mirror/browser
1
0
Fork 0
mirror of https://github.com/lightpanda-io/browser.git synced 2026-06-11 17:46:32 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2135 from lightpanda-io/mo_cleanup

Browse Source 
On Page cleanup, capture next linked list node _before_ releasing MO
This commit is contained in:
Karl Seguin
2026-04-13 22:49:59 +08:00
committed by GitHub
parent 9a05fd8af4 da8d206c52
commit 2f2f74ef4b
3 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/browser/Page.zig
View File

@@ -365,9 +365,10 @@ pub fn deinit(self: *Page, abort_http: bool) void {
}
{
var it: ?*std.DoublyLinkedList.Node = self._mutation_observers.first;
while (it) |node| : (it = node.next) {
const observer: *MutationObserver = @fieldParentPtr("node", node);
var node: ?*std.DoublyLinkedList.Node = self._mutation_observers.first;
while (node) |n| {
node = n.next; // capture before we potentially delete observer
const observer: *MutationObserver = @fieldParentPtr("node", n);
observer.releaseRef(session);
}
}

2
src/browser/webapi/IntersectionObserver.zig
View File

@@ -71,7 +71,7 @@ pub const ObserverInit = struct {
};
pub fn init(callback: js.Function.Temp, options: ?ObserverInit, page: *Page) !*IntersectionObserver {
const arena = try page.getArena(.medium, "IntersectionObserver");
const arena = try page.getArena(.small, "IntersectionObserver");
errdefer page.releaseArena(arena);
const opts = options orelse ObserverInit{};

3
src/browser/webapi/MutationObserver.zig
View File

@@ -76,9 +76,8 @@ pub const ObserveOptions = struct {
};
pub fn init(callback: js.Function.Temp, page: *Page) !*MutationObserver {
const arena = try page.getArena(.medium, "MutationObserver");
const arena = try page.getArena(.small, "MutationObserver");
errdefer page.releaseArena(arena);
const self = try arena.create(MutationObserver);
self.* = .{
._arena = arena,