From c8a46e14553e9dca1f5e0be37898f573adebe5de Mon Sep 17 00:00:00 2001 From: Hai Zhang Date: Sat, 10 Dec 2022 23:21:10 -0800 Subject: [PATCH] Start AccessCheckingService in SystemServer. The service constructor and onStart() only does trivial work and its start time is also tracked by TimingsTraceAndSlog. The new app op and permission implementation will only be active when they are later selected inside the existing permission and app op system services and that selection will happen behind a feature flag disabled by default. Bug: 182523293 Test: presubmit Change-Id: I2d678ef2d5747440afebc92868f92c4a02565856 --- services/java/com/android/server/SystemServer.java | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/services/java/com/android/server/SystemServer.java b/services/java/com/android/server/SystemServer.java index e41e781b7c222..509d75bf9b76c 100644 --- a/services/java/com/android/server/SystemServer.java +++ b/services/java/com/android/server/SystemServer.java @@ -154,6 +154,7 @@ import com.android.server.os.DeviceIdentifiersPolicyService; import com.android.server.os.NativeTombstoneManagerService; import com.android.server.os.SchedulingPolicyService; import com.android.server.people.PeopleService; +import com.android.server.permission.access.AccessCheckingService; import com.android.server.pm.ApexManager; import com.android.server.pm.ApexSystemServiceInfo; import com.android.server.pm.BackgroundInstallControlService; @@ -1110,6 +1111,11 @@ public final class SystemServer implements Dumpable { startMemtrackProxyService(); t.traceEnd(); + // Start AccessCheckingService which provides new implementation for permission and app op. + t.traceBegin("StartAccessCheckingService"); + mSystemServiceManager.startService(AccessCheckingService.class); + t.traceEnd(); + // Activity manager runs the show. t.traceBegin("StartActivityManager"); // TODO: Might need to move after migration to WM.