Merge "MusicFx: add a KeepAliveService to allow client app to bind" into main am: b34fac6fb9 am: 41bdc4ce5f
Original change: https://android-review.googlesource.com/c/platform/packages/apps/MusicFX/+/2736934
Change-Id: I2fffe7eb486fdb4a7967b459e78930bdf692e9c4
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
diff --git a/AndroidManifest.xml b/AndroidManifest.xml
index 4642aa7..e0fd477 100644
--- a/AndroidManifest.xml
+++ b/AndroidManifest.xml
@@ -81,6 +81,7 @@
</intent-filter>
</receiver>
<service android:name="Compatibility$Service" android:exported="false" />
+ <service android:name="KeepAliveService" android:exported="true" />
<activity
android:name="Compatibility$Redirector"
android:exported="true">
diff --git a/src/com/android/musicfx/KeepAliveService.java b/src/com/android/musicfx/KeepAliveService.java
new file mode 100644
index 0000000..5771619
--- /dev/null
+++ b/src/com/android/musicfx/KeepAliveService.java
@@ -0,0 +1,52 @@
+/*
+ * Copyright (C) 2023 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.
+ */
+
+package com.android.musicfx;
+
+import android.app.Service;
+import android.content.Intent;
+import android.os.Binder;
+import android.os.IBinder;
+import android.util.Log;
+
+/**
+ * This Service provides a way for client to bind to MusicFx, so that MusicFx can run at same
+ * Process State (see @ProcessStateEnum) as the client.
+ *
+ * Currently, MusicFx does not run in a high priority state (often in CACHED_EMPTY) and can be
+ * easily killed by the LowMemoryKiller. However, its users (music apps, for example) are usually
+ * in a higher priority state, which means they are less likely to be killed. This can lead to
+ * MusicFx and its users being out of sync. To avoid this, framework side will keep record of all
+ * active MusicFx audio sessions, promote the procstate of MusicFx to foreground with the first
+ * audio session open, and remove the foreground procstate delegate with the last audio session
+ * close, or the last user of MusicFx is gone.
+ *
+ * MusicFx user APPs do not need to do anything.
+ *
+ */
+public class KeepAliveService extends Service {
+ private final String TAG = "MusicFxKeepAliveService";
+
+ // Binder given to clients with onBind() callback, the client app receive it as IBinder
+ // parameter of onServiceConnected which can not be used.
+ private final IBinder mBinder = new Binder();
+
+ @Override
+ public IBinder onBind(Intent intent) {
+ Log.i(TAG, "onBind with intent " + intent);
+ return mBinder;
+ }
+}
