MacViews/a11y: Allow accessibility clients to focus and unfocus focusable Views.

ui/views/accessibility/native_view_accessibility.cc

Index: ui/views/accessibility/native_view_accessibility.cc
diff --git a/ui/views/accessibility/native_view_accessibility.cc b/ui/views/accessibility/native_view_accessibility.cc
index db1c1c472d8c60cf849cae4f8d4980f1622798e0..769442d7f87610640fc6a1aeba17498f650e2e9f 100644
--- a/ui/views/accessibility/native_view_accessibility.cc
+++ b/ui/views/accessibility/native_view_accessibility.cc
@@ -71,7 +71,8 @@ const ui::AXNodeData& NativeViewAccessibility::GetData() {
   data_.AddStringAttribute(ui::AX_ATTR_DESCRIPTION,
                            base::UTF16ToUTF8(description));
 
-  data_.state |= (1 << ui::AX_STATE_FOCUSABLE);
+  if (view_->IsAccessibilityFocusable())

[dmazzoni, 2016/11/14 23:13:37]

This is probably a good change, but just a warning
that it may break something. We made everything in
"views" focusable as a hack years ago when we switched
from native UI to views, because so many things were
not marked focusable that should have been.

I'm guessing we're in much better shape now, but
I wouldn't be surprised if a few things are not
marked focusable that should be.

Would it work to land without this change in M56,
then fix it as a follow-up in M57 so we have lots
of time to catch regressions?

[Patti Lor, 2016/11/15 04:13:35]

That sounds OK to me - either way it will be an improvement in the current
behaviour since it doesn't work at all at the moment. Have split out this change
into https://codereview.chromium.org/2506483002/ for now and won't try to land
this til after Nov 17, which seems to be when m56 splits off. Is there a way we
can pro-actively try and catch these cases instead of waiting for a bug to be
reported? e.g. go through all View subclasses, or something like that?

+    data_.state |= (1 << ui::AX_STATE_FOCUSABLE);
 
   if (!view_->enabled())
     data_.state |= (1 << ui::AX_STATE_DISABLED);
@@ -217,6 +218,17 @@ bool NativeViewAccessibility::CanSetStringValue() {
   return !ui::AXNodeData::IsFlagSet(GetData().state, ui::AX_STATE_READ_ONLY);
 }
 
+bool NativeViewAccessibility::SetFocused(bool focused) {
+  if (!ui::AXNodeData::IsFlagSet(GetData().state, ui::AX_STATE_FOCUSABLE))
+    return false;
+
+  if (focused)
+    view_->RequestFocus();
+  else if (view_->HasFocus())
+    view_->GetFocusManager()->ClearFocus();
+  return true;
+}
+
 void NativeViewAccessibility::OnWidgetDestroying(Widget* widget) {
   if (parent_widget_ == widget) {
     parent_widget_->RemoveObserver(this);