diff options
author | Kevin Chyn <kchyn@google.com> | 2020-05-19 23:48:10 -0700 |
---|---|---|
committer | Kevin Chyn <kchyn@google.com> | 2020-06-08 12:54:51 -0700 |
commit | 84f162aff1bed7c6e93186ed6a719b8103d22a92 (patch) | |
tree | 2b0aba082e632c4d307763039946a09b7e068f2f /tools/aapt2/java/JavaClassGenerator_test.cpp | |
parent | beb20bb21e83ee8277aa942244cd290ff6280a5e (diff) |
Clean up ServiceListener / ClientMonitor
1) ClientMonitors should be less "abstract", and should be instantiated
with knowledge of "sensorId" and "strength"
2) ClientMonitors should not care about who their client is (e.g.
FingerprintManager, FaceManager, BiometricService). As such,
added a new shim (ClientMonitorCallbackConverter) that receives
callbacks from ClientMonitor and forwards it to the client it was
instantiated with.
Fixes: 157077040
Bug: 157184083
Test: atest com.android.server.biometrics
Test: 1) Enroll, auth on keyguard, auth in BiometricPromptDemo,
remove via settings
2) Enumerate/cleanup - modify FingerprintUtils/FaceUtils to either
store an extra template, or no template in the framework cache
upon enroll completion. reboot device, notice either "framework
template" being removed, or "HAL template" being removed
Note: We should move InternalEnumerateClient / InternalRemovalClient
into their own monitor subclass to remove global state tracking
in BiometricServiceBase
Note: We can consider breaking down ClientMonitorCallbackConverter into
an interface or abstract class, to be implemented by receiver
specific implementations, but it might be a premature optimization
for now
Change-Id: I4716fdfc3f8156ce0b7d2e8ab1af344200b417d1
Diffstat (limited to 'tools/aapt2/java/JavaClassGenerator_test.cpp')
0 files changed, 0 insertions, 0 deletions