Why the events listener were not removed

[Zen33]

Stage.ts:
EVENTS.forEach(([event, methodName]) => {
this.content.addEventListener(
event,
(evt) => {
thismethodName;
},
{ passive: false }
);
});

Image.ts:
this._loadListener = () => {
this._requestDraw();
}

The events listener in the two files above were not removed, leading to memory leaks. In the memory snapshot of DevTools, there are a large number of Detached V8EventHandler instances.

[lavrton]

Can you make a demo when memory leak is visible?

[Zen33]

In Stage.ts at line 461, event listeners are added, but I couldn't find any code that removes these listeners. If the current element is repeatedly created, I understand that the event listeners may accumulate without being destroyed, leading to the existence of detached events. This is particularly relevant in the context of data-driven libraries like React. Is there a specific design intent behind not removing these event listeners, or are there other reasons for this? Please provide guidance. Thanks.

[lavrton]

As soon as the stage is removed from the DOM, garbage collection should remove the element and all its listeners automatically. Unless you have some tricky closers, keeping reference to the callback or node somewhere.
So it would be good if you can make a demo.