diff options
author | Brad Ebinger <breadley@google.com> | 2015-11-12 17:25:06 -0800 |
---|---|---|
committer | Brad Ebinger <breadley@google.com> | 2015-12-07 16:57:44 -0800 |
commit | f19000764f6b93dfa15561342cbdd9c1aa1d3e22 (patch) | |
tree | 3ef4615c51049397d34415e5595f08ddf8a3a648 /src/com/android/server/telecom/CallIdMapper.java | |
parent | 1b7b2584c8cde1a12abe7e168801219b01d598d6 (diff) | |
download | android_packages_services_Telecomm-f19000764f6b93dfa15561342cbdd9c1aa1d3e22.tar.gz android_packages_services_Telecomm-f19000764f6b93dfa15561342cbdd9c1aa1d3e22.tar.bz2 android_packages_services_Telecomm-f19000764f6b93dfa15561342cbdd9c1aa1d3e22.zip |
Refactor CreateConnectionProcessor
Currently, if a ConnectionManager refuses a connection from a Call, this
code will set a flag and call process() again. This finds all of
possible PhoneAccounts again (without the addition of the
ConnectionManager). In order to avoid this redundancy, a queue is utilized to
have the CreateConnectionProcessor fall back to another account if the
connection fails.
This change only changes how a ConnectionManager is being found/used.
Also, tests have been added for CreateConnectionProcessor
Change-Id: Id7479e5337a812fdae83411fb2fbc248d8c1a7da
Diffstat (limited to 'src/com/android/server/telecom/CallIdMapper.java')
-rw-r--r-- | src/com/android/server/telecom/CallIdMapper.java | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/com/android/server/telecom/CallIdMapper.java b/src/com/android/server/telecom/CallIdMapper.java index f3a3a440..b097bea1 100644 --- a/src/com/android/server/telecom/CallIdMapper.java +++ b/src/com/android/server/telecom/CallIdMapper.java @@ -18,10 +18,13 @@ package com.android.server.telecom; import android.util.ArrayMap; +import com.android.internal.annotations.VisibleForTesting; + import java.util.Map; /** Utility to map {@link Call} objects to unique IDs. IDs are generated when a call is added. */ -class CallIdMapper { +@VisibleForTesting +public class CallIdMapper { /** * A very basic bidirectional map. */ |