commit cb13e7c56b13582e976e396e69ec5a7a203d7c9c
author Hemant Gupta <hemantg@codeaurora.org> Wed May 17 17:29:01 2017 +0530
committer Hemant Gupta <hemantg@codeaurora.org> Wed May 17 17:30:06 2017 +0530
tree fd12155ba3719ee66decf776782ef86223c0eba1
parent 77bb5f0924034f9124e11e0d074ec731293b48c4

AVRCP: Fix returned attributes for Genre/Play Time

Usecase:

1) Connect DUT with remote device supporting AVRCP 1.4
2) Send request to retrieve genre or play time attributes.

Observation:
DUT returns incorrect attribute id

Root cause:
Missing break in code leading to corruption of values of attributes returned.

Fix:
Add break statement missing in code.

Test: Checked that genre and play time attributes are returned correctly by DUT.

Bug: 38372332
Change-Id: I0ec2f3b8bfc6668819a207046ceb65659037b368

src/com/android/bluetooth/avrcp/BrowsedMediaPlayer.java

diff --git a/src/com/android/bluetooth/avrcp/BrowsedMediaPlayer.java b/src/com/android/bluetooth/avrcp/BrowsedMediaPlayer.java
index ddcb1a0..da55714 100644
--- a/src/com/android/bluetooth/avrcp/BrowsedMediaPlayer.java
+++ b/src/com/android/bluetooth/avrcp/BrowsedMediaPlayer.java
@@ -601,9 +601,11 @@
 
                 case AvrcpConstants.ATTRID_GENRE:
                     attrValue = extras.getString(MediaMetadata.METADATA_KEY_GENRE);
+                    break;
 
                 case AvrcpConstants.ATTRID_PLAY_TIME:
                     attrValue = extras.getString(MediaMetadata.METADATA_KEY_DURATION);
+                    break;
 
                 case AvrcpConstants.ATTRID_COVER_ART:
                     Log.e(TAG, "getAttrValue: Cover art attribute not supported");