diff options
| author | Julia Reynolds <juliacr@google.com> | 2018-12-10 12:37:28 -0500 |
|---|---|---|
| committer | Julia Reynolds <juliacr@google.com> | 2018-12-11 13:49:47 -0500 |
| commit | 27c0a96cb5f73e402ffd1a64f06688950ff66ffb (patch) | |
| tree | 3afc2a3be92102b2c78b0448b1ab57281098916b /tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp | |
| parent | b62dad487c9153a48aa636be0b7c026d52727316 (diff) | |
Updates to Adjustment.KEY_IMPORTANCE handling
- Do not run signal extractors on enqueued notifications - the
assistant isn't told that information and the information that
the extractors base their decisions on it's final. Instead,
extract signals in the post runnable, before the sort/before we
generate the notification update for listeners
- Ignore importance adjustments sent from onNotificationEnqueued
that arrive after the notification was posted, because it's too
late to cancel the notification or silence it.
- Also fix a bug in the handling of importance adjustments
that are sent via adjustNotification: *listen* to the comment that says
'any field that can be changed in an extractor needs to be handled
in handleRankingSort'
Change-Id: I1603d6c5701fcc16f038231e22e21565dd366572
Fixes: 120601946
Test: atest, manual
Diffstat (limited to 'tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp')
0 files changed, 0 insertions, 0 deletions