googlemaps
diff --git a/‎.github/workflows/lint-report.yml
Copy file name to clipboard
+1-1Lines changed: 1 addition & 1 deletion b/‎.github/workflows/lint-report.yml
Copy file name to clipboard
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/‎README.md
Copy file name to clipboard
+4-22Lines changed: 4 additions & 22 deletions b/‎README.md
Copy file name to clipboard
+4-22Lines changed: 4 additions & 22 deletions
diff --git a/‎maps-compose/src/main/java/com/google/maps/android/compose/ComposeInfoWindowAdapter.kt
Copy file name to clipboard
+7-5Lines changed: 7 additions & 5 deletions b/‎maps-compose/src/main/java/com/google/maps/android/compose/ComposeInfoWindowAdapter.kt
Copy file name to clipboard
+7-5Lines changed: 7 additions & 5 deletions
diff --git a/‎maps-compose/src/main/java/com/google/maps/android/compose/GoogleMap.kt
Copy file name to clipboard
+13-85Lines changed: 13 additions & 85 deletions b/‎maps-compose/src/main/java/com/google/maps/android/compose/GoogleMap.kt
Copy file name to clipboard
+13-85Lines changed: 13 additions & 85 deletions
diff --git a/‎maps-compose/src/main/java/com/google/maps/android/compose/MapApplier.kt
Copy file name to clipboard
+3-2Lines changed: 3 additions & 2 deletions b/‎maps-compose/src/main/java/com/google/maps/android/compose/MapApplier.kt
Copy file name to clipboard
+3-2Lines changed: 3 additions & 2 deletions
diff --git a/‎maps-compose/src/main/java/com/google/maps/android/compose/MapComposeViewRender.kt
Copy file name to clipboard
+3-3Lines changed: 3 additions & 3 deletions b/‎maps-compose/src/main/java/com/google/maps/android/compose/MapComposeViewRender.kt
Copy file name to clipboard
+3-3Lines changed: 3 additions & 3 deletions
diff --git a/‎maps-compose/src/main/java/com/google/maps/android/compose/MapUpdater.kt
Copy file name to clipboard
+1-1Lines changed: 1 addition & 1 deletion b/‎maps-compose/src/main/java/com/google/maps/android/compose/MapUpdater.kt
Copy file name to clipboard
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/‎navigation-app/.gitignore
Copy file name to clipboard
-1Lines changed: 0 additions & 1 deletion b/‎navigation-app/.gitignore
Copy file name to clipboard
-1Lines changed: 0 additions & 1 deletion
@@ -41,7 +41,7 @@ jobs:
 
       - name: Merge SARIF files
         run: |
-          jq -s '{ "$schema": "https://json.schemastore.org/sarif-2.1.0", "version": "2.1.0", "runs": map(.runs) | add }'  maps-compose/build/reports/lint-results.sarif  maps-compose-utils/build/reports/lint-results.sarif  maps-compose-widgets/build/reports/lint-results.sarif  maps-app/build/reports/lint-results.sarif navigation-app/build/reports/lint-results.sarif > merged.sarif
+          jq -s '{ "$schema": "https://json.schemastore.org/sarif-2.1.0", "version": "2.1.0", "runs": map(.runs) | add }'  maps-compose/build/reports/lint-results.sarif  maps-compose-utils/build/reports/lint-results.sarif  maps-compose-widgets/build/reports/lint-results.sarif  maps-app/build/reports/lint-results.sarif > merged.sarif
 
       - name: Upload SARIF file
         uses: github/codeql-action/upload-sarif@v3
 
@@ -42,21 +42,15 @@ dependencies {
 
 ## Sample App
 
-Currently, there are two sample apps in the repository: [maps-app](maps-app) and [navigation-app](navigation-app). Each of them run a different version of the Android Maps Compose SDK, where either the maps or the navigation SDK are run, respectively.
+This repository includes a [sample app](maps-app).
 
-To run the maps demo app, ensure you've met the requirements above then:
+To run the demo app, ensure you've met the requirements above then:
 
 1. Open the `secrets.properties` file in your top-level directory, and then add the following code. Replace YOUR_API_KEY with your API key. Store your key in this file because secrets.properties is excluded from being checked into a version control system.
    If the `secrets.properties` file does not exist, create it in the same folder as the `local.default.properties` file.
    ```
    MAPS_API_KEY=YOUR_API_KEY
    ```
-   
-   If you want to use also the Navigation SDK, make sure the PLACES_API_KEY also contains an entry.
-   ```
-   PLACES_API_KEY=DEFAULT_API_KEY
-   ```
-   
 1. Build and run
 
 ## Documentation
@@ -397,18 +391,6 @@ GoogleMap(
 }
 ```
 
-## Using the Navigation SDK
-
-In order to use the Navigation SDK, make sure to include the Composable `NavigationScreen` as follows:
-
-```kotlin
-NavigationScreen(
-   modifier = Modifier.padding(innerPadding),
-   deviceLocation = location
-)
-```
-
-
 </details>
 
 ## Maps Compose Utility Library
@@ -444,7 +426,7 @@ The `fetchStreetViewData` method provides functionality to check whether a locat
 > Be sure to [enable Street View Static API](https://goo.gle/enable-sv-static-api) on the project associated with your API key.
 
 You can see example usage
-in the [`StreetViewActivity`](maps-app/src/main/java/com/google/maps/android/compose/StreetViewActivity.kt) of the demo app:
+in the [`StreetViewActivity`](https://github.com/googlemaps/android-maps-compose/blob/main/maps-app/src/main/java/com/google/maps/android/compose/StreetViewActivity.kt) of the demo app:
 
 ```kotlin
  streetViewResult =
@@ -507,7 +489,7 @@ This library is not a Google Maps Platform Core Service. Therefore, the Google M
 
 ## Support
 
-This library is offered via an open source [license]. It is not governed by the Google Maps Platform Support [Technical Support Services Guidelines], the [SLA], or the [Deprecation Policy]. However, any Google Maps Platform services used by the library remain subject to the Google Maps Platform Terms of Service.
+This library is offered via an open source [license]. It is not governed by the Google Maps Platform Support [Technical Support Services Guidelines, the SLA, or the [Deprecation Policy]. However, any Google Maps Platform services used by the library remain subject to the Google Maps Platform Terms of Service.
 
 This library adheres to [semantic versioning] to indicate when backwards-incompatible changes are introduced. Accordingly, while the library is in version 0.x, backwards-incompatible changes may be introduced at any time.
 
 
@@ -17,6 +17,7 @@ package com.google.maps.android.compose
 import android.view.View
 import androidx.compose.ui.platform.ComposeView
 import com.google.android.gms.maps.GoogleMap
+import com.google.android.gms.maps.MapView
 import com.google.android.gms.maps.model.Marker
 
 /**
@@ -35,7 +36,7 @@ import com.google.android.gms.maps.model.Marker
  * implementation should be updated.
  */
 internal class ComposeInfoWindowAdapter(
-    private val mapViewDelegate: AbstractMapViewDelegate<*>,
+    private val mapView: MapView,
     private val markerNodeFinder: (Marker) -> MarkerNode?
 ) : GoogleMap.InfoWindowAdapter {
 
@@ -45,10 +46,10 @@ internal class ComposeInfoWindowAdapter(
         if (content == null) {
             return null
         }
-        val view = ComposeView(mapViewDelegate.mapView.context).apply {
+        val view = ComposeView(mapView.context).apply {
             setContent { content(marker) }
         }
-        mapViewDelegate.renderComposeViewOnce(view, parentContext = markerNode.compositionContext)
+        mapView.renderComposeViewOnce(view, parentContext = markerNode.compositionContext)
         return view
     }
 
@@ -58,10 +59,11 @@ internal class ComposeInfoWindowAdapter(
         if (infoWindow == null) {
             return null
         }
-        val view = ComposeView(mapViewDelegate.mapView.context).apply {
+        val view = ComposeView(mapView.context).apply {
             setContent { infoWindow(marker) }
         }
-        mapViewDelegate.renderComposeViewOnce(view, parentContext = markerNode.compositionContext)
+        mapView.renderComposeViewOnce(view, parentContext = markerNode.compositionContext)
         return view
     }
+
 }
@@ -16,10 +16,10 @@ package com.google.maps.android.compose
 
 import android.content.ComponentCallbacks
 import android.content.ComponentCallbacks2
-import android.content.Context
 import android.content.res.Configuration
 import android.location.Location
 import android.os.Bundle
+import android.util.Log
 import android.view.View
 import androidx.compose.foundation.layout.Box
 import androidx.compose.foundation.layout.PaddingValues
@@ -36,15 +36,12 @@ import androidx.compose.runtime.rememberCoroutineScope
 import androidx.compose.runtime.rememberUpdatedState
 import androidx.compose.runtime.setValue
 import androidx.compose.ui.Modifier
-import androidx.compose.ui.platform.AbstractComposeView
-import androidx.compose.ui.platform.ComposeView
 import androidx.compose.ui.platform.LocalInspectionMode
 import androidx.compose.ui.viewinterop.AndroidView
 import androidx.lifecycle.Lifecycle
 import androidx.lifecycle.LifecycleEventObserver
 import androidx.lifecycle.LifecycleOwner
 import androidx.lifecycle.findViewTreeLifecycleOwner
-import com.google.android.gms.maps.GoogleMap
 import com.google.android.gms.maps.GoogleMapOptions
 import com.google.android.gms.maps.LocationSource
 import com.google.android.gms.maps.MapView
@@ -102,7 +99,6 @@ public fun GoogleMap(
     onPOIClick: ((PointOfInterest) -> Unit)? = null,
     contentPadding: PaddingValues = DefaultMapContentPadding,
     mapColorScheme: ComposeMapColorScheme? = null,
-    mapViewCreator: ((Context, GoogleMapOptions) -> AbstractMapViewDelegate<*>)? = null,
     content: @Composable @GoogleMapComposable () -> Unit = {},
 ) {
     // When in preview, early return a Box with the received modifier preserving layout
@@ -150,30 +146,19 @@ public fun GoogleMap(
     var subcompositionJob by remember { mutableStateOf<Job?>(null) }
     val parentCompositionScope = rememberCoroutineScope()
 
-    var delegate by remember {
-        mutableStateOf<AbstractMapViewDelegate<*>?>(null)
-    }
-
     AndroidView(
         modifier = modifier,
         factory = { context ->
-            if (mapViewCreator != null) {
-                mapViewCreator(context, googleMapOptionsFactory())
-            } else {
-                MapViewDelegate(MapView(context, googleMapOptionsFactory()))
-            }.also { mapViewDelegate: AbstractMapViewDelegate<*> ->
-                delegate = mapViewDelegate
-                val mapView = mapViewDelegate.mapView
-
+            MapView(context, googleMapOptionsFactory()).also { mapView ->
                 val componentCallbacks = object : ComponentCallbacks2 {
                     override fun onConfigurationChanged(newConfig: Configuration) {}
                     @Deprecated("Deprecated in Java", ReplaceWith("onTrimMemory(level)"))
-                    override fun onLowMemory() { mapViewDelegate.onLowMemory() }
-                    override fun onTrimMemory(level: Int) { mapViewDelegate.onLowMemory() }
+                    override fun onLowMemory() { mapView.onLowMemory() }
+                    override fun onTrimMemory(level: Int) { mapView.onLowMemory() }
                 }
                 context.registerComponentCallbacks(componentCallbacks)
 
-                val lifecycleObserver = MapLifecycleEventObserver(mapViewDelegate)
+                val lifecycleObserver = MapLifecycleEventObserver(mapView)
 
                 mapView.tag = MapTagData(componentCallbacks, lifecycleObserver)
 
@@ -195,21 +180,22 @@ public fun GoogleMap(
                 }
 
                 mapView.addOnAttachStateChangeListener(onAttachStateListener)
-            }.mapView
+            }
         },
         onReset = { /* View is detached. */ },
         onRelease = { mapView ->
-            val (componentCallbacks, lifecycleObserver) = delegate!!.tagData
+            val (componentCallbacks, lifecycleObserver) = mapView.tagData
             mapView.context.unregisterComponentCallbacks(componentCallbacks)
             lifecycleObserver.moveToDestroyedState()
             mapView.tag = null
         },
         update = { mapView ->
             if (subcompositionJob == null) {
+                Log.d("Gollum", "subcomposition running")
                 subcompositionJob = parentCompositionScope.launchSubcomposition(
                     mapUpdaterState,
                     parentComposition,
-                    delegate!!,
+                    mapView,
                     mapClickListeners,
                     currentContent,
                 )
@@ -225,15 +211,15 @@ public fun GoogleMap(
 private fun CoroutineScope.launchSubcomposition(
     mapUpdaterState: MapUpdaterState,
     parentComposition: CompositionContext,
-    mapViewDelegate: AbstractMapViewDelegate<*>,
+    mapView: MapView,
     mapClickListeners: MapClickListeners,
     content: @Composable @GoogleMapComposable () -> Unit,
 ): Job {
     // Use [CoroutineStart.UNDISPATCHED] to kick off GoogleMap loading immediately
     return launch(start = CoroutineStart.UNDISPATCHED) {
-        val map = mapViewDelegate.awaitMap()
+        val map = mapView.awaitMap()
         val composition = Composition(
-            applier = MapApplier(map, mapViewDelegate, mapClickListeners),
+            applier = MapApplier(map, mapView, mapClickListeners),
             parent = parentComposition
         )
 
@@ -324,65 +310,7 @@ public fun googleMapFactory(
     }
 }
 
-public interface AbstractMapViewDelegate<T : View> {
-    public fun onCreate(savedInstanceState: Bundle?)
-    public fun onStart()
-    public fun onResume()
-    public fun onPause()
-    public fun onStop()
-    public fun onLowMemory()
-    public fun onDestroy()
-    public suspend fun awaitMap(): GoogleMap
-    public fun renderComposeViewOnce(
-        view: AbstractComposeView,
-        parentContext: CompositionContext,
-        onAddedToWindow: ((View) -> Unit)? = null,
-    )
-
-    public fun startRenderingComposeView(
-        view: AbstractComposeView,
-        parentContext: CompositionContext,
-    ): ComposeUiViewRenderer.RenderHandle
-
-    public val mapView: T
-}
-
-private val <T : View> AbstractMapViewDelegate<T>.tagData: MapTagData
-    get() = mapView.tag as MapTagData
-
-public class MapViewDelegate(override val mapView: MapView) : AbstractMapViewDelegate<MapView> {
-    override fun onCreate(savedInstanceState: Bundle?): Unit = mapView.onCreate(savedInstanceState)
-    override fun onStart(): Unit = mapView.onStart()
-    override fun onResume(): Unit = mapView.onResume()
-    override fun onPause(): Unit = mapView.onPause()
-    override fun onStop(): Unit = mapView.onStop()
-    override fun onLowMemory(): Unit = mapView.onLowMemory()
-    override fun onDestroy(): Unit = mapView.onDestroy()
-    override suspend fun awaitMap(): GoogleMap = mapView.awaitMap()
-    override fun renderComposeViewOnce(
-        view: AbstractComposeView,
-        parentContext: CompositionContext,
-        onAddedToWindow: ((View) -> Unit)?
-    ) {
-        mapView.renderComposeViewOnce(
-            view = view,
-            parentContext = parentContext,
-            onAddedToWindow = onAddedToWindow
-        )
-    }
-
-    override fun startRenderingComposeView(
-        view: AbstractComposeView,
-        parentContext: CompositionContext
-    ): ComposeUiViewRenderer.RenderHandle {
-        return mapView.startRenderingComposeView(
-            view = view,
-            parentContext = parentContext,
-        )
-    }
-}
-
-private class MapLifecycleEventObserver(private val mapView: AbstractMapViewDelegate<*>) : LifecycleEventObserver {
+private class MapLifecycleEventObserver(private val mapView: MapView) : LifecycleEventObserver {
     private var currentLifecycleState: Lifecycle.State = Lifecycle.State.INITIALIZED
 
     override fun onStateChanged(source: LifecycleOwner, event: Lifecycle.Event) {
 
@@ -16,6 +16,7 @@ package com.google.maps.android.compose
 
 import androidx.compose.runtime.AbstractApplier
 import com.google.android.gms.maps.GoogleMap
+import com.google.android.gms.maps.MapView
 import com.google.android.gms.maps.model.Circle
 import com.google.android.gms.maps.model.GroundOverlay
 import com.google.android.gms.maps.model.Marker
@@ -37,7 +38,7 @@ private object MapNodeRoot : MapNode
 // for that particular listener; yet MapClickListeners never actually changes.
 internal class MapApplier(
     val map: GoogleMap,
-    internal val mapViewDelegate: AbstractMapViewDelegate<*>,
+    internal val mapView: MapView,
     val mapClickListeners: MapClickListeners,
 ) : AbstractApplier<MapNode>(MapNodeRoot) {
 
@@ -210,7 +211,7 @@ internal class MapApplier(
         })
         map.setInfoWindowAdapter(
             ComposeInfoWindowAdapter(
-                mapViewDelegate,
+                mapView,
                 markerNodeFinder = { marker ->
                     decorations.firstOrNull { it is MarkerNode && it.marker == marker }
                             as MarkerNode?
 
@@ -22,7 +22,7 @@ import java.io.Closeable
  * to a window. [onAddedToWindow] is called in place, and then [view] is removed from the window
  * before returning.
  */
-public fun MapView.renderComposeViewOnce(
+internal fun MapView.renderComposeViewOnce(
     view: AbstractComposeView,
     onAddedToWindow: ((View) -> Unit)? = null,
     parentContext: CompositionContext,
@@ -38,7 +38,7 @@ public fun MapView.renderComposeViewOnce(
  * to a window. A [ComposeUiViewRenderer.RenderHandle] is returned, which must be disposed after
  * this view no longer needs to render. Disposing removes [view] from the [MapView].
  */
-public fun MapView.startRenderingComposeView(
+internal fun MapView.startRenderingComposeView(
     view: AbstractComposeView,
     parentContext: CompositionContext,
 ): ComposeUiViewRenderer.RenderHandle {
@@ -69,7 +69,7 @@ private fun MapView.ensureContainerView(): NoDrawContainerView {
 @RestrictTo(RestrictTo.Scope.LIBRARY_GROUP)
 @Composable
 public fun rememberComposeUiViewRenderer(): ComposeUiViewRenderer {
-    val mapView = (currentComposer.applier as MapApplier).mapViewDelegate
+    val mapView = (currentComposer.applier as MapApplier).mapView
     val compositionContext = rememberCompositionContext()
 
     return remember(compositionContext) {
 
@@ -100,7 +100,7 @@ public val DefaultMapContentPadding: PaddingValues = PaddingValues()
 @Composable
 internal inline fun MapUpdater(mapUpdaterState: MapUpdaterState) = with(mapUpdaterState) {
     val map = (currentComposer.applier as MapApplier).map
-    val mapView = (currentComposer.applier as MapApplier).mapViewDelegate.mapView
+    val mapView = (currentComposer.applier as MapApplier).mapView
     if (mergeDescendants) {
         mapView.importantForAccessibility = IMPORTANT_FOR_ACCESSIBILITY_NO_HIDE_DESCENDANTS
     }
Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,7 @@ package com.google.maps.android.compose`
`17`	`17`	`import android.view.View`
`18`	`18`	`import androidx.compose.ui.platform.ComposeView`
`19`	`19`	`import com.google.android.gms.maps.GoogleMap`
	`20`	`+import com.google.android.gms.maps.MapView`
`20`	`21`	`import com.google.android.gms.maps.model.Marker`
`21`	`22`
`22`	`23`	`/**`
`@@ -35,7 +36,7 @@ import com.google.android.gms.maps.model.Marker`
`35`	`36`	`* implementation should be updated.`
`36`	`37`	`*/`
`37`	`38`	`internal class ComposeInfoWindowAdapter(`
`38`		`- private val mapViewDelegate: AbstractMapViewDelegate<*>,`
	`39`	`+ private val mapView: MapView,`
`39`	`40`	`private val markerNodeFinder: (Marker) -> MarkerNode?`
`40`	`41`	`) : GoogleMap.InfoWindowAdapter {`
`41`	`42`
`@@ -45,10 +46,10 @@ internal class ComposeInfoWindowAdapter(`
`45`	`46`	`if (content == null) {`
`46`	`47`	`return null`
`47`	`48`	`}`
`48`		`- val view = ComposeView(mapViewDelegate.mapView.context).apply {`
	`49`	`+ val view = ComposeView(mapView.context).apply {`
`49`	`50`	`setContent { content(marker) }`
`50`	`51`	`}`
`51`		`- mapViewDelegate.renderComposeViewOnce(view, parentContext = markerNode.compositionContext)`
	`52`	`+ mapView.renderComposeViewOnce(view, parentContext = markerNode.compositionContext)`
`52`	`53`	`return view`
`53`	`54`	`}`
`54`	`55`
`@@ -58,10 +59,11 @@ internal class ComposeInfoWindowAdapter(`
`58`	`59`	`if (infoWindow == null) {`
`59`	`60`	`return null`
`60`	`61`	`}`
`61`		`- val view = ComposeView(mapViewDelegate.mapView.context).apply {`
	`62`	`+ val view = ComposeView(mapView.context).apply {`
`62`	`63`	`setContent { infoWindow(marker) }`
`63`	`64`	`}`
`64`		`- mapViewDelegate.renderComposeViewOnce(view, parentContext = markerNode.compositionContext)`
	`65`	`+ mapView.renderComposeViewOnce(view, parentContext = markerNode.compositionContext)`
`65`	`66`	`return view`
`66`	`67`	`}`
	`68`	`+`
`67`	`69`	`}`
Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,7 @@ public val DefaultMapContentPadding: PaddingValues = PaddingValues()`
`100`	`100`	`@Composable`
`101`	`101`	`internal inline fun MapUpdater(mapUpdaterState: MapUpdaterState) = with(mapUpdaterState) {`
`102`	`102`	`val map = (currentComposer.applier as MapApplier).map`
`103`		`- val mapView = (currentComposer.applier as MapApplier).mapViewDelegate.mapView`
	`103`	`+ val mapView = (currentComposer.applier as MapApplier).mapView`
`104`	`104`	`if (mergeDescendants) {`
`105`	`105`	`mapView.importantForAccessibility = IMPORTANT_FOR_ACCESSIBILITY_NO_HIDE_DESCENDANTS`
`106`	`106`	`}`