From f62c4d38b3a1446368fbda3d253b3e41cc8b500f Mon Sep 17 00:00:00 2001 From: Eric Biggers Date: Fri, 5 May 2023 18:37:05 +0000 Subject: [PATCH] Remove obsolete comment about isVisiblePatternEnabled() The comment in LockPatternUtils#isVisiblePatternEnabled() was only needed because there were two sources of truth for the default value. Now that there is only one source of truth, it is not needed. Bug: 270013005 Change-Id: Ia531f84bcedc957393f93222679efae848a82822 --- core/java/com/android/internal/widget/LockPatternUtils.java | 3 --- 1 file changed, 3 deletions(-) diff --git a/core/java/com/android/internal/widget/LockPatternUtils.java b/core/java/com/android/internal/widget/LockPatternUtils.java index 192ef047fc09a..809ce96d08346 100644 --- a/core/java/com/android/internal/widget/LockPatternUtils.java +++ b/core/java/com/android/internal/widget/LockPatternUtils.java @@ -1040,9 +1040,6 @@ public class LockPatternUtils { */ @UnsupportedAppUsage public boolean isVisiblePatternEnabled(int userId) { - // Default to true, since this gets explicitly set to true when a pattern is first set - // anyway, which makes true the user-visible default. The low-level default should be the - // same, in order for FRP credential verification to get the same default. return getBoolean(Settings.Secure.LOCK_PATTERN_VISIBLE, true, userId); }