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

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

Issue 2447773002: Use setSequentialFocusNavigationStartingPoint in ChromeVox (Closed)
Patch Set: Links or controls 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/automation_predicate.js
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_predicate.js b/chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_predicate.js
index 76a901891b599094885f44098c624d00fda7cefd..a9b70e4eb34e730a9d23469d51ed4ede8ce8b634 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_predicate.js
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_predicate.js
@@ -100,14 +100,41 @@ AutomationPredicate.formField = AutomationPredicate.match({
],
anyRole: [
Role.checkBox,
+ Role.colorWell,
David Tseng 2016/10/27 22:42:38 Looks like this is just input type color. Needs an
Role.listBox,
Role.slider,
- Role.tab,
+ Role.switch,
David Tseng 2016/10/27 22:42:38 Didn't know about this role... this one needs an o
dmazzoni 2016/10/27 22:56:02 What would you think if automation exposed a much
Role.tree
]
});
/** @type {AutomationPredicate.Unary} */
+AutomationPredicate.control = AutomationPredicate.match({
+ anyPredicate: [
+ AutomationPredicate.formField,
+ ],
+ anyRole: [
+ Role.disclosureTriangle,
+ Role.menuItem,
+ Role.menuItemCheckBox,
+ Role.menuItemRadio,
+ Role.menuListOption,
+ Role.scrollBar,
+ Role.tab
+ ]
+});
+
+/** @type {AutomationPredicate.Unary} */
+AutomationPredicate.linkOrControl = AutomationPredicate.match({
+ anyPredicate: [
+ AutomationPredicate.control
+ ],
+ anyRole: [
+ Role.link
+ ]
+});
+
+/** @type {AutomationPredicate.Unary} */
AutomationPredicate.landmark = AutomationPredicate.roles([
Role.application,
Role.banner,

Powered by Google App Engine
This is Rietveld 408576698