Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 6db58cb0 authored by Chris Wren's avatar Chris Wren Committed by Android (Google) Code Review
Browse files

Merge "Bring consistency to FLAG_ONLY_ALERT_ONCE semantics." into lmp-dev

parents d9037a48 d93d5adf
Loading
Loading
Loading
Loading
+4 −5
Original line number Diff line number Diff line
@@ -1783,7 +1783,7 @@ public abstract class BaseStatusBar extends SystemUI implements
                oldEntry.notification.getNotification().tickerText);

        final boolean shouldInterrupt = shouldInterrupt(notification);
        final boolean alertAgain = alertAgain(oldEntry);
        final boolean alertAgain = alertAgain(oldEntry, n);
        boolean updateSuccessful = false;
        if (contentsUnchanged && bigContentsUnchanged && headsUpContentsUnchanged
                && publicUnchanged) {
@@ -1947,10 +1947,9 @@ public abstract class BaseStatusBar extends SystemUI implements
        }
    }

    private boolean alertAgain(Entry entry) {
        final StatusBarNotification sbn = entry.notification;
        return entry == null || !entry.hasInterrupted()
                || (sbn.getNotification().flags & Notification.FLAG_ONLY_ALERT_ONCE) == 0;
    private boolean alertAgain(Entry oldEntry, Notification newNotification) {
        return oldEntry == null || !oldEntry.hasInterrupted()
                || (newNotification.flags & Notification.FLAG_ONLY_ALERT_ONCE) == 0;
    }

    protected boolean shouldInterrupt(StatusBarNotification sbn) {