From e192879719b20a97e35d5f0fa00045569ad141eb Mon Sep 17 00:00:00 2001 From: Thomas Stuart Date: Wed, 13 Jul 2022 10:45:19 -0700 Subject: [PATCH] update CallScreeningService#setSkipCallLog to address 3rd pty apps A 3rd party developer reported CallScreeningService.CallResponse#setSkipCallLog(true) was still logging the blocked call to Dialer. This turned out to be true. Turns out we do not allow 3rd party devs from toggling the setSkipCallLog function. Only carriers are allowed to toggle the function. To make this clear, the docs are being updated. bug: 238892157 130213778 (original) Test: not required; docs only change. Change-Id: I6b796421f7f178808eab301e12a4367b67ceaa71 --- telecomm/java/android/telecom/CallScreeningService.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/telecomm/java/android/telecom/CallScreeningService.java b/telecomm/java/android/telecom/CallScreeningService.java index 37b4e657973b4..d1d16ff8b6413 100644 --- a/telecomm/java/android/telecom/CallScreeningService.java +++ b/telecomm/java/android/telecom/CallScreeningService.java @@ -495,6 +495,9 @@ public abstract class CallScreeningService extends Service { * Note: Calls will still be logged with type * {@link android.provider.CallLog.Calls#BLOCKED_TYPE}, regardless of how this property * is set. + *

+ * Note: Only the carrier and system call screening apps can use this parameter; + * this parameter is ignored otherwise. */ public Builder setSkipCallLog(boolean shouldSkipCallLog) { mShouldSkipCallLog = shouldSkipCallLog;