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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/renderer/accessibility/render_accessibility_impl.cc

Issue 2474363002: MacViews/a11y: Accessibility actions use AXActionData in AXPlatformNodeDelegate. (Closed)
Patch Set: Review comments. Created 4 years, 1 month 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
« no previous file with comments | « content/browser/accessibility/browser_accessibility_manager.cc ('k') | ui/accessibility/ax_enums.idl » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/accessibility/render_accessibility_impl.cc
diff --git a/content/renderer/accessibility/render_accessibility_impl.cc b/content/renderer/accessibility/render_accessibility_impl.cc
index d74d873bd1685aaf4d9aec10abf97d6bf1b2e05d..f6a4b851e1035e690534a4247c96f65455689f37 100644
--- a/content/renderer/accessibility/render_accessibility_impl.cc
+++ b/content/renderer/accessibility/render_accessibility_impl.cc
@@ -450,6 +450,9 @@ void RenderAccessibilityImpl::OnPerformAction(
WebAXObject focus = document.accessibilityObjectFromID(data.focus_node_id);
switch (data.action) {
+ case ui::AX_ACTION_BLUR:
+ target.setFocused(false);
+ break;
case ui::AX_ACTION_DECREMENT:
target.decrement();
break;
@@ -474,7 +477,7 @@ void RenderAccessibilityImpl::OnPerformAction(
case ui::AX_ACTION_SET_ACCESSIBILITY_FOCUS:
OnSetAccessibilityFocus(target);
break;
- case ui::AX_ACTION_SET_FOCUS:
+ case ui::AX_ACTION_FOCUS:
// By convention, calling SetFocus on the root of the tree should
// clear the current focus. Otherwise set the focus to the new node.
if (data.target_node_id == root.axID())
« no previous file with comments | « content/browser/accessibility/browser_accessibility_manager.cc ('k') | ui/accessibility/ax_enums.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698