Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1232)

Unified Diff: content/public/android/java/src/org/chromium/content/browser/accessibility/BrowserAccessibilityManager.java

Issue 2811803002: Fixes accessibility navigation out of autofill popup. (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/public/android/java/src/org/chromium/content/browser/accessibility/BrowserAccessibilityManager.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/accessibility/BrowserAccessibilityManager.java b/content/public/android/java/src/org/chromium/content/browser/accessibility/BrowserAccessibilityManager.java
index 54ad7171810080c1d4434793253717ebb4deec92..ac96bd20bbf154b3b3518952167796289762e8b5 100644
--- a/content/public/android/java/src/org/chromium/content/browser/accessibility/BrowserAccessibilityManager.java
+++ b/content/public/android/java/src/org/chromium/content/browser/accessibility/BrowserAccessibilityManager.java
@@ -368,6 +368,16 @@ public class BrowserAccessibilityManager {
}
}
+ public void onAutofillPopupAccessibilityFocusCleared() {
+ if (mAccessibilityManager.isEnabled() && mNativeObj != 0) {
+ int id = nativeOnAutofillPopupAccessibilityFocusCleared(mNativeObj);
+ if (id == 0) return;
+
+ moveAccessibilityFocusToId(id);
+ nativeScrollToMakeNodeVisible(mNativeObj, mAccessibilityFocusId);
+ }
+ }
+
/**
* @see View#onHoverEvent(MotionEvent)
*/
@@ -431,8 +441,7 @@ public class BrowserAccessibilityManager {
if (id == 0) return false;
moveAccessibilityFocusToId(id);
- nativeScrollToMakeNodeVisible(
- mNativeObj, mAccessibilityFocusId);
+ nativeScrollToMakeNodeVisible(mNativeObj, mAccessibilityFocusId);
return true;
}
@@ -1178,6 +1187,8 @@ public class BrowserAccessibilityManager {
long nativeBrowserAccessibilityManagerAndroid);
private native void nativeOnAutofillPopupDismissed(
long nativeBrowserAccessibilityManagerAndroid);
+ private native int nativeOnAutofillPopupAccessibilityFocusCleared(
+ long nativeBrowserAccessibilityManagerAndroid);
private native int nativeGetRootId(long nativeBrowserAccessibilityManagerAndroid);
private native boolean nativeIsNodeValid(long nativeBrowserAccessibilityManagerAndroid, int id);
private native boolean nativeIsAutofillPopupNode(

Powered by Google App Engine
This is Rietveld 408576698