Log FP & face enrollment state only if it changes.

Bug: 267336592
Test: NA, logging changes
Change-Id: I0e877bfe6869ddebf4f9879143ac4fb1f94920a1
Merged-In: I0e877bfe6869ddebf4f9879143ac4fb1f94920a1
This commit is contained in:
Chandru S
2023-02-27 12:19:27 -08:00
parent db72e4fab4
commit 474de29208

View File

@@ -2424,8 +2424,10 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab
() -> mFaceManager != null && mFaceManager.isHardwareDetected()
&& mFaceManager.hasEnrolledTemplates(userId)
&& mBiometricEnabledForUser.get(userId));
if (mIsFaceEnrolled != isFaceEnrolled) {
mLogger.logFaceEnrolledUpdated(mIsFaceEnrolled, isFaceEnrolled);
}
mIsFaceEnrolled = isFaceEnrolled;
mLogger.logFaceEnrolledUpdated(mIsFaceEnrolled, isFaceEnrolled);
}
public boolean isFaceSupported() {
@@ -3080,12 +3082,13 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab
@VisibleForTesting
boolean isUnlockWithFingerprintPossible(int userId) {
// TODO (b/242022358), make this rely on onEnrollmentChanged event and update it only once.
boolean fpEnrolled = mFpm != null && mFpm.isHardwareDetected()
boolean newFpEnrolled = mFpm != null && mFpm.isHardwareDetected()
&& !isFingerprintDisabled(userId) && mFpm.hasEnrolledTemplates(userId);
mLogger.logFpEnrolledUpdated(userId,
mIsUnlockWithFingerprintPossible.getOrDefault(userId, false),
fpEnrolled);
mIsUnlockWithFingerprintPossible.put(userId, fpEnrolled);
Boolean oldFpEnrolled = mIsUnlockWithFingerprintPossible.getOrDefault(userId, false);
if (oldFpEnrolled != newFpEnrolled) {
mLogger.logFpEnrolledUpdated(userId, oldFpEnrolled, newFpEnrolled);
}
mIsUnlockWithFingerprintPossible.put(userId, newFpEnrolled);
return mIsUnlockWithFingerprintPossible.get(userId);
}