Initial support for native accessibility in ARC

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

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_ARC_ACCESSIBILITY_ARC_ACCESSIBILITY_HELPER_BRIDGE_H_
 #define CHROME_BROWSER_CHROMEOS_ARC_ACCESSIBILITY_ARC_ACCESSIBILITY_HELPER_BRIDGE_H_
 
+#include <memory>
+
 #include "components/arc/arc_service.h"
 #include "components/arc/common/accessibility_helper.mojom.h"
 #include "components/arc/instance_holder.h"
+#include "components/exo/wm_helper.h"

[oshima, 2017/02/16 04:16:45]

Hmm, I think you're not supposed to use this outside of exo.

+peng in case I'm wrong.

If you simply want to get activation event, you can use

ui/wm/public/activation_client.h

[David Tseng, 2017/02/16 18:36:40]

It's also being used to get the focused window.

[Peng, 2017/02/16 18:51:04]

I added WMHelper to avoid using ash code in exo directly. It is  because the exo
and ash will live in different processes for mus+ash. If AAHB can archive the
functionality without using ash and WMHelper, it will be great. Otherwise,
probably it is okay to use WMHelper, or refactor WMHelper out to a common place
which can be used by both here and exo. Thought?

[David Tseng, 2017/02/16 19:14:37]

For getting the focused window, we need ash (Shell::GetPrimaryRootWindow) to get
the focus client. It sounds like doing this directly in AAHB won't be sufficient
for mus+ash.

 #include "mojo/public/cpp/bindings/binding.h"
 
+namespace views {
+
+class View;
+
+}  // namespace views
+
 namespace arc {
 
 class ArcBridgeService;
+class AXTreeSourceArc;
 
 // ArcAccessibilityHelperBridge is an instance to receive converted Android
 // accessibility events and info via mojo interface and dispatch them to chrome
 // os components.
 class ArcAccessibilityHelperBridge
     : public ArcService,
       public mojom::AccessibilityHelperHost,
-      public InstanceHolder<mojom::AccessibilityHelperInstance>::Observer {
+      public InstanceHolder<mojom::AccessibilityHelperInstance>::Observer,
+      public exo::WMHelper::ActivationObserver {
  public:
   explicit ArcAccessibilityHelperBridge(ArcBridgeService* bridge_service);
   ~ArcAccessibilityHelperBridge() override;
 
   // InstanceHolder<mojom::AccessibilityHelperInstance>::Observer overrides.
   void OnInstanceReady() override;
 
   // mojom::AccessibilityHelperHost overrides.
-  void OnAccessibilityEvent(
+  void OnAccessibilityEventDeprecated(
       mojom::AccessibilityEventType event_type,
       mojom::AccessibilityNodeInfoDataPtr event_source) override;
+  void OnAccessibilityEvent(
+      mojom::AccessibilityEventDataPtr event_data) override;
 
  private:
+  // exo::WMHelper::ActivationObserver overrides.
+  void OnWindowActivated(aura::Window* gained_active,
+                         aura::Window* lost_active) override;
+
   mojo::Binding<mojom::AccessibilityHelperHost> binding_;
 
+  std::unique_ptr<AXTreeSourceArc> tree_source_;
+  std::unique_ptr<views::View> focus_stealer_;
+
   DISALLOW_COPY_AND_ASSIGN(ArcAccessibilityHelperBridge);
 };
 
 }  // namespace arc
 
 #endif  // CHROME_BROWSER_CHROMEOS_ARC_ACCESSIBILITY_ARC_ACCESSIBILITY_HELPER_BRIDGE_H_