From 5893e9e008d46cf4be0c7f709a6e77e2652c3ddd Mon Sep 17 00:00:00 2001 From: Andrew Stadler Date: Mon, 8 Feb 2010 23:09:05 -0800 Subject: [PATCH] Rewrite notes to acknowledge wipe-after-bad-passwords It turns out that we have already implemented the built-in version of local-wipe-after-failed-passwords, and the notes about it were not necessary. It should be possible to connect to an account with local wipe requirements and see proper operation. --- src/com/android/email/SecurityPolicy.java | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/src/com/android/email/SecurityPolicy.java b/src/com/android/email/SecurityPolicy.java index 5551e8e75..6d20dbf8a 100644 --- a/src/com/android/email/SecurityPolicy.java +++ b/src/com/android/email/SecurityPolicy.java @@ -43,8 +43,6 @@ import android.net.Uri; * TODO: When accounts are deleted, reduce policy and/or give up admin status * TODO: Provide a way to check for policy issues at synchronous times such as entering * message list or folder list. - * TODO: Implement local wipe after failed passwords - * */ public class SecurityPolicy { @@ -630,6 +628,8 @@ public class SecurityPolicy { /** * Device Policy administrator. This is primarily a listener for device state changes. * Note: This is instantiated by incoming messages. + * Note: We do not implement onPasswordFailed() because the default behavior of the + * DevicePolicyManager - complete local wipe after 'n' failures - is sufficient. */ public static class PolicyAdmin extends DeviceAdmin { @@ -656,13 +656,5 @@ public class SecurityPolicy { public void onPasswordChanged(Context context, Intent intent) { SecurityPolicy.getInstance(context).clearAccountHoldFlags(); } - - /** - * Called after the user has failed at entering their current password. - */ - @Override - public void onPasswordFailed(Context context, Intent intent) { - // do something - } } }