summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorandroid-build-prod (mdb) <android-build-team-robot@google.com>2017-11-10 01:16:40 +0000
committerGerrit Code Review <noreply-gerritcodereview@google.com>2017-11-10 01:16:40 +0000
commit165e353e520e19bf9ac19f3e3440bdcd1bbc09c9 (patch)
treebf04cfdc71ba8bfe00a615d0de19238e3509532a
parentda04aa1c0dc2c34d9c5cfa94f894607617027e49 (diff)
parent7b8a112bedd0f6213d229bc8a8c7c00941343dcd (diff)
downloadBluetooth-165e353e520e19bf9ac19f3e3440bdcd1bbc09c9.tar.gz
-rw-r--r--src/com/android/bluetooth/avrcp/Avrcp.java12
1 files changed, 10 insertions, 2 deletions
diff --git a/src/com/android/bluetooth/avrcp/Avrcp.java b/src/com/android/bluetooth/avrcp/Avrcp.java
index 08adf818d..61d7adf06 100644
--- a/src/com/android/bluetooth/avrcp/Avrcp.java
+++ b/src/com/android/bluetooth/avrcp/Avrcp.java
@@ -2043,7 +2043,12 @@ public final class Avrcp {
/* prepare media list & return the media player list response object */
private MediaPlayerListRsp prepareMediaPlayerRspObj() {
synchronized (mMediaPlayerInfoList) {
- int numPlayers = mMediaPlayerInfoList.size();
+ // TODO(apanicke): This hack will go away as soon as a developer
+ // option to enable or disable player selection is created. Right
+ // now this is needed to fix BMW i3 carkits and any other carkits
+ // that might try to connect to a player that isnt the current
+ // player based on this list
+ int numPlayers = 1;
int[] playerIds = new int[numPlayers];
byte[] playerTypes = new byte[numPlayers];
@@ -2058,7 +2063,10 @@ public final class Avrcp {
int players = mMediaPlayerInfoList.containsKey(mCurrAddrPlayerID) ? 1 : 0;
for (Map.Entry<Integer, MediaPlayerInfo> entry : mMediaPlayerInfoList.entrySet()) {
int idx = players;
- if (entry.getKey() == mCurrAddrPlayerID) idx = 0;
+ if (entry.getKey() == mCurrAddrPlayerID)
+ idx = 0;
+ else
+ continue; // TODO(apanicke): Remove, see above note
MediaPlayerInfo info = entry.getValue();
playerIds[idx] = entry.getKey();
playerTypes[idx] = info.getMajorType();