Tile events are not being cleaned up

gerhardmeyer commented 1 month ago

Checklist

[X] I've looked at the documentation to make sure the behavior isn't documented and expected.
[X] I'm sure this is an issue with Leaflet, not with my app or other dependencies (Angular, Cordova, React, etc.).
[X] I've searched through the current issues to make sure this hasn't been reported yet.
[X] I agree to follow the Code of Conduct that this project adheres to.

Steps to reproduce

I noticed when zoomin and out on the map the event listeners are growing quite quickly. It looks like the createTile function is creating events inside the TileLayer.js. I don't think these are cleaned up properly.

To reproduce just look at the event listerners inside Chromes performance monitor while zooming in and out. The events keep climbing.

Expected behavior

Events should be cleaned up when calling map.off() or map.remove().

Current behavior

"load" and "error" Events are leaking

Minimal example reproducing the issue

No response

Environment

Leaflet version: latest
Browser (with version): Chrome Version 123.0.6312.122
OS/Platform (with version): Windows 11

gerhardmeyer commented 1 month ago

Never mind its just a bit delayed.

lmdc45 commented 5 days ago

Noticed the issue as well. While the delay is no problems on android or desktop, it seems to induce some crash/map reload on iOS due maybe to more strict memory management

Leaflet / Leaflet