Merge "Need to check current user when triggering notification lights" am: 819d39bf3e am: 5fc7c9d63a

Change-Id: I339ec8991f585d118cd7cae6f795ad244db58621

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

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

services/tests/uiservicestests/src/com/android/server/notification/BuzzBeepBlinkTest.java

@@ -1345,6 +1345,22 @@ public class BuzzBeepBlinkTest extends UiServiceTestCase {
         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
     public void testListenerHintCall() throws Exception {
         NotificationChannel ringtoneChannel =