From 11ab928812fa8c7bcc78aeff3b1d2f11598d7c47 Mon Sep 17 00:00:00 2001
From: Ryan Mitchell <rtmitchell@google.com>
Date: Thu, 10 Oct 2019 08:15:55 -0700
Subject: [PATCH] santoni: Use component-override to configure crosshatch
 Bluetooth

Runtime Resource Overlays (RROs) can no longer change the value of
resources in manifest when read during PackagerParser.

This change uses component-override to enable the service by default
on crosshatch devices.

Remove usages of profile_supported_sap from overlays as they no
longer configure the enable state of .sap.SapService.

Bug: 135048762
Test: adb shell pm query-services \
      com.android.bluetooth/.sap.SapService

Change-Id: I89ba2aa61492f65b269849597282594507d2fdc3
---
 component-overrides.xml                       | 20 +++++++++++++++
 device.mk                                     |  4 +++
 .../apps/Bluetooth/res/values/config.xml      | 25 -------------------
 3 files changed, 24 insertions(+), 25 deletions(-)
 create mode 100644 component-overrides.xml
 delete mode 100644 overlay/packages/apps/Bluetooth/res/values/config.xml

diff --git a/component-overrides.xml b/component-overrides.xml
new file mode 100644
index 00000000..bb3419d8
--- /dev/null
+++ b/component-overrides.xml
@@ -0,0 +1,20 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!-- Copyright (C) 2019 The Android Open Source Project
+
+     Licensed under the Apache License, Version 2.0 (the "License");
+     you may not use this file except in compliance with the License.
+     You may obtain a copy of the License at
+
+          http://www.apache.org/licenses/LICENSE-2.0
+
+     Unless required by applicable law or agreed to in writing, software
+     distributed under the License is distributed on an "AS IS" BASIS,
+     WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+     See the License for the specific language governing permissions and
+     limitations under the License.
+-->
+<config>
+    <component-override package="com.android.bluetooth" >
+        <component class=".sap.SapService" enabled="true" />
+    </component-override>
+</config>
\ No newline at end of file
diff --git a/device.mk b/device.mk
index c3f0e957..98a28ba7 100644
--- a/device.mk
+++ b/device.mk
@@ -160,6 +160,10 @@ PRODUCT_PACKAGES += \
     antradio_app \
     AntHalService
 
+# Bluetooth
+PRODUCT_COPY_FILES += \
+    $(LOCAL_PATH)/component-overrides.xml:$(TARGET_COPY_OUT_VENDOR)/etc/sysconfig/component-overrides.xml
+
 # Camera
 PRODUCT_PACKAGES += \
     camera.msm8937 \
diff --git a/overlay/packages/apps/Bluetooth/res/values/config.xml b/overlay/packages/apps/Bluetooth/res/values/config.xml
deleted file mode 100644
index b9ddeb95..00000000
--- a/overlay/packages/apps/Bluetooth/res/values/config.xml
+++ /dev/null
@@ -1,25 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<!-- Copyright (C) 2009-2012 Broadcom Corporation
-   Licensed under the Apache License, Version 2.0 (the "License");
-   you may not use this file except in compliance with the License.
-   You may obtain a copy of the License at
-
-       http://www.apache.org/licenses/LICENSE-2.0
-
-   Unless required by applicable law or agreed to in writing, software
-   distributed under the License is distributed on an "AS IS" BASIS,
-   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-   See the License for the specific language governing permissions and
-   limitations under the License.
--->
-<resources>
-    <bool name="profile_supported_hfpclient">false</bool>
-    <bool name="profile_supported_avrcp_controller">false</bool>
-    <bool name="profile_supported_sap">false</bool>
-    <bool name="profile_supported_pbapclient">false</bool>
-
-    <!-- If true, we will require location to be enabled on the device to
-         fire Bluetooth LE scan result callbacks in addition to having one
-         of the location permissions. -->
-    <bool name="strict_location_check">false</bool>
-</resources>