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

Unified Diff: chrome/browser/chromeos/arc/accessibility/arc_accessibility_helper_bridge.cc

Issue 2640123004: Initial support for native accessibility in ARC (Closed)
Patch Set: Rebase. Created 3 years, 10 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/chromeos/arc/accessibility/arc_accessibility_helper_bridge.cc
diff --git a/chrome/browser/chromeos/arc/accessibility/arc_accessibility_helper_bridge.cc b/chrome/browser/chromeos/arc/accessibility/arc_accessibility_helper_bridge.cc
index 6e3c1a48f0a761e6d21c41b53bf39bb0fb0b2bb0..cbe7c699d3d325173048d73859f7d13a7566b8ac 100644
--- a/chrome/browser/chromeos/arc/accessibility/arc_accessibility_helper_bridge.cc
+++ b/chrome/browser/chromeos/arc/accessibility/arc_accessibility_helper_bridge.cc
@@ -4,11 +4,72 @@
#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"
#include "components/arc/arc_bridge_service.h"
-#include "components/exo/wm_helper.h"
+#include "components/exo/shell_surface.h"
+#include "components/exo/surface.h"
+#include "ui/aura/client/aura_constants.h"
#include "ui/aura/window.h"
#include "ui/gfx/geometry/rect.h"
+#include "ui/views/view.h"
+#include "ui/views/widget/widget.h"
+
+namespace {
+
+// This class keeps focus on a |ShellSurface| without interfering with default
+// focus management in |ShellSurface|. For example, touch causes the
+// |ShellSurface| to lose focus to its ancestor containing View.
+class FocusStealer : public views::View {
+ public:
+ explicit FocusStealer(int32_t id) : id_(id) {
+ SetFocusBehavior(views::View::FocusBehavior::ALWAYS);
+ set_owned_by_client();
+ }
+
+ // views::View overrides.
+ void GetAccessibleNodeData(ui::AXNodeData* node_data) override {
+ node_data->AddIntAttribute(ui::AX_ATTR_CHILD_TREE_ID, id_);
+ }
+
+ private:
+ int32_t id_;
+ DISALLOW_COPY_AND_ASSIGN(FocusStealer);
+};
+
+exo::Surface* GetArcSurface(const aura::Window* window) {
+ if (!window)
+ return nullptr;
+
+ exo::Surface* arc_surface = exo::Surface::AsSurface(window);
+ if (!arc_surface)
+ arc_surface = exo::ShellSurface::GetMainSurface(window);
+ return arc_surface;
+}
+
+void DispatchFocusChange(arc::mojom::AccessibilityNodeInfoData* node_data) {
+ chromeos::AccessibilityManager* accessibility_manager =
+ chromeos::AccessibilityManager::Get();
+ if (!accessibility_manager)
+ return;
+
+ exo::WMHelper* wmHelper = exo::WMHelper::GetInstance();
+ aura::Window* focused_window = wmHelper->GetFocusedWindow();
+ if (!focused_window)
+ return;
+
+ aura::Window* toplevel_window = focused_window->GetToplevelWindow();
+
+ gfx::Rect bounds_in_screen = gfx::ScaleToEnclosingRect(
+ node_data->boundsInScreen,
+ 1.0f / toplevel_window->layer()->device_scale_factor());
+
+ accessibility_manager->OnViewFocusedInArc(bounds_in_screen);
+}
+
+} // namespace
namespace arc {
@@ -27,37 +88,75 @@ void ArcAccessibilityHelperBridge::OnInstanceReady() {
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());
+ 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::OnAccessibilityEvent(
+void ArcAccessibilityHelperBridge::OnAccessibilityEventDeprecated(
mojom::AccessibilityEventType event_type,
mojom::AccessibilityNodeInfoDataPtr event_source) {
- if (event_type != mojom::AccessibilityEventType::VIEW_FOCUSED ||
- event_source.is_null()) {
+ if (event_type == arc::mojom::AccessibilityEventType::VIEW_FOCUSED)
+ DispatchFocusChange(event_source.get());
+}
+
+void ArcAccessibilityHelperBridge::OnAccessibilityEvent(
+ mojom::AccessibilityEventDataPtr event_data) {
+ if (tree_source_) {
+ tree_source_->NotifyAccessibilityEvent(event_data.get());
return;
}
- chromeos::AccessibilityManager* accessibility_manager =
- chromeos::AccessibilityManager::Get();
-
- if (!accessibility_manager ||
- !accessibility_manager->IsFocusHighlightEnabled()) {
+ if (event_data->eventType != arc::mojom::AccessibilityEventType::VIEW_FOCUSED)
return;
- }
- exo::WMHelper* wmHelper = exo::WMHelper::GetInstance();
+ CHECK_EQ(1U, event_data.get()->nodeData.size());
+ DispatchFocusChange(event_data.get()->nodeData[0].get());
+}
- aura::Window* focused_window = wmHelper->GetFocusedWindow();
- if (!focused_window)
+void ArcAccessibilityHelperBridge::OnWindowActivated(
+ aura::Window* gained_active,
+ aura::Window* lost_active) {
+ if (gained_active == lost_active || !tree_source_)
return;
- aura::Window* toplevel_window = focused_window->GetToplevelWindow();
+ exo::Surface* active_surface = GetArcSurface(gained_active);
+ exo::Surface* inactive_surface = GetArcSurface(lost_active);
- gfx::Rect bounds_in_screen = gfx::ScaleToEnclosingRect(
- event_source.get()->boundsInScreen,
- 1.0f / toplevel_window->layer()->device_scale_factor());
+ // Detach the accessibility tree from an inactive ShellSurface so that any
+ // client walking the desktop tree gets non-duplicated linearization.
+ if (inactive_surface) {
+ views::Widget* widget = views::Widget::GetWidgetForNativeView(lost_active);
+ if (widget && widget->GetContentsView()) {
+ views::View* view = widget->GetContentsView();
+ view->RemoveChildView(focus_stealer_.get());
+ view->NotifyAccessibilityEvent(ui::AX_EVENT_CHILDREN_CHANGED, false);
+ }
+ }
- accessibility_manager->OnViewFocusedInArc(bounds_in_screen);
+ if (!active_surface)
+ return;
+
+ 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);
+ }
}
} // namespace arc

Powered by Google App Engine
This is Rietveld 408576698