commit f129460cc6fd4ab2309f969da5a30ee0a1a5509a
author Cassie(Yitong) Wang <cassieyw@google.com> Mon Aug 12 18:34:37 2019 -0700
committer Cassie(Yitong) Wang <cassieyw@google.com> Fri Aug 16 16:19:06 2019 -0700
tree 36a426673620cc4bcd3795f0ceaec790be23551b
parent 7e432b0cff08cf71addc729c94e1236dc201a773

Fix Dialer crash

Bug: 139378568
Test: Manually
Change-Id: I0150fdf02b285d0e42433c4d5bf1c8e6723585c6

src/com/android/car/dialer/telecom/InCallRouter.java

diff --git a/src/com/android/car/dialer/telecom/InCallRouter.java b/src/com/android/car/dialer/telecom/InCallRouter.java
index 0704470..9447cd4 100644
--- a/src/com/android/car/dialer/telecom/InCallRouter.java
+++ b/src/com/android/car/dialer/telecom/InCallRouter.java
@@ -21,6 +21,8 @@
 import android.os.Handler;
 import android.telecom.Call;
 
+import androidx.annotation.MainThread;
+
 import com.android.car.dialer.log.L;
 import com.android.car.dialer.notification.InCallNotificationController;
 import com.android.car.dialer.ui.activecall.InCallActivity;
@@ -95,13 +97,15 @@
         }
     }
 
+    @MainThread
     void registerActiveCallListChangedCallback(
             InCallServiceImpl.ActiveCallListChangedCallback callback) {
-        mMainHandler.post(() -> mActiveCallListChangedCallbacks.add(callback));
+        mActiveCallListChangedCallbacks.add(callback);
     }
 
+    @MainThread
     void unregisterActiveCallHandler(InCallServiceImpl.ActiveCallListChangedCallback callback) {
-        mMainHandler.post(() -> mActiveCallListChangedCallbacks.remove(callback));
+        mActiveCallListChangedCallbacks.remove(callback);
     }
 
     /** Dispatches the call to {@link InCallServiceImpl.ActiveCallListChangedCallback}. */