diff options
3 files changed, 0 insertions, 59 deletions
diff --git a/services/core/java/com/android/server/biometrics/AuthenticationStatsCollector.java b/services/core/java/com/android/server/biometrics/AuthenticationStatsCollector.java index c9fb78546c42..0380756addea 100644 --- a/services/core/java/com/android/server/biometrics/AuthenticationStatsCollector.java +++ b/services/core/java/com/android/server/biometrics/AuthenticationStatsCollector.java @@ -18,14 +18,10 @@ package com.android.server.biometrics; import android.annotation.NonNull; import android.annotation.Nullable; -import android.content.BroadcastReceiver; import android.content.Context; -import android.content.Intent; -import android.content.IntentFilter; import android.content.pm.PackageManager; import android.hardware.face.FaceManager; import android.hardware.fingerprint.FingerprintManager; -import android.os.UserHandle; import com.android.internal.R; import com.android.internal.annotations.VisibleForTesting; @@ -60,17 +56,6 @@ public class AuthenticationStatsCollector { @NonNull private AuthenticationStatsPersister mAuthenticationStatsPersister; @NonNull private BiometricNotification mBiometricNotification; - private final BroadcastReceiver mBroadcastReceiver = new BroadcastReceiver() { - @Override - public void onReceive(@NonNull Context context, @NonNull Intent intent) { - final int userId = intent.getIntExtra(Intent.EXTRA_USER_HANDLE, UserHandle.USER_NULL); - if (userId != UserHandle.USER_NULL - && intent.getAction().equals(Intent.ACTION_USER_REMOVED)) { - onUserRemoved(userId); - } - } - }; - public AuthenticationStatsCollector(@NonNull Context context, int modality, @NonNull BiometricNotification biometricNotification) { mContext = context; @@ -79,8 +64,6 @@ public class AuthenticationStatsCollector { mUserAuthenticationStatsMap = new HashMap<>(); mModality = modality; mBiometricNotification = biometricNotification; - - context.registerReceiver(mBroadcastReceiver, new IntentFilter(Intent.ACTION_USER_REMOVED)); } private void initializeUserAuthenticationStatsMap() { @@ -163,11 +146,6 @@ public class AuthenticationStatsCollector { } } - private void onUserRemoved(final int userId) { - mUserAuthenticationStatsMap.remove(userId); - mAuthenticationStatsPersister.removeFrrStats(userId); - } - /** * Only being used in tests. Callers should not make any changes to the returned * authentication stats. diff --git a/services/core/java/com/android/server/biometrics/AuthenticationStatsPersister.java b/services/core/java/com/android/server/biometrics/AuthenticationStatsPersister.java index 7217956c61a5..96150a655342 100644 --- a/services/core/java/com/android/server/biometrics/AuthenticationStatsPersister.java +++ b/services/core/java/com/android/server/biometrics/AuthenticationStatsPersister.java @@ -94,29 +94,6 @@ public class AuthenticationStatsPersister { } /** - * Remove frr data for a specific user. - */ - public void removeFrrStats(int userId) { - try { - // Copy into a new HashSet to avoid iterator exception. - Set<String> frrStatsSet = new HashSet<>(readFrrStats()); - - // Remove the old authentication stat for the user if it exists. - for (Iterator<String> iterator = frrStatsSet.iterator(); iterator.hasNext();) { - String frrStats = iterator.next(); - JSONObject frrStatJson = new JSONObject(frrStats); - if (getValue(frrStatJson, USER_ID).equals(String.valueOf(userId))) { - iterator.remove(); - break; - } - } - - mSharedPreferences.edit().putStringSet(KEY, frrStatsSet).apply(); - } catch (JSONException ignored) { - } - } - - /** * Persist frr data for a specific user. */ public void persistFrrStats(int userId, int totalAttempts, int rejectedAttempts, diff --git a/services/tests/servicestests/src/com/android/server/biometrics/AuthenticationStatsPersisterTest.java b/services/tests/servicestests/src/com/android/server/biometrics/AuthenticationStatsPersisterTest.java index 66a8ff31018a..455625cf69ec 100644 --- a/services/tests/servicestests/src/com/android/server/biometrics/AuthenticationStatsPersisterTest.java +++ b/services/tests/servicestests/src/com/android/server/biometrics/AuthenticationStatsPersisterTest.java @@ -211,20 +211,6 @@ public class AuthenticationStatsPersisterTest { assertThat(mStringSetArgumentCaptor.getValue()).contains(expectedFrrStats); } - @Test - public void removeFrrStats_existingUser_shouldUpdateRecord() throws JSONException { - AuthenticationStats authenticationStats = new AuthenticationStats(USER_ID_1, - 300 /* totalAttempts */, 10 /* rejectedAttempts */, - 0 /* enrollmentNotifications */, BiometricsProtoEnums.MODALITY_FACE); - when(mSharedPreferences.getStringSet(eq(KEY), anySet())).thenReturn( - Set.of(buildFrrStats(authenticationStats))); - - mAuthenticationStatsPersister.removeFrrStats(USER_ID_1); - - verify(mEditor).putStringSet(eq(KEY), mStringSetArgumentCaptor.capture()); - assertThat(mStringSetArgumentCaptor.getValue()).doesNotContain(authenticationStats); - } - private String buildFrrStats(AuthenticationStats authenticationStats) throws JSONException { if (authenticationStats.getModality() == BiometricsProtoEnums.MODALITY_FACE) { |