Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't preserve a11y focus in hierarchy state.

Browse Source 
This change rolls back functionality added in ag/747748.
The idea was to restore accessibility focus on
configuration changes, but the restoration was happening
too early, which caused ViewRootImpl and the View to
disagree about accessibility focus. This confusion makes
some views unreachable by swiping after a configuration
change.

Bug: 30042251
Change-Id: Iae32b5425a1e7de9a8f5be965feb912fd2d9a0bb
This commit is contained in:
Phil Weaver
2016-08-09 16:12:47 -07:00
parent 0cf4d47a40
commit f40ee509c6
1 changed files with 0 additions and 59 deletions
Download Patch File Download Diff File Expand all files Collapse all files

59
core/java/com/android/internal/policy/PhoneWindow.java
View File

@@ -92,8 +92,6 @@ import android.util.EventLog;
import android.util.Log;
import android.util.SparseArray;
import android.util.TypedValue;
import android.view.accessibility.AccessibilityNodeInfo;
import android.view.accessibility.AccessibilityNodeProvider;
import android.view.animation.Animation;
import android.view.animation.AnimationUtils;
import android.widget.FrameLayout;
@@ -2057,9 +2055,6 @@ public class PhoneWindow extends Window implements MenuBuilder.Callback {
}
static private final String FOCUSED_ID_TAG = "android:focusedViewId";
static private final String ACCESSIBILITY_FOCUSED_ID_TAG = "android:accessibilityFocusedViewId";
static private final String ACCESSIBILITY_FOCUSED_VIRTUAL_ID_TAG =
"android:accessibilityFocusedVirtualViewId";
static private final String VIEWS_TAG = "android:views";
static private final String PANELS_TAG = "android:Panels";
static private final String ACTION_BAR_TAG = "android:ActionBar";
@@ -2082,26 +2077,6 @@ public class PhoneWindow extends Window implements MenuBuilder.Callback {
outState.putInt(FOCUSED_ID_TAG, focusedView.getId());
}
// Save the accessibility focused view ID.
if (mDecor != null) {
final ViewRootImpl viewRootImpl = mDecor.getViewRootImpl();
if (viewRootImpl != null) {
final View accessFocusHost = viewRootImpl.getAccessibilityFocusedHost();
if (accessFocusHost != null && accessFocusHost.getId() != View.NO_ID) {
outState.putInt(ACCESSIBILITY_FOCUSED_ID_TAG, accessFocusHost.getId());
// If we have a focused virtual node ID, save that too.
final AccessibilityNodeInfo accessFocusedNode =
viewRootImpl.getAccessibilityFocusedVirtualView();
if (accessFocusedNode != null) {
final int virtualNodeId = AccessibilityNodeInfo.getVirtualDescendantId(
accessFocusedNode.getSourceNodeId());
outState.putInt(ACCESSIBILITY_FOCUSED_VIRTUAL_ID_TAG, virtualNodeId);
}
}
}
}
// save the panels
SparseArray<Parcelable> panelStates = new SparseArray<Parcelable>();
savePanelState(panelStates);
@@ -2144,13 +2119,6 @@ public class PhoneWindow extends Window implements MenuBuilder.Callback {
}
}
// Restore the accessibility focused view.
final int accessFocusHostViewId = savedInstanceState.getInt(
ACCESSIBILITY_FOCUSED_ID_TAG, View.NO_ID);
final int accessFocusVirtualViewId = savedInstanceState.getInt(
ACCESSIBILITY_FOCUSED_VIRTUAL_ID_TAG, AccessibilityNodeInfo.UNDEFINED_ITEM_ID);
tryRestoreAccessibilityFocus(accessFocusHostViewId, accessFocusVirtualViewId);
// Restore the panels.
SparseArray<Parcelable> panelStates = savedInstanceState.getSparseParcelableArray(PANELS_TAG);
if (panelStates != null) {
@@ -2170,33 +2138,6 @@ public class PhoneWindow extends Window implements MenuBuilder.Callback {
}
}
private void tryRestoreAccessibilityFocus(int hostViewId, int virtualViewId) {
if (hostViewId != View.NO_ID && mDecor != null) {
final View needsAccessFocus = mDecor.findViewById(hostViewId);
if (needsAccessFocus != null) {
if (!tryFocusingVirtualView(needsAccessFocus, virtualViewId)
&& !needsAccessFocus.requestAccessibilityFocus()) {
Log.w(TAG, "Failed to restore focus to previously accessibility"
+ " focused view with id " + hostViewId);
}
} else {
Log.w(TAG, "Previously accessibility focused view reported id " + hostViewId
+ " during save, but can't be found during restore.");
}
}
}
private boolean tryFocusingVirtualView(View host, int virtualViewId) {
if (virtualViewId != AccessibilityNodeInfo.UNDEFINED_ITEM_ID) {
final AccessibilityNodeProvider nodeProvider = host.getAccessibilityNodeProvider();
if (nodeProvider != null) {
return nodeProvider.performAction(virtualViewId,
AccessibilityNodeInfo.ACTION_ACCESSIBILITY_FOCUS, null);
}
}
return false;
}
/**
* Invoked when the panels should freeze their state.
*