fix: prevent location permission crashes from FusedLocation API calls (#2653)

Co-authored-by: Nan <nan@onesignal.com>
Co-authored-by: Cursor <cursoragent@cursor.com>

Author: 3 people

OneSignalSDK/onesignal/location/src/main/java/com/onesignal/location/internal/controller/impl/FusedLocationApiWrapperImpl.kt

@@ -13,10 +13,14 @@ internal class FusedLocationApiWrapperImpl : IFusedLocationApiWrapper {
         googleApiClient: GoogleApiClient,
         locationListener: LocationListener,
     ) {
-        if (googleApiClient.isConnected) {
-            LocationServices.FusedLocationApi.removeLocationUpdates(googleApiClient, locationListener)
-        } else {
-            Logging.warn("GoogleApiClient is not connected. Unable to cancel location updates.")
+        try {
+            if (googleApiClient.isConnected) {
+                LocationServices.FusedLocationApi.removeLocationUpdates(googleApiClient, locationListener)
+            } else {
+                Logging.warn("GoogleApiClient is not connected. Unable to cancel location updates.")
+            }
+        } catch (t: Throwable) {
+            Logging.warn("FusedLocationApi.cancelLocationUpdates failed!", t)
         }
     }
 
@@ -38,8 +42,12 @@ internal class FusedLocationApiWrapperImpl : IFusedLocationApiWrapper {
     }
 
     override fun getLastLocation(googleApiClient: GoogleApiClient): Location? {
-        if (googleApiClient.isConnected) {
-            return LocationServices.FusedLocationApi.getLastLocation(googleApiClient)
+        try {
+            if (googleApiClient.isConnected) {
+                return LocationServices.FusedLocationApi.getLastLocation(googleApiClient)
+            }
+        } catch (t: Throwable) {
+            Logging.warn("FusedLocationApi.getLastLocation failed!", t)
         }
         return null
     }