Skip to content

Commit

Permalink
Do not assistant delay for media notifications
Browse files Browse the repository at this point in the history
Due to delay, pressing the pause/play button quickly several times may cause malfunction.
In addition, media notifications do not need an assistant service.

Test: Make media notification
Test: pressing pause/play/Previous/Next button quickly
Test: Check music player
Change-Id: I6ad9153506147db2d2d686d6d0bfffd9eef5717a
Signed-off-by: Dmitrii <[email protected]>
Signed-off-by: Jis G Jacob <[email protected]>
  • Loading branch information
WoongKi Min authored and fazilsheik96 committed Feb 27, 2025
1 parent 41b5b00 commit 5a5a58b
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -8651,7 +8651,7 @@ private boolean enqueueNotification() {
}

// tell the assistant service about the notification
if (mAssistants.isEnabled()) {
if (mAssistants.isEnabled() && !notification.isMediaNotification()) {
mAssistants.onNotificationEnqueuedLocked(r);
mHandler.postDelayed(
new PostNotificationRunnable(r.getKey(), r.getSbn().getPackageName(),
Expand Down

0 comments on commit 5a5a58b

Please sign in to comment.