commit 38a614c99ef962b2514b7818ee268d5206efed4d
author Jordan Liu <jminjie@google.com> Wed Mar 20 15:01:17 2019 -0700
committer Jordan Liu <jminjie@google.com> Wed Mar 20 15:01:17 2019 -0700
tree bfbff6b5647e2d88a0be319da71a8c19ecb7b291
parent 69cfbfcc7b7247e53d5b459f85f7015f5de0bf95

Fix usage of obtainMessage()

Instead of invoking obtainMessage(what, obj), make sure to pass the
phoneId along as arg1 by including arg2=-1. This solves the issue of
loading the carrier config from the carrier app on mSIM devices (with
phoneIds other than 0).

Bug: 122365375
Test: manual
Change-Id: Ie4ffe1532610dcee90523978b0368aba088e0398

src/com/android/phone/CarrierConfigLoader.java

diff --git a/src/com/android/phone/CarrierConfigLoader.java b/src/com/android/phone/CarrierConfigLoader.java
index 137d3d1..6a3e928 100644
--- a/src/com/android/phone/CarrierConfigLoader.java
+++ b/src/com/android/phone/CarrierConfigLoader.java
@@ -324,7 +324,7 @@
                     final String carrierPackageName = getCarrierPackageForPhoneId(phoneId);
                     if (carrierPackageName != null) {
                         log("Found carrier config app: " + carrierPackageName);
-                        sendMessage(obtainMessage(EVENT_DO_FETCH_CARRIER, phoneId));
+                        sendMessage(obtainMessage(EVENT_DO_FETCH_CARRIER, phoneId, -1));
                     } else {
                         broadcastConfigChangedIntent(phoneId);
                     }