Initial support for accessibility actions on ARC.

chrome/browser/chromeos/arc/accessibility/arc_accessibility_helper_bridge.cc

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/arc/accessibility/arc_accessibility_helper_bridge.h"
 
 #include "base/command_line.h"
 #include "chrome/browser/chromeos/accessibility/accessibility_manager.h"
 #include "chrome/browser/chromeos/arc/accessibility/ax_tree_source_arc.h"
 #include "chromeos/chromeos_switches.h"
@@ skipping 77 matching lines @@
       arc_bridge_service()->accessibility_helper(), Init);
   DCHECK(instance);
   instance->Init(binding_.CreateInterfacePtrAndBind());
 
   chromeos::AccessibilityManager* accessibility_manager =
       chromeos::AccessibilityManager::Get();
   if (base::CommandLine::ForCurrentProcess()->HasSwitch(
           chromeos::switches::kEnableChromeVoxArcSupport)) {
     instance->SetFilter(arc::mojom::AccessibilityFilterType::ALL);
     if (!tree_source_) {
-      tree_source_.reset(new AXTreeSourceArc());
+      tree_source_.reset(new AXTreeSourceArc(tree_id()));
       focus_stealer_.reset(new FocusStealer(tree_source_->tree_id()));
       exo::WMHelper::GetInstance()->AddActivationObserver(this);
     }
   } else if (accessibility_manager &&
              accessibility_manager->IsFocusHighlightEnabled()) {
     instance->SetFilter(arc::mojom::AccessibilityFilterType::FOCUS);
   }
 }
 
 void ArcAccessibilityHelperBridge::OnAccessibilityEventDeprecated(
@@ skipping 43 matching lines @@
   views::Widget* widget = views::Widget::GetWidgetForNativeView(gained_active);
   if (widget && widget->GetContentsView()) {
     views::View* view = widget->GetContentsView();
     if (!view->Contains(focus_stealer_.get()))
       view->AddChildView(focus_stealer_.get());
     focus_stealer_->RequestFocus();
     view->NotifyAccessibilityEvent(ui::AX_EVENT_CHILDREN_CHANGED, false);
   }
 }
 
+void ArcAccessibilityHelperBridge::PerformAction(const ui::AXActionData& data) {
+  arc::mojom::AccessibilityActionType mojo_action =
+      arc::mojom::AccessibilityActionType::FOCUS;

[dmazzoni, 2017/02/20 04:50:46]

Is there a "none"?

Might be clearer to initialize it to "none" and set it to "focus" in the
default case of the switch with a comment saying that we're temporarily
just mapping all others to focus for now.

[David Tseng, 2017/02/21 19:09:05]

Nope, no none. The intention wasn't to map everything to focus, but give the
local an initial value. Doing nothing (returning) for the default case is right.

And, it looks like it's fine (compliation wise) to leave it uninitialized, so
went ahead and did that.

+  switch (data.action) {
+    case ui::AX_ACTION_DO_DEFAULT:
+      mojo_action = arc::mojom::AccessibilityActionType::CLICK;
+      break;
+    default:
+      return;
+  }
+
+  auto* instance = ARC_GET_INSTANCE_FOR_METHOD(
+      arc_bridge_service()->accessibility_helper(), PerformAction);
+  instance->PerformAction(data.target_node_id, mojo_action);
+}
+
 }  // namespace arc