Merge "Need to check current user when triggering notification lights"

This commit is contained in:
Treehugger Robot
2020-04-10 22:30:28 +00:00
committed by Gerrit Code Review
2 changed files with 20 additions and 0 deletions

View File

@@ -6065,6 +6065,10 @@ public class NotificationManagerService extends SystemService {
if (isInCall() || mScreenOn) { if (isInCall() || mScreenOn) {
return false; return false;
} }
// check current user
if (!isNotificationForCurrentUser(record)) {
return false;
}
return true; return true;
} }

View File

@@ -1291,6 +1291,22 @@ public class BuzzBeepBlinkTest extends UiServiceTestCase {
assertEquals(-1, group.getLastAudiblyAlertedMs()); assertEquals(-1, group.getLastAudiblyAlertedMs());
} }
@Test
public void testLightsCheckCurrentUser() {
final Notification n = new Builder(getContext(), "test")
.setSmallIcon(android.R.drawable.sym_def_app_icon).build();
int userId = mUser.getIdentifier() + 10;
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);
verifyNeverLights();
assertFalse(r.isInterruptive());
assertEquals(-1, r.getLastAudiblyAlertedMs());
}
@Test @Test
public void testListenerHintCall() throws Exception { public void testListenerHintCall() throws Exception {
NotificationRecord r = getCallRecord(1, true); NotificationRecord r = getCallRecord(1, true);