Revert of Create AXAction and AXActionData as a way to simplify accessibility actions

content/browser/accessibility/browser_accessibility_manager.cc

Index: content/browser/accessibility/browser_accessibility_manager.cc
diff --git a/content/browser/accessibility/browser_accessibility_manager.cc b/content/browser/accessibility/browser_accessibility_manager.cc
index 53768e2712b90aefce483a7fdee29596186508d1..a2cea7b7bfafc51c9581c42039777c02f44171f4 100644
--- a/content/browser/accessibility/browser_accessibility_manager.cc
+++ b/content/browser/accessibility/browser_accessibility_manager.cc
@@ -459,10 +459,7 @@
   if (!child_manager || !child_manager->delegate())
     return;
 
-  ui::AXActionData action_data;
-  action_data.target_point = point;
-  action_data.action = ui::AX_ACTION_HIT_TEST;
-  return child_manager->delegate()->AccessibilityPerformAction(action_data);
+  return child_manager->delegate()->AccessibilityHitTest(point);
 }
 
 void BrowserAccessibilityManager::ActivateFindInPageResult(
@@ -560,13 +557,8 @@
 }
 
 void BrowserAccessibilityManager::SetFocus(const BrowserAccessibility& node) {
-  if (!delegate_)
-    return;
-
-  ui::AXActionData action_data;
-  action_data.action = ui::AX_ACTION_SET_FOCUS;
-  action_data.target_node_id = node.GetId();
-  delegate_->AccessibilityPerformAction(action_data);
+  if (delegate_)
+    delegate_->AccessibilitySetFocus(node.GetId());
 }
 
 void BrowserAccessibilityManager::SetFocusLocallyForTesting(
@@ -584,110 +576,46 @@
 
 void BrowserAccessibilityManager::DoDefaultAction(
     const BrowserAccessibility& node) {
-  if (!delegate_)
-    return;
-
-  ui::AXActionData action_data;
-  action_data.action = ui::AX_ACTION_DO_DEFAULT;
-  action_data.target_node_id = node.GetId();
-  delegate_->AccessibilityPerformAction(action_data);
-}
-
-void BrowserAccessibilityManager::ShowContextMenu(
-    const BrowserAccessibility& node) {
-  if (!delegate_)
-    return;
-
-  ui::AXActionData action_data;
-  action_data.action = ui::AX_ACTION_SHOW_CONTEXT_MENU;
-  action_data.target_node_id = node.GetId();
-  delegate_->AccessibilityPerformAction(action_data);
+  if (delegate_)
+    delegate_->AccessibilityDoDefaultAction(node.GetId());
 }
 
 void BrowserAccessibilityManager::ScrollToMakeVisible(
     const BrowserAccessibility& node, gfx::Rect subfocus) {
-  if (!delegate_)
-    return;
-
-  ui::AXActionData action_data;
-  action_data.target_node_id = node.GetId();
-  action_data.action = ui::AX_ACTION_SCROLL_TO_MAKE_VISIBLE;
-  action_data.target_rect = subfocus;
-  delegate_->AccessibilityPerformAction(action_data);
+  if (delegate_) {
+    delegate_->AccessibilityScrollToMakeVisible(node.GetId(), subfocus);
+  }
 }
 
 void BrowserAccessibilityManager::ScrollToPoint(
     const BrowserAccessibility& node, gfx::Point point) {
-  if (!delegate_)
-    return;
-
-  ui::AXActionData action_data;
-  action_data.target_node_id = node.GetId();
-  action_data.action = ui::AX_ACTION_SCROLL_TO_POINT;
-  action_data.target_point = point;
-  delegate_->AccessibilityPerformAction(action_data);
+  if (delegate_) {
+    delegate_->AccessibilityScrollToPoint(node.GetId(), point);
+  }
 }
 
 void BrowserAccessibilityManager::SetScrollOffset(
     const BrowserAccessibility& node, gfx::Point offset) {
-  if (!delegate_)
-    return;
-
-  ui::AXActionData action_data;
-  action_data.target_node_id = node.GetId();
-  action_data.action = ui::AX_ACTION_SET_SCROLL_OFFSET;
-  action_data.target_point = offset;
-  delegate_->AccessibilityPerformAction(action_data);
+  if (delegate_) {
+    delegate_->AccessibilitySetScrollOffset(node.GetId(), offset);
+  }
 }
 
 void BrowserAccessibilityManager::SetValue(
     const BrowserAccessibility& node,
     const base::string16& value) {
-  if (!delegate_)
-    return;
-
-  ui::AXActionData action_data;
-  action_data.target_node_id = node.GetId();
-  action_data.action = ui::AX_ACTION_SET_VALUE;
-  action_data.value = value;
-  delegate_->AccessibilityPerformAction(action_data);
+  if (delegate_)
+    delegate_->AccessibilitySetValue(node.GetId(), value);
 }
 
 void BrowserAccessibilityManager::SetTextSelection(
     const BrowserAccessibility& node,
     int start_offset,
     int end_offset) {
-  if (!delegate_)
-    return;
-
-  ui::AXActionData action_data;
-  action_data.anchor_node_id = node.GetId();
-  action_data.anchor_offset = start_offset;
-  action_data.focus_node_id = node.GetId();
-  action_data.focus_offset = end_offset;
-  action_data.action = ui::AX_ACTION_SET_SELECTION;
-  delegate_->AccessibilityPerformAction(action_data);
-}
-
-void BrowserAccessibilityManager::SetAccessibilityFocus(
-    const BrowserAccessibility& node) {
-  if (!delegate_)
-    return;
-
-  ui::AXActionData action_data;
-  action_data.action = ui::AX_ACTION_SET_ACCESSIBILITY_FOCUS;
-  action_data.target_node_id = node.GetId();
-  delegate_->AccessibilityPerformAction(action_data);
-}
-
-void BrowserAccessibilityManager::HitTest(const gfx::Point& point) {
-  if (!delegate_)
-    return;
-
-  ui::AXActionData action_data;
-  action_data.action = ui::AX_ACTION_HIT_TEST;
-  action_data.target_point = point;
-  delegate_->AccessibilityPerformAction(action_data);
+  if (delegate_) {
+    delegate_->AccessibilitySetSelection(node.GetId(), start_offset,
+                                         node.GetId(), end_offset);
+  }
 }
 
 gfx::Rect BrowserAccessibilityManager::GetViewBounds() {
@@ -1174,7 +1102,8 @@
   if (delegate()) {
     // This triggers an asynchronous request to compute the true object that's
     // under |screen_point|.
-    HitTest(screen_point - GetViewBounds().OffsetFromOrigin());
+    gfx::Point local_point = screen_point - GetViewBounds().OffsetFromOrigin();
+    delegate()->AccessibilityHitTest(local_point);
 
     // Unfortunately we still have to return an answer synchronously because
     // the APIs were designed that way. The best case scenario is that the