Merge "Add additonal logging around user switching in KeyguardUpdateMonitor" into udc-qpr-dev

This commit is contained in:
Chandru S
2023-07-27 04:35:47 +00:00
committed by Android (Google) Code Review
2 changed files with 35 additions and 4 deletions

View File

@@ -3563,6 +3563,7 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab
*/
@VisibleForTesting
void handleUserSwitching(int userId, CountDownLatch latch) {
mLogger.logUserSwitching(userId, "from UserTracker");
Assert.isMainThread();
clearBiometricRecognized();
boolean trustUsuallyManaged = mTrustManager.isTrustUsuallyManaged(userId);
@@ -3583,6 +3584,7 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab
*/
@VisibleForTesting
void handleUserSwitchComplete(int userId) {
mLogger.logUserSwitchComplete(userId, "from UserTracker");
Assert.isMainThread();
for (int i = 0; i < mCallbacks.size(); i++) {
KeyguardUpdateMonitorCallback cb = mCallbacks.get(i).get();
@@ -4036,6 +4038,11 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab
@AnyThread
public void setSwitchingUser(boolean switching) {
if (switching) {
mLogger.logUserSwitching(getCurrentUser(), "from setSwitchingUser");
} else {
mLogger.logUserSwitchComplete(getCurrentUser(), "from setSwitchingUser");
}
mSwitchingUser = switching;
// Since this comes in on a binder thread, we need to post it first
mHandler.post(() -> updateBiometricListeningState(BIOMETRIC_ACTION_UPDATE,

View File

@@ -602,10 +602,10 @@ constructor(@KeyguardUpdateMonitorLog private val logBuffer: LogBuffer) {
fun allowFingerprintOnCurrentOccludingActivityChanged(allow: Boolean) {
logBuffer.log(
TAG,
VERBOSE,
{ bool1 = allow },
{ "allowFingerprintOnCurrentOccludingActivityChanged: $bool1" }
TAG,
VERBOSE,
{ bool1 = allow },
{ "allowFingerprintOnCurrentOccludingActivityChanged: $bool1" }
)
}
@@ -727,4 +727,28 @@ constructor(@KeyguardUpdateMonitorLog private val logBuffer: LogBuffer) {
{ "notifying about enrollments changed: $str1" }
)
}
fun logUserSwitching(userId: Int, context: String) {
logBuffer.log(
TAG,
DEBUG,
{
int1 = userId
str1 = context
},
{ "userCurrentlySwitching: $str1, userId: $int1" }
)
}
fun logUserSwitchComplete(userId: Int, context: String) {
logBuffer.log(
TAG,
DEBUG,
{
int1 = userId
str1 = context
},
{ "userSwitchComplete: $str1, userId: $int1" }
)
}
}