From 3269079e321cd1c124cb7fe890c9e0f01385eb81 Mon Sep 17 00:00:00 2001 From: Chandru Date: Fri, 7 Oct 2022 10:21:01 +0000 Subject: [PATCH] Log the ApiRequestReason whenever face auth is requested Fixes: 251421075 Test: atest KeyguardUpdateMonitorTest Change-Id: I262c46daab5fc7dc7eeb19451a6687a237a986db --- .../src/com/android/keyguard/KeyguardUpdateMonitor.java | 2 +- .../keyguard/logging/KeyguardUpdateMonitorLogger.kt | 9 +++++---- .../src/com/android/systemui/log/dagger/LogModule.java | 2 +- 3 files changed, 7 insertions(+), 6 deletions(-) diff --git a/packages/SystemUI/src/com/android/keyguard/KeyguardUpdateMonitor.java b/packages/SystemUI/src/com/android/keyguard/KeyguardUpdateMonitor.java index d14666de4b3e6..cc8668e62d3c4 100644 --- a/packages/SystemUI/src/com/android/keyguard/KeyguardUpdateMonitor.java +++ b/packages/SystemUI/src/com/android/keyguard/KeyguardUpdateMonitor.java @@ -2319,7 +2319,7 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab */ public void requestFaceAuth(boolean userInitiatedRequest, @FaceAuthApiRequestReason String reason) { - mLogger.logFaceAuthRequested(userInitiatedRequest); + mLogger.logFaceAuthRequested(userInitiatedRequest, reason); updateFaceListeningState(BIOMETRIC_ACTION_START, apiRequestReasonToUiEvent(reason)); } diff --git a/packages/SystemUI/src/com/android/keyguard/logging/KeyguardUpdateMonitorLogger.kt b/packages/SystemUI/src/com/android/keyguard/logging/KeyguardUpdateMonitorLogger.kt index bf9f4c88bde34..8703c4ba99132 100644 --- a/packages/SystemUI/src/com/android/keyguard/logging/KeyguardUpdateMonitorLogger.kt +++ b/packages/SystemUI/src/com/android/keyguard/logging/KeyguardUpdateMonitorLogger.kt @@ -108,10 +108,11 @@ class KeyguardUpdateMonitorLogger @Inject constructor( }, { "Face help received, msgId: $int1 msg: $str1" }) } - fun logFaceAuthRequested(userInitiatedRequest: Boolean) { - logBuffer.log(TAG, DEBUG, - { bool1 = userInitiatedRequest }, - { "requestFaceAuth() userInitiated=$bool1" }) + fun logFaceAuthRequested(userInitiatedRequest: Boolean, reason: String) { + logBuffer.log(TAG, DEBUG, { + bool1 = userInitiatedRequest + str1 = reason + }, { "requestFaceAuth() userInitiated=$bool1 reason=$str1" }) } fun logFaceAuthSuccess(userId: Int) { diff --git a/packages/SystemUI/src/com/android/systemui/log/dagger/LogModule.java b/packages/SystemUI/src/com/android/systemui/log/dagger/LogModule.java index d298ff9a8de98..529d0662028ae 100644 --- a/packages/SystemUI/src/com/android/systemui/log/dagger/LogModule.java +++ b/packages/SystemUI/src/com/android/systemui/log/dagger/LogModule.java @@ -322,7 +322,7 @@ public class LogModule { @SysUISingleton @KeyguardUpdateMonitorLog public static LogBuffer provideKeyguardUpdateMonitorLogBuffer(LogBufferFactory factory) { - return factory.create("KeyguardUpdateMonitorLog", 200); + return factory.create("KeyguardUpdateMonitorLog", 400); } /**