Skip to content

Commit 2f87014

Browse files
sganovAndroid (Google) Code Review
authored andcommitted
Merge "Enabling accessibility focus only if explore by touch is on." into jb-dev
2 parents 0e8fd5e + 07b726c commit 2f87014

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

core/java/android/view/View.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5960,7 +5960,8 @@ boolean isAccessibilityFocused() {
59605960
* @hide
59615961
*/
59625962
public boolean requestAccessibilityFocus() {
5963-
if (!AccessibilityManager.getInstance(mContext).isEnabled()) {
5963+
AccessibilityManager manager = AccessibilityManager.getInstance(mContext);
5964+
if (!manager.isEnabled() || !manager.isTouchExplorationEnabled()) {
59645965
return false;
59655966
}
59665967
if ((mViewFlags & VISIBILITY_MASK) != VISIBLE) {

core/java/android/view/ViewRootImpl.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2318,7 +2318,8 @@ private boolean drawSoftware(Surface surface, AttachInfo attachInfo, int yoff,
23182318
* @param canvas The canvas on which to draw.
23192319
*/
23202320
private void drawAccessibilityFocusedDrawableIfNeeded(Canvas canvas) {
2321-
if (!AccessibilityManager.getInstance(mView.mContext).isEnabled()) {
2321+
AccessibilityManager manager = AccessibilityManager.getInstance(mView.mContext);
2322+
if (!manager.isEnabled() || !manager.isTouchExplorationEnabled()) {
23222323
return;
23232324
}
23242325
if (mAccessibilityFocusedHost == null || mAccessibilityFocusedHost.mAttachInfo == null) {

0 commit comments

Comments
 (0)