diff options
author | Tyler Gunn <tgunn@google.com> | 2020-03-25 17:49:35 -0700 |
---|---|---|
committer | Tyler Gunn <tgunn@google.com> | 2020-03-25 18:11:56 -0700 |
commit | cf2cd8dcd8cb38f489c7f02b5724aac2c1d61208 (patch) | |
tree | 881ef30398be20d739e506b66f01e9ef7dfd674c /tools/aapt2/java/AnnotationProcessor_test.cpp | |
parent | da8101d2701d1c05ebbf6ae6f066b2a84aaef51f (diff) |
Fix potential issue where sub info change listener is not registered.
SubscriptionManager#addOnSubscriptionsChangedListener can fail to
register a listener if the TELEPHONY_REGISTRY system service is not up.
Currently this is just silently ignored.
Adding a callback method on the listener to notify the registrant that the
listener failed to be registered, and adding exponential backoff code in
TelecomAccountRegistry to retry registration.
Test: Manual; edited code in Subscriptionmanager to fail the first attempts
to add a listener for TelecomAccountRegistry. Verified the backoff took
place and registration still occurred for the listener.
Test: Tried to write a mockito test but gave up because this code has
far too many intertwined dependencies and is not inherently testable
Fixes: 152217039
Change-Id: Icf3133cdeca526979cb621f29659b880127b680a
Diffstat (limited to 'tools/aapt2/java/AnnotationProcessor_test.cpp')
0 files changed, 0 insertions, 0 deletions