From 44b3b91bf087f040678af756b60ea1e00ee4926f Mon Sep 17 00:00:00 2001 From: Phil Weaver Date: Wed, 16 Mar 2016 18:20:16 -0700 Subject: [PATCH] Support SET_TEXT for editable text views. Bug: 25486055 Change-Id: I38e5a65e216b7ea7da42a01040126b2c2cc8c661 --- core/java/android/widget/EditText.java | 3 +++ core/java/android/widget/TextView.java | 14 ++++++++++++++ 2 files changed, 17 insertions(+) diff --git a/core/java/android/widget/EditText.java b/core/java/android/widget/EditText.java index 434e3eb8237e9..ad355504a0ee6 100644 --- a/core/java/android/widget/EditText.java +++ b/core/java/android/widget/EditText.java @@ -155,6 +155,9 @@ public class EditText extends TextView { public boolean performAccessibilityActionInternal(int action, Bundle arguments) { switch (action) { case AccessibilityNodeInfo.ACTION_SET_TEXT: { + if (!isEnabled()) { + return false; + } CharSequence text = (arguments != null) ? arguments.getCharSequence( AccessibilityNodeInfo.ACTION_ARGUMENT_SET_TEXT_CHARSEQUENCE) : null; setText(text); diff --git a/core/java/android/widget/TextView.java b/core/java/android/widget/TextView.java index fbedbdac9065a..77c7691778e9c 100644 --- a/core/java/android/widget/TextView.java +++ b/core/java/android/widget/TextView.java @@ -9101,6 +9101,9 @@ public class TextView extends View implements ViewTreeObserver.OnPreDrawListener if (mBufferType == BufferType.EDITABLE) { info.setEditable(true); + if (isEnabled()) { + info.addAction(AccessibilityNodeInfo.AccessibilityAction.ACTION_SET_TEXT); + } } if (mEditor != null) { @@ -9232,6 +9235,17 @@ public class TextView extends View implements ViewTreeObserver.OnPreDrawListener } } } return false; + case AccessibilityNodeInfo.ACTION_SET_TEXT: { + if (!isEnabled() || (mBufferType != BufferType.EDITABLE)) { + return false; + } + CharSequence text = (arguments != null) ? arguments.getCharSequence( + AccessibilityNodeInfo.ACTION_ARGUMENT_SET_TEXT_CHARSEQUENCE) : null; + setText(text); + if (text != null && text.length() > 0) { + Selection.setSelection((Spannable) mText, text.length()); + } + } return true; default: { return super.performAccessibilityActionInternal(action, arguments); }