Fix memory leak by removing listener

[HarelM]

Launch Checklist

When using a profiler it was evident that the memory leak was coming from a unregistered event when removing the map.

• Memory leak on map.remove() #4811
• Memory Leak while interacting with maplibre-gl library #4650

Fixes #4811
Fixes #4650

• Confirm your changes do not include backports from Mapbox projects (unless with compliant license) - if you are not sure about this, please ask!
• Briefly describe the changes in this PR.
• Link to related issues.
• Include before/after visuals or gifs if this PR includes visual changes.
• Add an entry to CHANGELOG.md under the ## main section.

After - using 10 map remove.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.99%. Comparing base (c4b083f) to head (9b37b31).
Report is 3 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4824      +/-   ##
==========================================
+ Coverage   87.87%   87.99%   +0.11%     
==========================================
  Files         265      265              
  Lines       37590    37591       +1     
  Branches     2369     2339      -30     
==========================================
+ Hits        33034    33078      +44     
+ Misses       3508     3484      -24     
+ Partials     1048     1029      -19     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ibesora]

Nice catch!
Code changes look good and minimal.
Tested it via adding and removing a map and couldn’t see the memory increasing.

[elo58]

Amazing!
Thanks