Stop an IllegalStateException in rare conditions

Bug fixed by inspection after a single report in production. Because
there are multiple threads involved, but the
OfflineLocationTimeZoneDelegate is single threaded, there's probably a
race after location listening is stopped when the delivery of a location
can still occur (i.e. one that has been blocked while the delegate was
transitioning into the stopped state).

This removes the IllegalStateException throw. All callers to the method
have a return statement immediately afterwards, so this won't
significantly alter behavior, just prevent the exception being thrown.

Test: atest OfflineLocationTimeZoneProviderTests
Test: build / boot
Bug: 188743586
Change-Id: I75ebe7a1847fd30fe6defa73b50fa1e25b9c1f24
1 file changed