Remove legacy GpsStatus listener

[barbeau]

Describe the bug
Reported 23 times in the Google Play Developer Console over the last 30 days:

java.lang.ArrayIndexOutOfBoundsException: 
  at com.android.gpstest.view.GpsSkyView.setSats (GpsSkyView.java:316)
  at com.android.gpstest.GpsSkyFragment.onGpsStatusChanged (GpsSkyFragment.java:189)
  at com.android.gpstest.GpsTestActivity$4.onGpsStatusChanged (GpsTestActivity.java:1125)
  at android.location.LocationManager$GnssStatusListenerManager$1.onSatelliteStatusChanged (LocationManager.java:2958)
  at android.location.LocationManager$GnssStatusListenerManager$GnssStatusListener.lambda$onSvStatusChanged$1 (LocationManager.java:3014)
  at android.location.-$$Lambda$LocationManager$GnssStatusListenerManager$GnssStatusListener$4EPi22o4xuVnpNhFHnDvebH4TG8.accept (Unknown Source:4)
  at android.location.AbstractListenerManager$Registration.accept (AbstractListenerManager.java:83)
  at android.location.AbstractListenerManager$Registration.lambda$XpiThbVaDDpOnFWIkrt38Bf4yx0 (Unknown Source)
  at android.location.-$$Lambda$AbstractListenerManager$Registration$XpiThbVaDDpOnFWIkrt38Bf4yx0.accept (Unknown Source:4)
  at com.android.internal.util.function.pooled.PooledLambdaImpl.doInvoke (PooledLambdaImpl.java:278)
  at com.android.internal.util.function.pooled.PooledLambdaImpl.invoke (PooledLambdaImpl.java:201)
  at com.android.internal.util.function.pooled.OmniFunction.run (OmniFunction.java:97)
  at android.os.Handler.handleCallback (Handler.java:938)
  at android.os.Handler.dispatchMessage (Handler.java:99)
  at android.os.Looper.loop (Looper.java:236)
  at android.app.ActivityThread.main (ActivityThread.java:8057)
  at java.lang.reflect.Method.invoke (Native Method)
  at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run (RuntimeInit.java:656)
  at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:967)

To Reproduce
Unknown

Expected behavior
Don't crash

Observed behavior
Crash

App, Device and Android version:

All crashes are with GPSTest version 18076 (v3.8.4) and all on Android 11. Devices are Samsung Galaxy M31, Xiaomi Mi 10T, Xiaomi Mi Note 10 Lite, Samsung Galaxy Z Flip 5G

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[barbeau]

Looks like this is a bug in Android 11 that's triggered when the user unchecks the GPSTest "Use GNSS APIs" setting within the app, which then triggers the legacy onGpsStatusChanged() API.

That setting was introduced during the transition between Android 6 and 7 and can be removed now. Less than 3% of users are on devices less than Android 7, so going forward with the architectural changes required for #313 and #299 I may look at removing these listeners entirely and supporting only Android 7 and up going forward.