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

Unified Diff: chrome/browser/extensions/api/automation_internal/automation_internal_api.cc

Issue 2255093005: Pass accessible location change messages to the automation API. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Speculative fix to cast_shell Created 4 years, 4 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/extensions/api/automation_internal/automation_internal_api.cc
diff --git a/chrome/browser/extensions/api/automation_internal/automation_internal_api.cc b/chrome/browser/extensions/api/automation_internal/automation_internal_api.cc
index b52ee36034ec5ef1ed7dce87fc262c4a4c040f68..c7f2c9973042602749ce2dd3143cf57995c09885 100644
--- a/chrome/browser/extensions/api/automation_internal/automation_internal_api.cc
+++ b/chrome/browser/extensions/api/automation_internal/automation_internal_api.cc
@@ -203,10 +203,7 @@ class AutomationWebContentsObserver
void AccessibilityEventReceived(
const std::vector<content::AXEventNotificationDetails>& details)
override {
- std::vector<content::AXEventNotificationDetails>::const_iterator iter =
- details.begin();
- for (; iter != details.end(); ++iter) {
- const content::AXEventNotificationDetails& event = *iter;
+ for (const auto& event : details) {
ExtensionMsg_AccessibilityEventParams params;
params.tree_id = event.ax_tree_id;
params.id = event.id;
@@ -220,6 +217,19 @@ class AutomationWebContentsObserver
}
}
+ void AccessibilityLocationChangesReceived(
+ const std::vector<content::AXLocationChangeNotificationDetails>& details)
+ override {
+ for (const auto& src : details) {
+ ExtensionMsg_AccessibilityLocationChangeParams dst;
+ dst.id = src.id;
+ dst.tree_id = src.ax_tree_id;
+ dst.new_location = src.new_location;
+ AutomationEventRouter* router = AutomationEventRouter::GetInstance();
+ router->DispatchAccessibilityLocationChange(dst);
+ }
+ }
+
void RenderFrameDeleted(
content::RenderFrameHost* render_frame_host) override {
int tree_id = render_frame_host->GetAXTreeID();

Powered by Google App Engine
This is Rietveld 408576698