From 12e81b98f1033350a06e29f4116b50abc29c5823 Mon Sep 17 00:00:00 2001 From: Nathan Harold Date: Wed, 21 Jun 2023 13:30:40 -0700 Subject: [PATCH] Switch OnSubscriptionsChangedListener to use existing Handler Use the main Handler for DevicePolicyManagerService to post the callback for OnSubscriptionsChanged to move away from using the hidden (internal) handler. Bug: 278814050 Test: atest FrameworksTelephonyTests Test: atest SubscriptionManagerTest#testCreateSubscriptionChangedListenerWithoutLooper Change-Id: I347408af8cd6cbcd3a25facc056c7c2541d5b428 --- .../android/server/devicepolicy/DevicePolicyManagerService.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java b/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java index 7857b259e2a32..77f8de57f3aec 100644 --- a/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java +++ b/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java @@ -24132,7 +24132,7 @@ public class DevicePolicyManagerService extends IDevicePolicyManager.Stub { // When listener is added onSubscriptionsChanged gets called immediately for once // (even if subscriptions are not changed) and later on when subscriptions changes. subscriptionManager.addOnSubscriptionsChangedListener( - mSubscriptionsChangedListener.getHandlerExecutor(), + mHandler::post, mSubscriptionsChangedListener); } finally { mInjector.binderRestoreCallingIdentity(id);