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

Eliminate delay before contacts lookup for a Notification

The RankingReconsideration produced by ValidateNotificationPeople had getDelay() == 1000 ms, which meant NMS waiting a full second before starting the work. This wasted half of the time available to decide whether a notification should produce sound during DND, for no good reason (despite what the comment claimed, this delay is not used as a timeout).

This CL also moves all the contacts-querying methods in ValidateNotificationPeople to PeopleRankingReconsideration. This makes it clear that the querying is only done as part of the reconsideration, and process() only queries the cache.

(This is a roll-forward with adjusted CTS tests).

Bug: 263839687
Test: atest
Change-Id: I80707fc1381410a5b64c3ac128e448d8d9a35b11
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