From 3399fe3a011b02a961b3009ba0a39c45ba93cb73 Mon Sep 17 00:00:00 2001
From: Ryan Mitchell <rtmitchell@google.com>
Date: Thu, 10 Oct 2019 08:09:29 -0700
Subject: [PATCH] universal7885: Use component-override to configure 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.

Remove usages of profile_supported_* from overlays as they no
longer configure the enable state of Bluetooth components.

Bug: 135048762
Test: adb shell pm query-services \
      com.android.bluetooth/.avrcpcontroller.AvrcpControllerService
      adb shell pm query-services \
      com.android.bluetooth/.hid.HidDeviceService
      adb shell pm query-services \
      com.android.bluetooth/.sap.SapService
---
 .../configs/sysconfig/component-overrides.xml | 22 +++++++++++++++++++
 .../{pixel => sysconfig}/pixel_eureka.xml     |  0
 universal7885-common/universal7885-common.mk  | 10 +++++----
 3 files changed, 28 insertions(+), 4 deletions(-)
 create mode 100644 universal7885-common/configs/sysconfig/component-overrides.xml
 rename universal7885-common/configs/{pixel => sysconfig}/pixel_eureka.xml (100%)

diff --git a/universal7885-common/configs/sysconfig/component-overrides.xml b/universal7885-common/configs/sysconfig/component-overrides.xml
new file mode 100644
index 0000000..99c2115
--- /dev/null
+++ b/universal7885-common/configs/sysconfig/component-overrides.xml
@@ -0,0 +1,22 @@
+<?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=".avrcpcontroller.AvrcpControllerService" enabled="true" />
+        <component class=".hid.HidDeviceService" enabled="true" />
+        <component class=".sap.SapService" enabled="true" />
+    </component-override>
+</config>
diff --git a/universal7885-common/configs/pixel/pixel_eureka.xml b/universal7885-common/configs/sysconfig/pixel_eureka.xml
similarity index 100%
rename from universal7885-common/configs/pixel/pixel_eureka.xml
rename to universal7885-common/configs/sysconfig/pixel_eureka.xml
diff --git a/universal7885-common/universal7885-common.mk b/universal7885-common/universal7885-common.mk
index d48c6a1..1ee2ec7 100644
--- a/universal7885-common/universal7885-common.mk
+++ b/universal7885-common/universal7885-common.mk
@@ -174,10 +174,6 @@ PRODUCT_COPY_FILES += \
 DEVICE_PACKAGE_OVERLAYS += $(LOCAL_PATH)/overlay
 DEVICE_PACKAGE_OVERLAYS += $(LOCAL_PATH)/overlay-derp
 
-# Google Photos
-PRODUCT_COPY_FILES += \
-    $(LOCAL_PATH)/configs/pixel/pixel_eureka.xml:$(TARGET_COPY_OUT_PRODUCT)/etc/sysconfig/pixel_eureka.xml
-
 # Permissions
 PRODUCT_COPY_FILES += \
     frameworks/native/data/etc/android.hardware.audio.low_latency.xml:$(TARGET_COPY_OUT_VENDOR)/etc/permissions/android.hardware.audio.low_latency.xml \
@@ -287,6 +283,12 @@ PRODUCT_SOONG_NAMESPACES += \
     hardware/google/pixel \
     device/samsung/universal7885-common/aidl/power-libperfmgr
 
+# Sysconfig
+PRODUCT_COPY_FILES += \
+    $(LOCAL_PATH)/configs/sysconfig/component-overrides.xml:$(TARGET_COPY_OUT_VENDOR)/etc/sysconfig/component-overrides-bt.xml \
+    $(LOCAL_PATH)/configs/sysconfig/pixel_eureka.xml:$(TARGET_COPY_OUT_PRODUCT)/etc/sysconfig/pixel_eureka.xml
+
+
 # Thermal HAL
 PRODUCT_PACKAGES += \
     android.hardware.thermal@2.0-service.pixel \