commit b574d5f3037186539bd2f222534af2753a1b4ef8
author Hall Liu <hallliu@google.com> Thu Aug 29 13:21:26 2019 -0700
committer Hall Liu <hallliu@google.com> Thu Aug 29 13:41:03 2019 -0700
tree 5abd626faf213497e4749c88ae74bae94c859702
parent cd4a6376e1e6033679ed56a7a6ed9ba9eeeeb38c

Fix call to ImsManager#getInstance

Previous CL had passed in the subId to ImsManager#getInstance instead of
phoneId. This is incorrect and causes the later call to
ImsManager#setRttEnabled to be a no-op. Fix this by correctly passing in
phoneId.

Fixes: 139710080
Test: manual
Change-Id: Ica8c6a0a9634611a82c208d59040dd7a3c95b3fc

src/com/android/phone/settings/AccessibilitySettingsFragment.java

diff --git a/src/com/android/phone/settings/AccessibilitySettingsFragment.java b/src/com/android/phone/settings/AccessibilitySettingsFragment.java
index b074f20..73f5de9 100644
--- a/src/com/android/phone/settings/AccessibilitySettingsFragment.java
+++ b/src/com/android/phone/settings/AccessibilitySettingsFragment.java
@@ -166,7 +166,8 @@
             for (int subId : SubscriptionController.getInstance().getActiveSubIdList(true)) {
                 if (!configManager.getConfigForSubId(subId).getBoolean(
                         CarrierConfigManager.KEY_IGNORE_RTT_MODE_SETTING_BOOL, false)) {
-                    ImsManager imsManager = ImsManager.getInstance(getContext(), subId);
+                    int phoneId = SubscriptionController.getInstance().getPhoneId(subId);
+                    ImsManager imsManager = ImsManager.getInstance(getContext(), phoneId);
                     imsManager.setRttEnabled(mButtonRtt.isChecked());
                 }
             }