From 09f73b32a68a51146b55f970fe8a1b389a5ef650 Mon Sep 17 00:00:00 2001
From: Harlan Wilton <harlan@harlanzw.com>
Date: Thu, 19 Mar 2026 02:28:07 +1100
Subject: [PATCH] fix: prevent memory leak in
 ScriptGoogleMapsAdvancedMarkerElement

Track unmounted state so async marker creation is aborted if the
component unmounts before `importLibrary('marker')` resolves.
Previously the marker would be created and attached to the map
with no owner to clean it up.

Resolves #646
---
 .../GoogleMaps/ScriptGoogleMapsAdvancedMarkerElement.vue | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/runtime/components/GoogleMaps/ScriptGoogleMapsAdvancedMarkerElement.vue b/src/runtime/components/GoogleMaps/ScriptGoogleMapsAdvancedMarkerElement.vue
index 5a7ba91b6..2aaa65c0d 100644
--- a/src/runtime/components/GoogleMaps/ScriptGoogleMapsAdvancedMarkerElement.vue
+++ b/src/runtime/components/GoogleMaps/ScriptGoogleMapsAdvancedMarkerElement.vue
@@ -1,7 +1,7 @@
 <script lang="ts">
 import type { InjectionKey, ShallowRef } from 'vue'
 import { whenever } from '@vueuse/core'
-import { inject, onUnmounted, provide, shallowRef } from 'vue'
+import { inject, onUnmounted, provide, ref, shallowRef } from 'vue'
 import { MAP_INJECTION_KEY } from './ScriptGoogleMaps.vue'
 import { MARKER_CLUSTERER_INJECTION_KEY } from './ScriptGoogleMapsMarkerClusterer.vue'
 
@@ -51,10 +51,15 @@ const mapContext = inject(MAP_INJECTION_KEY, undefined)
 const markerClustererContext = inject(MARKER_CLUSTERER_INJECTION_KEY, undefined)
 
 const advancedMarkerElement = shallowRef<google.maps.marker.AdvancedMarkerElement | undefined>(undefined)
+const isUnmounted = ref(false)
 
 whenever(() => mapContext?.map.value && mapContext.mapsApi.value, async () => {
   await mapContext!.mapsApi.value!.importLibrary('marker')
 
+  // component was unmounted while awaiting the library import
+  if (isUnmounted.value)
+    return
+
   advancedMarkerElement.value = new mapContext!.mapsApi.value!.marker.AdvancedMarkerElement(props.options)
 
   setupAdvancedMarkerElementEventListeners(advancedMarkerElement.value)
@@ -79,6 +84,8 @@ whenever(() => mapContext?.map.value && mapContext.mapsApi.value, async () => {
 })
 
 onUnmounted(() => {
+  isUnmounted.value = true
+
   if (!advancedMarkerElement.value || !mapContext?.mapsApi.value) {
     return
   }