Fix: Catch IllegalStateException on call to Google's location library

[jennantilla]

Description

One Line Summary

Catch IllegalStateException on call to Google's location library to avoid crash.

Details

Motivation

IllegalStateException was reported in #1821, potentially due to and issue with a device's "Google Play services" app. Catching these exceptions and logging them will avoid crashes on calls to Google's location library.

OPTIONAL - Other

While the original crash was not reproducible, adding this try/catch block to log the exception should prevent the app from crashing.

Manual testing

Since crash was not reproducible, just tested running app to ensure that change does not impact location sharing on Android.

Affected code checklist

• Notifications
  • Display
  • Open
  • Push Processing
  • Confirm Deliveries
• Outcomes
• Sessions
• In-App Messaging
• REST API requests
• Public API changes

Checklist

Overview

• I have filled out all REQUIRED sections above
• PR does one thing
  • If it is hard to explain how any codes changes are related to each other then it most likely needs to be more than one PR
• Any Public API changes are explained in the PR details and conform to existing APIs

Testing

• I have included test coverage for these changes, or explained why they are not needed
• All automated tests pass, or I explained why that is not possible
• I have personally tested this on my device, or explained why that is not possible

Final pass

• Code is as readable as possible.
  • Simplify with less code, followed by splitting up code into well named functions and variables, followed by adding comments to the code.
• I have reviewed this PR myself, ensuring it meets each checklist item
  • WIP (Work In Progress) is ok, but explain what is still in progress and what you would like feedback on. Start the PR title with "WIP" to indicate this.

---

This change is 

[nan-li]

Are there other reports of IllegalStateException aside from GoogleApiClient is not connected yet?

I think it may be clearer to check for googleApiClient.isConnected instead of using a try-catch, such as in the getLastLocation() or refreshRequest() method.

        fun getLastLocation(googleApiClient: GoogleApiClient): Location? {
            if (googleApiClient.isConnected) {
                return LocationServices.FusedLocationApi.getLastLocation(googleApiClient)
            }
            return null
        }

        private fun refreshRequest() {
            if (!googleApiClient.isConnected) {
                Logging.warn("Attempt to refresh location request but not currently connected!")
                return
            }
            // ... stuff

[jennantilla]

I've updated the method per @nan-li's suggestion. Let me know if this looks good! cc @jkasten2