Prevent crash on alarms going off with a different user active #886
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This attempts to fix #876, which reports a SecurityException crash when a library Alarm goes off after somebody switched users on the phone. The fix is to detect the user switch and cancel the alarm.
Testing this on the Google Pixel 3a, I start the reference app (with a foreground service enabled), then switch to the Guest User. I see that the logs show the alarm gets cancelled:
As a side note, this test showed some fascinating things about how the library behaves after user switching, at lest on a Pixel 3a with Android 9.0 and the library running with a foreground service:
Maybe the above is expected behavior, but I was shocked to see that all work! I figured I'd note it for the record.