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

Commit 549a7dde authored by Grace Jia's avatar Grace Jia
Browse files

Always pass handle uri to NotificationManager when query if we should 

ring for a contact.

Currently, we're passing contact query uri if the incoming call coming
from a known contact. This uri sometimes can contain query parameters
which will cause NotificationManager failed to detect repeated call.
Change Ringer to pass the handle uri all the time to fix this.

Bug: 183924362
Test: TelecomUnitTest, manually test repeated call in DND mode
Change-Id: Ie390f37356438966cce1ebeb0ff70e6a5b470273
parent ae810381

src/com/android/server/telecom/Ringer.java

+1 −1
Original line number Original line Diff line number Diff line
@@ -512,7 +512,7 @@ public class Ringer { 




        boolean isVolumeOverZero = mAudioManager.getStreamVolume(AudioManager.STREAM_RING) > 0;

        boolean isVolumeOverZero = mAudioManager.getStreamVolume(AudioManager.STREAM_RING) > 0;

        timer.record("isVolumeOverZero");

        timer.record("isVolumeOverZero");

        boolean shouldRingForContact = shouldRingForContact(call.getContactUri());

        boolean shouldRingForContact = shouldRingForContact(call.getHandle());

        timer.record("shouldRingForContact");

        timer.record("shouldRingForContact");

        boolean isRingtonePresent = !(mRingtoneFactory.getRingtone(call) == null);

        boolean isRingtonePresent = !(mRingtoneFactory.getRingtone(call) == null);

        timer.record("getRingtone");

        timer.record("getRingtone");