Attempt to unhold remaining holding call on disconnect (1/2)
Bug: 17480900
Change-Id: I9d16419a969f833aa830de7f9ab5df8772a935c4
diff --git a/src/com/android/server/telecom/CallsManager.java b/src/com/android/server/telecom/CallsManager.java
index dc84db3..b8ba4e1 100644
--- a/src/com/android/server/telecom/CallsManager.java
+++ b/src/com/android/server/telecom/CallsManager.java
@@ -37,6 +37,7 @@
import java.util.Arrays;
import java.util.Collections;
+import java.util.HashSet;
import java.util.List;
import java.util.Objects;
import java.util.Set;
@@ -117,6 +118,7 @@
private final Context mContext;
private final PhoneAccountRegistrar mPhoneAccountRegistrar;
private final MissedCallNotifier mMissedCallNotifier;
+ private final Set<Call> mLocallyDisconnectingCalls = new HashSet<>();
/**
* The call the user is currently interacting with. This is the call that should have audio
@@ -566,6 +568,7 @@
if (!mCalls.contains(call)) {
Log.w(this, "Unknown call (%s) asked to disconnect", call);
} else {
+ mLocallyDisconnectingCalls.add(call);
call.disconnect();
}
}
@@ -704,7 +707,6 @@
void markCallAsDisconnected(Call call, DisconnectCause disconnectCause) {
call.setDisconnectCause(disconnectCause);
setCallState(call, CallState.DISCONNECTED);
- removeCall(call);
}
/**
@@ -712,6 +714,12 @@
*/
void markCallAsRemoved(Call call) {
removeCall(call);
+ if (mLocallyDisconnectingCalls.contains(call)) {
+ mLocallyDisconnectingCalls.remove(call);
+ if (mForegroundCall != null && mForegroundCall.getState() == CallState.ON_HOLD) {
+ mForegroundCall.unhold();
+ }
+ }
}
/**
@@ -988,9 +996,7 @@
Log.v(this, "Updating foreground call, %s -> %s.", mForegroundCall, newForegroundCall);
Call oldForegroundCall = mForegroundCall;
mForegroundCall = newForegroundCall;
- if (mForegroundCall != null && mForegroundCall.getState() == CallState.ON_HOLD) {
- mForegroundCall.unhold();
- }
+
for (CallsManagerListener listener : mListeners) {
listener.onForegroundCallChanged(oldForegroundCall, mForegroundCall);
}