OSDN Git Service

Re-enable browsing support for AVRCP
authorAjay Panicker <apanicke@google.com>
Wed, 5 Oct 2016 17:34:02 +0000 (10:34 -0700)
committerAjay Panicker <apanicke@google.com>
Wed, 5 Oct 2016 20:08:57 +0000 (13:08 -0700)
Bug: 31941615
Change-Id: Ia6846b3d6133c42863ae3cc630c980bea3e6bf20

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

index 8952b79..d90e77d 100644 (file)
@@ -63,7 +63,7 @@ import java.util.Map;
  ******************************************************************************/
 
 public final class Avrcp {
-    private static final boolean DEBUG = false;
+    private static final boolean DEBUG = true;
     private static final String TAG = "Avrcp";
     private static final String ABSOLUTE_VOLUME_BLACKLIST = "absolute_volume_blacklist";
 
@@ -1917,20 +1917,18 @@ public final class Avrcp {
      * @return true if it supports browsing, else false.
      */
     private synchronized boolean isBrowseSupported(String packageName) {
-        boolean response = false;
-
         /* check if Browsable Player's list contains this package name */
         for (BrowsePlayerInfo info : mBrowsePlayerInfoList) {
             if (info.packageName.equals(packageName)) {
-                // TODO: (apanicke) Currently browsing isn't implemented
-                // properly and causes metadata to break. Fix browsing
-                // interface and change this to true.
-                response = false;
+                if (DEBUG) Log.v(TAG, "isBrowseSupported for " + packageName +
+                                      ": true");
+                return true;
             }
         }
 
-        if (DEBUG) Log.v(TAG, "isBrowseSupported for " + packageName + ": " + response);
-        return response;
+        if (DEBUG) Log.v(TAG, "isBrowseSupported for " + packageName +
+                              ": false");
+        return false;
     }
 
     /* from the global object, getting the current addressed player's package name */