From c39c0be88fb6d2ce34d78f19cc988079c131dd19 Mon Sep 17 00:00:00 2001 From: Jiakai Zhang Date: Wed, 16 Nov 2022 18:41:23 +0000 Subject: [PATCH] Revert "Expose ServiceManager.waitForService as system API." This reverts commit ee13e8d435ec3abe821118827226db47e1d7fe53. Reason for revert: As discussed in b/233915142, an API that can obtain an arbitrary binder should not be exposed, and mainline modules should generally follow go/android-api-guidelines#the-moduleservicemanager-pattern. Bug: 233915142 Test: Presubmit Change-Id: I67e6e177d48ed213306aa0c10f83fe3b670daf1d --- core/api/module-lib-current.txt | 1 - core/java/android/os/ServiceManager.java | 2 -- 2 files changed, 3 deletions(-) diff --git a/core/api/module-lib-current.txt b/core/api/module-lib-current.txt index 07af1d5f1634c..b6e2d2a3e98bb 100644 --- a/core/api/module-lib-current.txt +++ b/core/api/module-lib-current.txt @@ -360,7 +360,6 @@ package android.os { method @NonNull public static String[] getDeclaredInstances(@NonNull String); method public static boolean isDeclared(@NonNull String); method @Nullable public static android.os.IBinder waitForDeclaredService(@NonNull String); - method @Nullable public static android.os.IBinder waitForService(@NonNull String); } public class StatsServiceManager { diff --git a/core/java/android/os/ServiceManager.java b/core/java/android/os/ServiceManager.java index b6ff102b00b13..9ea42780981d9 100644 --- a/core/java/android/os/ServiceManager.java +++ b/core/java/android/os/ServiceManager.java @@ -278,8 +278,6 @@ public final class ServiceManager { * @return {@code null} only if there are permission problems or fatal errors. * @hide */ - @SystemApi(client = SystemApi.Client.MODULE_LIBRARIES) - @Nullable public static IBinder waitForService(@NonNull String name) { return Binder.allowBlocking(waitForServiceNative(name)); }