#4268 fix async race when adding/removing geolocate rapidly

[nkundu]

Attempt to fix #4268 via the suggested method of initially creating the icon in the disabled state, then updating it to active in the callback.

Launch Checklist

• 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.
• Write tests for all new functionality.
• Document any changes to public APIs.
• Post benchmark scores.
• Add an entry to CHANGELOG.md under the ## main section.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.73%. Comparing base (c0f1936) to head (4265456).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4454      +/-   ##
==========================================
- Coverage   87.79%   87.73%   -0.06%     
==========================================
  Files         246      246              
  Lines       33404    33412       +8     
  Branches     2184     2223      +39     
==========================================
- Hits        29327    29314      -13     
+ Misses       3088     3077      -11     
- Partials      989     1021      +32     

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

[HarelM]

THANKS!