commit 4d544b6d551b7c69011980ca1242f7b33e48d60c
author Nathan Harold <nharold@google.com> Wed Jun 21 13:48:40 2023 -0700
committer Nathan Harold <nharold@google.com> Wed Jul 12 20:23:50 2023 +0000
tree 4ddd34f66d32f18aa4666d6c512ed5ee491c389d
parent bc463ec9b849a56c8e66129d590dab6baaeaaf61

Migrate OnSubscriptionsChangedListener to different Handler

Remove calls to the internal method of OnSubscriptionsChangedListener.
This causes a minor change to the threading since posts now go through
the same handler twice, but it should be benign and thread safer.

Bug: 278814050
Test: atest FrameworksTelephonyTests
Change-Id: I3081003ec6092334df9dca7833ed53b96d313b98

src/com/android/phone/ImsProvisioningController.java

diff --git a/src/com/android/phone/ImsProvisioningController.java b/src/com/android/phone/ImsProvisioningController.java
index a62980e..b2e34ae 100644
--- a/src/com/android/phone/ImsProvisioningController.java
+++ b/src/com/android/phone/ImsProvisioningController.java
@@ -839,7 +839,7 @@
         mSubscriptionManager = mApp.getSystemService(SubscriptionManager.class);
         mTelephonyRegistryManager = mApp.getSystemService(TelephonyRegistryManager.class);
         mTelephonyRegistryManager.addOnSubscriptionsChangedListener(
-                mSubChangedListener, mSubChangedListener.getHandlerExecutor());
+                mSubChangedListener, mHandler::post);
         mImsProvisioningLoader = imsProvisioningLoader;
 
         PhoneConfigurationManager.registerForMultiSimConfigChange(mHandler,