Skip to content
Commit 4630b64c authored by Matías Hernández's avatar Matías Hernández
Browse files

Ignore isInterruptive when sorting NotificationRecords

The existing behavior is:
* New notifications are always interruptive.
* Updated notifications are interruptive if and only if they change some subset of fields.
* We put interruptive notifications before non-interruptive ones when sorting (before considering their "when" time).

This leads to some unexpected behaviors, such as:
NM.notify(1, notif1);
NM.notify(2, notif2);
NM.notify(2, notif2);
placing notif1 before notif2, even though it's older and the third notify() call is a sorta-no-op.

This CL creates a flag to remove this specific comparation criterion so we can evaluate the behavior.

Bug: 204367726
Test: atest
Change-Id: I07404f0894d5a0d5592b25d70e67c4b2857f8bcf
parent 58197e43
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment