Inappropriate way of sending locations is used

The Signal app currently uses the following method of sharing a user's location, after the user has selected a location on the in-app map.

1. Load Google Map.
2. Wait for Google Map to finish loading.
3. Take snapshot of Google Map.
4. Remove Google Map and replace it with its snapshot.
5. Once the user hits send, send this snapshot as an image.

This is implemented in the following snippet:

https://github.com/signalapp/Signal-Android/blob/55af6ca84e32683662a96f9f621728d47fabae9e/app/src/main/java/org/thoughtcrime/securesms/components/location/SignalMapView.java#L77-L96

The two issues are:

• 1.-4. is an inappropriate way of generating a snapshot. It is better to use a lite mode map instead, because this implements exactly what is needed (non-interactive snapshot of a map).
  • An instance in which the current implementation is inappropriate is when onMapLoaded and onSnapshotReady have not yet been called, but the activity hosting the map is destroyed. In this case, the activity's onDestroy method must call the map's onDestroy method (which is not implemented). Lite mode maps do not expect lifecycle methods to be called, except for onCreate.
  • In fact, Signal currently forgets to call a few of the lifecycle methods. See https://github.com/signalapp/Signal-Android/pull/13008.
  • Additional info: this is also related to https://github.com/signalapp/Signal-Android/issues/12723#issuecomment-1595810834, because microG does not implement GoogleMap.OnMapLoadedCallback correctly (https://github.com/microg/GmsCore/issues/1960).
• ⁠5. is in violation of the contract of the snapshot API, and by extension likely also of the terms of usage of the Google Maps API.

  Note: Images of the map must not be transmitted to your servers, or otherwise used outside of the application. If you need to send a map to another application or user, send data that allows them to reconstruct the map for the new user instead of a snapshot.

Read SignalMapView.java around lines 77-96 and the linked Google Maps Lite mode lifecycle guidance. Compare the current snapshot flow with the location-send path and review PR #13008 for related lifecycle omissions. Done means the map-sharing flow no longer transmits a map snapshot and follows the documented API constraints.