Merge "Cleanup of package_monitor_dedicated_thread" into main (64f3da00) · Commits · e / os / android_frameworks_base

services/accessibility/accessibility.aconfig

+0 −10

Original line number	Diff line number	Diff line
		@@ -233,16 +233,6 @@ flag {
		}
		}

		flag {
		name: "package_monitor_dedicated_thread"
		namespace: "accessibility"
		description: "Runs the A11yManagerService PackageMonitor on a dedicated thread"
		bug: "348138695"
		metadata {
		purpose: PURPOSE_BUGFIX
		}
		}

		flag {
		name: "only_reset_magnification_if_needed_when_destroy_handler"
		namespace: "accessibility"

+7 −12

Original line number	Diff line number	Diff line
		@@ -1088,18 +1088,13 @@ public class AccessibilityManagerService extends IAccessibilityManager.Stub
		private void registerBroadcastReceivers() {
		// package changes
		mPackageMonitor = new ManagerPackageMonitor(this);
		final Looper packageMonitorLooper;
		if (Flags.packageMonitorDedicatedThread()) {
		// Use a dedicated thread because the default BackgroundThread used by PackageMonitor
		// is shared by other components and can get busy, causing a delay and eventual ANR when
		// responding to broadcasts sent to this PackageMonitor.
		HandlerThread packageMonitorThread = new HandlerThread(LOG_TAG + " PackageMonitor",
		Process.THREAD_PRIORITY_BACKGROUND);
		packageMonitorThread.start();
		packageMonitorLooper = packageMonitorThread.getLooper();
		} else {
		packageMonitorLooper = null;
		}
		final Looper packageMonitorLooper = packageMonitorThread.getLooper();
		mPackageMonitor.register(mContext, packageMonitorLooper, UserHandle.ALL, true);

		// user change and unlock