commit 1983621106184ccb03370fe0523ff550a59d6220
author android-build-team Robot <android-build-team-robot@google.com> Thu Aug 23 03:06:34 2018 +0000
committer android-build-team Robot <android-build-team-robot@google.com> Thu Aug 23 03:06:34 2018 +0000
tree 0d9f2236313703143d8b0b6075abd5b881005550
parent 7b7ce469b6f88a2945a4905e676ccc33c551b200
parent 1ccb7c3489d6775821921ce97bf8b98eb7f7a70c

Snap for 4972147 from 1ccb7c3489d6775821921ce97bf8b98eb7f7a70c to pi-qpr1-release

Change-Id: Icfd7be2d5dd59b922d98f315ea4bcf5d46a9077c

device/Android.bp

diff --git a/device/Android.bp b/device/Android.bp
index f72a16b..0b7cda7 100644
--- a/device/Android.bp
+++ b/device/Android.bp
@@ -3,6 +3,7 @@
 cc_library_static {
     name: "libbtdevice",
     defaults: ["fluoride_defaults"],
+    host_supported: true,
     local_include_dirs: [
         "include",
     ],

profile/avrcp/Android.bp

diff --git a/profile/avrcp/Android.bp b/profile/avrcp/Android.bp
index 1e27861..3d95e9f 100644
--- a/profile/avrcp/Android.bp
+++ b/profile/avrcp/Android.bp
@@ -37,8 +37,11 @@
     static_libs: [
         "libgmock",
         "lib-bt-packets",
-        "libosi",
         "avrcp-target-service",
+        "libbtdevice",
+        "libosi",
+        "liblog",
+        "libcutils",
     ],
     sanitize: {
         cfi: false,

profile/avrcp/connection_handler.cc

diff --git a/profile/avrcp/connection_handler.cc b/profile/avrcp/connection_handler.cc
index 874bf77..66237ed 100644
--- a/profile/avrcp/connection_handler.cc
+++ b/profile/avrcp/connection_handler.cc
@@ -28,7 +28,9 @@
 // TODO (apanicke): Remove dependency on this header once we cleanup feature
 // handling.
 #include "bta/include/bta_av_api.h"
+#include "device/include/interop.h"
 #include "osi/include/allocator.h"
+#include "osi/include/properties.h"
 
 namespace bluetooth {
 namespace avrcp {
@@ -41,6 +43,20 @@
   return instance_;
 }
 
+bool IsAbsoluteVolumeEnabled(const RawAddress* bdaddr) {
+  char volume_disabled[PROPERTY_VALUE_MAX] = {0};
+  osi_property_get("persist.bluetooth.disableabsvol", volume_disabled, "false");
+  if (strncmp(volume_disabled, "true", 4) == 0) {
+    LOG(INFO) << "Absolute volume disabled by property";
+    return false;
+  }
+  if (interop_match_addr(INTEROP_DISABLE_ABSOLUTE_VOLUME, bdaddr)) {
+    LOG(INFO) << "Absolute volume disabled by IOP table";
+    return false;
+  }
+  return true;
+}
+
 bool ConnectionHandler::Initialize(const ConnectionCallback& callback,
                                    AvrcpInterface* avrcp, SdpInterface* sdp,
                                    VolumeInterface* vol) {
@@ -428,7 +444,9 @@
           if (categories & AVRC_SUPF_CT_CAT2) {
             LOG(INFO) << __PRETTY_FUNCTION__ << ": Device " << bdaddr.ToString()
                       << " supports advanced control";
-            peer_features |= (BTA_AV_FEAT_ADV_CTRL);
+            if (IsAbsoluteVolumeEnabled(&bdaddr)) {
+              peer_features |= (BTA_AV_FEAT_ADV_CTRL);
+            }
           }
           if (categories & AVRC_SUPF_CT_BROWSE) {
             LOG(INFO) << __PRETTY_FUNCTION__ << ": Device " << bdaddr.ToString()
@@ -467,7 +485,9 @@
           if (categories & AVRC_SUPF_CT_CAT2) {
             LOG(INFO) << __PRETTY_FUNCTION__ << ": Device " << bdaddr.ToString()
                       << " supports advanced control";
-            peer_features |= (BTA_AV_FEAT_ADV_CTRL);
+            if (IsAbsoluteVolumeEnabled(&bdaddr)) {
+              peer_features |= (BTA_AV_FEAT_ADV_CTRL);
+            }
           }
         }
       }