Merge "Do not send AccessibilityEvent if notification is for different user." into rvc-dev (f914ec17) · Commits · e / os / android_frameworks_base

services/core/java/com/android/server/notification/NotificationManagerService.java

+2 −1

Original line number	Diff line number	Diff line
		@@ -7014,7 +7014,8 @@ public class NotificationManagerService extends SystemService {
		&& (record.getSuppressedVisualEffects() & SUPPRESSED_EFFECT_STATUS_BAR) != 0;
		if (!record.isUpdate
		&& record.getImportance() > IMPORTANCE_MIN
		&& !suppressedByDnd) {
		&& !suppressedByDnd
		&& isNotificationForCurrentUser(record)) {
		sendAccessibilityEvent(record);
		sentAccessibilityEvent = true;
		}

+15 −0

Original line number	Diff line number	Diff line
		@@ -1274,6 +1274,21 @@ public class BuzzBeepBlinkTest extends UiServiceTestCase {
		verify(mAccessibilityService, times(1)).sendAccessibilityEvent(any(), anyInt());
		}

		@Test
		public void testA11yCrossUserEventNotSent() throws Exception {
		final Notification n = new Builder(getContext(), "test")
		.setSmallIcon(android.R.drawable.sym_def_app_icon).build();
		int userId = mUser.getIdentifier() + 1;
		StatusBarNotification sbn = new StatusBarNotification(mPkg, mPkg, 0, mTag, mUid,
		mPid, n, UserHandle.of(userId), null, System.currentTimeMillis());
		NotificationRecord r = new NotificationRecord(getContext(), sbn,
		new NotificationChannel("test", "test", IMPORTANCE_HIGH));

		mService.buzzBeepBlinkLocked(r);

		verify(mAccessibilityService, never()).sendAccessibilityEvent(any(), anyInt());
		}

		@Test
		public void testLightsScreenOn() {
		mService.mScreenOn = true;