From e18f51f240e8f1645613e04a2b7e2514a55e7e7b Mon Sep 17 00:00:00 2001 From: Felipe Leme Date: Fri, 7 Sep 2018 11:18:27 -0700 Subject: [PATCH] Improved logging on CharSequenceTransformation. Test: m framework Bug: 113593220 Change-Id: I6860b158decb34cfa0432122baac2a33c756b4f6 --- .../android/service/autofill/CharSequenceTransformation.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/core/java/android/service/autofill/CharSequenceTransformation.java b/core/java/android/service/autofill/CharSequenceTransformation.java index fa6bd65c0fd45..271707addb019 100644 --- a/core/java/android/service/autofill/CharSequenceTransformation.java +++ b/core/java/android/service/autofill/CharSequenceTransformation.java @@ -90,7 +90,7 @@ public final class CharSequenceTransformation extends InternalTransformation imp try { final Matcher matcher = field.first.matcher(value); if (!matcher.find()) { - if (sDebug) Log.d(TAG, "match for " + field.first + " failed on id " + id); + if (sDebug) Log.d(TAG, "Match for " + field.first + " failed on id " + id); return; } // replaceAll throws an exception if the subst is invalid @@ -103,6 +103,9 @@ public final class CharSequenceTransformation extends InternalTransformation imp throw e; } } + // Cannot log converted, it might have PII + Log.d(TAG, "Converting text on child " + childViewId + " to " + converted.length() + + "_chars"); parentTemplate.setCharSequence(childViewId, "setText", converted); }