Smart reply updates are not visually interruptive and bubbles should not
show flyout for them, since flyout text remains the same.
1) Modify NoMan.isVisuallyInterruptive to skip evaluation of fields
irrelevant to bubbles
2) Modify NotificationComparator to rank interruptive notifs higher
3) Pipe bool (isInterruptive) from system_process to SysUI
NoMan --- set bool on notif record and ranking
Ranking --- parcel bool for cross-process transport
SysUI notif entry --- get bool from ranking
SysUI bubble data --- on notif entry update, suppress flyout if isInterruptive=false
Considered adding "isInterruptive" bool to StatusBarNotification.
Did not because "visually-interruptive" is additional information that the
system figured out and SBNs should be limited to info from the app.
4) NoMan --- schedule ranking update if interruptive changes for bubble
Fixes: 138755533
Test: manual - send one sms => flyout appears once
Test: manual - send multiple sms in a row => flyout appears for each one
Test: atest FrameworksUiServicesTests
Test: atest NotificationComparatorTest
Test: atest SystemUITests
Change-Id: Id4b855054689ee73a109bb7cd18004531b41f28c
(cherry picked from commit 0dddc61824d091e48962e36939828cae3cde2aa9)
am: 345a2b80c0 -s ours
am skip reason: change_id I80f07d0beb30c779c4bce70bebf2bb4ab22f6bfe with SHA1 01a2093248 is in history
Change-Id: Ic8f9a0e3170a695e47d77b221297f36016b815a9
am: 62f7def277 -s ours
am skip reason: change_id I80f07d0beb30c779c4bce70bebf2bb4ab22f6bfe with SHA1 1516d5290c is in history
Change-Id: Ib66eb7ca5774fbf0c688822fb7c871ed5f71f72d
am: 5cc97f9856 -s ours
am skip reason: change_id I80f07d0beb30c779c4bce70bebf2bb4ab22f6bfe with SHA1 1516d5290c is in history
Change-Id: I581c6fb0d271188d248fbbe46ed9c420fac5916c
am: 3cadf0d9ea -s ours
am skip reason: change_id I80f07d0beb30c779c4bce70bebf2bb4ab22f6bfe with SHA1 1516d5290c is in history
Change-Id: I737b7e4ab97289c83bb616086753b2dcd9cb4724
am: 28fa70f0e0 -s ours
am skip reason: change_id I80f07d0beb30c779c4bce70bebf2bb4ab22f6bfe with SHA1 01a2093248 is in history
Change-Id: I2e13e788bb58ac45356aa21c465662c10dd3f189
This is a first step to restructure the app transition, and the change is
behind the flag persist.wm.new_transit_animations.
Bug: 131661052
Change-Id: I9a23440c6c5d28ea937121e95985b3bd6f3cc451
Test: No-op for production. Pass existing tests.
This reverts commit 84d6b0bfc3.
Reason for revert: This change does not cover state switching between AOD and PULSING.
Change-Id: I06b7375f57dd9035269563d9ae1476b8fc8f86f4
ObserverInternal#packages is a public final field which doesn't need a
getter.
Bug: 140598494
Test: atest PackageWatchdogTest
Change-Id: Ibe3e0b59bebe65b45147044f783a6c72e5bce258
am: 2d2faf674d -s ours
am skip reason: change_id I2140398ad3bbc8076f729c843f0515c654553aaf with SHA1 560c3ac1dc is in history
Change-Id: I7f085ffd88b5e65b0645002d71496eb6a68264dc
am: ae089b82bf -s ours
am skip reason: change_id I98bf4131b7b4a1006e37aa42bcbc7dda7e137d1d with SHA1 ecf98fb416 is in history
Change-Id: I6485554c2f88b21e911f49f728339c0734eac35d
am: 27d33b157c -s ours
am skip reason: change_id I2140398ad3bbc8076f729c843f0515c654553aaf with SHA1 560c3ac1dc is in history
Change-Id: Id0e53f87347acda2879c566ce2a231fd2d217530