Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Merge "ServiceManager: waitForService uses allowBlocking" am: b801a513d1 am: 726c374fa3" into sc-dev-plus-aosp am: 578d06c984

Browse Source 
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1793027

Change-Id: I3ff40e03dff825dad6b890a5cea45bcb72211319
This commit is contained in:
Automerger Merge Worker
2021-08-11 17:52:26 +00:00
parent ef43faede1 578d06c984
commit 42be3a6439
2 changed files with 11 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
core/java/android/os/ServiceManager.java
View File

@@ -257,7 +257,12 @@ public final class ServiceManager {
*
* @return {@code null} only if there are permission problems or fatal errors.
*/
public static native IBinder waitForService(@NonNull String name);
public static IBinder waitForService(@NonNull String name) {
return Binder.allowBlocking(waitForServiceNative(name));
}
private static native IBinder waitForServiceNative(@NonNull String name);
/**
* Returns the specified service from the service manager, if declared.

16
core/jni/android_os_ServiceManager.cpp
View File

@@ -29,11 +29,8 @@ namespace android {
// Native because we have a client-side wait in waitForService() and we do not
// want an unnecessary second copy of it.
static jobject android_os_ServiceManager_waitForService(
JNIEnv *env,
jclass /* clazzObj */,
jstring serviceNameObj) {
static jobject android_os_ServiceManager_waitForServiceNative(JNIEnv* env, jclass /* clazzObj */,
jstring serviceNameObj) {
const jchar* serviceName = env->GetStringCritical(serviceNameObj, nullptr);
if (!serviceName) {
jniThrowNullPointerException(env, nullptr);
@@ -55,12 +52,9 @@ static jobject android_os_ServiceManager_waitForService(
// ----------------------------------------------------------------------------
static const JNINativeMethod method_table[] = {
/* name, signature, funcPtr */
{
"waitForService",
"(Ljava/lang/String;)Landroid/os/IBinder;",
(void*)android_os_ServiceManager_waitForService
},
/* name, signature, funcPtr */
{"waitForServiceNative", "(Ljava/lang/String;)Landroid/os/IBinder;",
(void*)android_os_ServiceManager_waitForServiceNative},
};
int register_android_os_ServiceManager(JNIEnv* env) {