diff options
author | Shunkai Yao <yaoshunkai@google.com> | 2023-08-30 22:47:54 +0000 |
---|---|---|
committer | Shunkai Yao <yaoshunkai@google.com> | 2023-12-11 21:58:56 +0000 |
commit | 197bd88bca223fb3e3b13e9953a7c6a1e5e10b28 (patch) | |
tree | 05f4c053e04fdd9ec53e9d6b93fa2f49ed0d11d4 | |
parent | 8e434c30d274dd524f514ea7d86cb8280233221e (diff) | |
download | MusicFX-197bd88bca223fb3e3b13e9953a7c6a1e5e10b28.tar.gz |
MusicFx: add a KeepAliveService to allow client app to bind
So MusicFx activity can be same ProcessState as client APP
Bug: 293204643
Bug: 265114634
Test: MusicFx BindService can be bind with the test app correctly
Test: dumpsys activity
Change-Id: Id751854861d10d5aa31dbb4ae471e403398b0e4e
Merged-In: Id751854861d10d5aa31dbb4ae471e403398b0e4e
-rw-r--r-- | AndroidManifest.xml | 1 | ||||
-rw-r--r-- | src/com/android/musicfx/KeepAliveService.java | 52 |
2 files changed, 53 insertions, 0 deletions
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; + } +} |