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

Commit 6dfaa935 authored by Yuri Lin's avatar Yuri Lin
Browse files

Update matchesCallFilter guidance to recommend tel: codes 

Using tel: or mailto: works for repeat callers, and the new documentation recommends against using a contacts content lookup URI, as we don't explicitly look up phone numbers associated with the contact at the time of the matchesCallFilter call.

Bug: 183924362
Bug: 203806689
Bug: 192592755
Test: NotificationManagerTest
Change-Id: I4786f3206f145ab4273a1151a9356101582e9044
parent ef75a778

core/java/android/app/NotificationManager.java

+12 −5
Original line number Diff line number Diff line
@@ -2573,11 +2573,18 @@ public class NotificationManager { 
     * method will return false regardless of input.

     * </p>

     * <p>

     * The provided URI must meet the requirements for a URI associated with a

     * {@link Person}: it may be the {@code String} representation of a

     * {@link android.provider.ContactsContract.Contacts#CONTENT_LOOKUP_URI}, or a

     * <code>mailto:</code> or <code>tel:</code> schema URI matching an entry in the

     * Contacts database. See also {@link Person.Builder#setUri} and

     * The provided URI should be a <code>tel:</code> or <code>mailto:</code> schema URI indicating

     * the source of the call. For an accurate answer regarding whether the caller matches the

     * user's permitted contacts, the path part of the URI must match an entry the Contacts database

     * in the appropriate column.

     * </p>

     * <p>

     * Passing in a {@link android.provider.ContactsContract.Contacts#CONTENT_LOOKUP_URI} is also

     * permissible, but should only be used for priority contact interruptions and may not provide

     * accurate results in the case of repeat callers.

     * </p>

     * <p>

     * See also {@link Person.Builder#setUri} and

     * {@link android.provider.ContactsContract.Contacts#CONTENT_LOOKUP_URI}

     * for more information.

     * </p>