Merge "Fix bugs found during cts test." into tm-dev am: 761bb05aaa

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/17489934

Change-Id: I84aa5b1df7ecca053f9263da4e99ec5e5d30e92a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Kathy Chen
2022-03-30 15:43:14 +00:00
committed by Automerger Merge Worker
2 changed files with 13 additions and 5 deletions

View File

@@ -217,9 +217,13 @@ final class AmbientContextManagerPerUserService extends
RemoteCallback detectionResultCallback, RemoteCallback statusCallback) { RemoteCallback detectionResultCallback, RemoteCallback statusCallback) {
Slog.d(TAG, "Requested detection of " + request.getEventTypes()); Slog.d(TAG, "Requested detection of " + request.getEventTypes());
synchronized (mLock) { synchronized (mLock) {
ensureRemoteServiceInitiated(); if (setUpServiceIfNeeded()) {
mRemoteService.startDetection(request, callingPackage, detectionResultCallback, ensureRemoteServiceInitiated();
statusCallback); mRemoteService.startDetection(request, callingPackage, detectionResultCallback,
statusCallback);
} else {
Slog.w(TAG, "No valid component found for AmbientContextDetectionService");
}
} }
} }
@@ -371,8 +375,10 @@ final class AmbientContextManagerPerUserService extends
void stopDetection(String packageName) { void stopDetection(String packageName) {
Slog.d(TAG, "Stop detection for " + packageName); Slog.d(TAG, "Stop detection for " + packageName);
synchronized (mLock) { synchronized (mLock) {
ensureRemoteServiceInitiated(); if (mComponentName != null) {
mRemoteService.stopDetection(packageName); ensureRemoteServiceInitiated();
mRemoteService.stopDetection(packageName);
}
} }
} }

View File

@@ -249,6 +249,8 @@ public class AmbientContextManagerService extends
Objects.requireNonNull(eventTypes); Objects.requireNonNull(eventTypes);
Objects.requireNonNull(callingPackage); Objects.requireNonNull(callingPackage);
assertCalledByPackageOwner(callingPackage); assertCalledByPackageOwner(callingPackage);
mContext.enforceCallingOrSelfPermission(
Manifest.permission.ACCESS_AMBIENT_CONTEXT_EVENT, TAG);
mService.onStartConsentActivity(eventTypes, callingPackage); mService.onStartConsentActivity(eventTypes, callingPackage);
} }