Merge "Ensure that health check controller is unbound correctly" into rvc-dev... (45b1622b) · Commits · e / os / android_frameworks_base

services/core/java/com/android/server/PackageWatchdog.java

+1 −6

Original line number	Diff line number	Diff line
		@@ -177,9 +177,6 @@ public class PackageWatchdog {
		// 0 if no prune is scheduled.
		@GuardedBy("mLock")
		private long mUptimeAtLastStateSync;
		// If true, sync explicit health check packages with the ExplicitHealthCheckController.
		@GuardedBy("mLock")
		private boolean mSyncRequired = false;

		@FunctionalInterface
		@VisibleForTesting
		@@ -255,7 +252,6 @@ public class PackageWatchdog {
		*/
		public void registerHealthObserver(PackageHealthObserver observer) {
		synchronized (mLock) {
		mSyncRequired = true;
		ObserverInternal internalObserver = mAllObservers.get(observer.getName());
		if (internalObserver != null) {
		internalObserver.registeredObserver = observer;
		@@ -642,7 +638,7 @@ public class PackageWatchdog {
		synchronized (mLock) {
		if (mIsPackagesReady) {
		Set<String> packages = getPackagesPendingHealthChecksLocked();
		if (!packages.equals(mRequestedHealthCheckPackages) \|\| mSyncRequired) {
		if (!packages.equals(mRequestedHealthCheckPackages) \|\| packages.isEmpty()) {
		syncRequired = true;
		mRequestedHealthCheckPackages = packages;
		}
		@@ -654,7 +650,6 @@ public class PackageWatchdog {
		Slog.i(TAG, "Syncing health check requests for packages: "
		+ mRequestedHealthCheckPackages);
		mHealthCheckController.syncRequests(mRequestedHealthCheckPackages);
		mSyncRequired = false;
		}
		}

+1 −0

Original line number	Diff line number	Diff line
		@@ -1113,6 +1113,7 @@ public class PackageWatchdogTest {
		mTestLooper.dispatchAll();

		List<Set> expectedSyncRequests = List.of(
		Set.of(),
		Set.of(APP_A),
		Set.of(APP_A, APP_B),
		Set.of(APP_A, APP_B, APP_C),