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

Unified Diff: chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js

Issue 2447773002: Use setSequentialFocusNavigationStartingPoint in ChromeVox (Closed)
Patch Set: Created 4 years, 2 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: chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js b/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
index da65208f2ba68c0b2d373f8c8b50f83dc116c6db..1e2a3426cbf589ca381e87d08cd04d90eda51eb9 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
@@ -435,8 +435,13 @@ Background.prototype = {
// Iframes, when focused, causes the child webArea to fire focus event.
// This can result in getting stuck when navigating backward.
if (actionNode.role != RoleType.iframe && !actionNode.state.focused &&
- !AutomationPredicate.structuralContainer(actionNode))
- actionNode.focus();
+ !AutomationPredicate.structuralContainer(actionNode)) {
+ if (actionNode.state.focusable) {
+ actionNode.focus();
+ } else {
+ actionNode.setSequentialFocusNavigationStartingPoint();
+ }
+ }
}
var prevRange = this.currentRange_;
this.setCurrentRange(range);

Powered by Google App Engine
This is Rietveld 408576698