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

Unified Diff: chrome/browser/ui/aura/accessibility/automation_manager_aura.cc

Issue 1155183006: Reimplement automation API on top of C++-backed AXTree. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@automation_faster_2
Patch Set: Copy observers Created 5 years, 6 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/ui/aura/accessibility/automation_manager_aura.cc
diff --git a/chrome/browser/ui/aura/accessibility/automation_manager_aura.cc b/chrome/browser/ui/aura/accessibility/automation_manager_aura.cc
index 91f4a40c328a0acc80afe1a47e6cc5e746a27a5b..1a295a4b25f544bdcb95acd12e32cfa0607f567e 100644
--- a/chrome/browser/ui/aura/accessibility/automation_manager_aura.cc
+++ b/chrome/browser/ui/aura/accessibility/automation_manager_aura.cc
@@ -8,8 +8,9 @@
#include "base/memory/singleton.h"
#include "chrome/browser/browser_process.h"
-#include "chrome/browser/extensions/api/automation_internal/automation_util.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/aura/window.h"
@@ -19,6 +20,7 @@
#include "ui/views/widget/widget.h"
using content::BrowserContext;
+using extensions::AutomationEventRouter;
// static
AutomationManagerAura* AutomationManagerAura::GetInstance() {
@@ -129,18 +131,11 @@ void AutomationManagerAura::ResetSerializer() {
void AutomationManagerAura::SendEvent(BrowserContext* context,
views::AXAuraObjWrapper* aura_obj,
ui::AXEvent event_type) {
- ui::AXTreeUpdate update;
- current_tree_serializer_->SerializeChanges(aura_obj, &update);
-
- // Route this event to special process/routing ids recognized by the
- // Automation API as the desktop tree.
- // TODO(dtseng): Would idealy define these special desktop constants in idl.
- content::AXEventNotificationDetails detail(
- update.node_id_to_clear, update.nodes, event_type, aura_obj->GetID(),
- 0, /* process_id */
- 0 /* routing_id */);
- std::vector<content::AXEventNotificationDetails> details;
- details.push_back(detail);
- extensions::automation_util::DispatchAccessibilityEventsToAutomation(
- details, context, gfx::Vector2d());
+ ExtensionMsg_AccessibilityEventParams params;
+ current_tree_serializer_->SerializeChanges(aura_obj, &params.update);
+ params.tree_id = 0;
+ params.id = aura_obj->GetID();
+ params.event_type = event_type;
+ AutomationEventRouter* router = AutomationEventRouter::GetInstance();
+ router->DispatchAccessibilityEvent(params);
}
« no previous file with comments | « chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js ('k') | chrome/chrome_browser_extensions.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698