Initial support for accessibility actions on ARC.

chrome/browser/ui/aura/accessibility/automation_manager_aura.cc

 // Copyright 2014 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/ui/aura/accessibility/automation_manager_aura.h"
 
 #include <stddef.h>
 
 #include <vector>
 
 #include "base/memory/singleton.h"
 #include "build/build_config.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/extensions/api/automation_internal/automation_event_router.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/common/extensions/chrome_extension_messages.h"
 #include "content/public/browser/ax_event_notification_details.h"
 #include "content/public/browser/browser_context.h"
 #include "ui/accessibility/ax_action_data.h"
 #include "ui/accessibility/ax_enums.h"
+#include "ui/accessibility/ax_tree_id_registry.h"
 #include "ui/aura/env.h"
 #include "ui/aura/window.h"
 #include "ui/views/accessibility/ax_aura_obj_wrapper.h"
 #include "ui/views/view.h"
 #include "ui/views/widget/widget.h"
 
 #if defined(OS_CHROMEOS)
 #include "ash/wm/window_util.h"  // nogncheck
 #endif
 
@@ skipping 108 matching lines @@
   if (!enabled_)
     return;
 
   if (!parent)
     parent = current_tree_->GetRoot();
 
   SendEvent(nullptr, parent, ui::AX_EVENT_CHILDREN_CHANGED);
 }
 
 AutomationManagerAura::AutomationManagerAura()
-    : enabled_(false), processing_events_(false) {}
+    : AXHostDelegate(0), enabled_(false), processing_events_(false) {}

[dmazzoni, 2017/02/22 07:44:28]

We really ought to have a constant somewhere in the code for this.
Rather than refactoring it in this change could you at least just add a
comment inside the initializer, something like this?

: AXHostDelegate(0 /* desktop tree id */), enabled_(false), ...

[yawano, 2017/02/22 08:39:05]

api::automation::kDesktopTreeId? -
https://cs.chromium.org/chromium/src/chrome/common/extensions/api/automation_...

[David Tseng, 2017/02/23 18:44:25]

Done.

[David Tseng, 2017/02/23 18:44:25]

Yeah, there's a constant (already being used) just forgot to use it :).

I actually meant to get rid of the id and make RFHI inherit fro m AXHost but
nexdt change.

 
 AutomationManagerAura::~AutomationManagerAura() {
 }
 
 void AutomationManagerAura::ResetSerializer() {
   current_tree_serializer_.reset(
       new AuraAXTreeSerializer(current_tree_.get()));
 }
 
 void AutomationManagerAura::SendEvent(BrowserContext* context,
@@ skipping 35 matching lines @@
 
   processing_events_ = false;
   auto pending_events_copy = pending_events_;
   pending_events_.clear();
   for (size_t i = 0; i < pending_events_copy.size(); ++i) {
     SendEvent(context,
               pending_events_copy[i].first,
               pending_events_copy[i].second);
   }
 }