MacViews/a11y: Accessibility actions use AXActionData in AXPlatformNodeDelegate.

ui/accessibility/platform/ax_platform_node_delegate.h

Index: ui/accessibility/platform/ax_platform_node_delegate.h
diff --git a/ui/accessibility/platform/ax_platform_node_delegate.h b/ui/accessibility/platform/ax_platform_node_delegate.h
index 8569e7ff52d493855f96ecc696b6ccf56cb0d752..a46eb9afa46013f07deb57bc6fbf84c9d3e81fde 100644
--- a/ui/accessibility/platform/ax_platform_node_delegate.h
+++ b/ui/accessibility/platform/ax_platform_node_delegate.h
@@ -5,6 +5,7 @@
 #ifndef UI_ACCESSIBILITY_PLATFORM_AX_PLATFORM_NODE_DELEGATE_H_
 #define UI_ACCESSIBILITY_PLATFORM_AX_PLATFORM_NODE_DELEGATE_H_
 
+#include "ui/accessibility/ax_action_data.h"
 #include "ui/accessibility/ax_enums.h"
 #include "ui/accessibility/ax_export.h"
 #include "ui/gfx/geometry/vector2d.h"
@@ -80,21 +81,14 @@ class AX_EXPORT AXPlatformNodeDelegate {
   // Actions.
   //
 
+  // Calls one of the methods below to perform an accessibility action,
+  // switching on the ui::AXAction provided in |data|.
+  virtual void AccessibilityPerformAction(const ui::AXActionData& data) = 0;
+
   // Perform the default action, e.g. click a button, follow a link, or
   // toggle a checkbox.
   virtual void DoDefaultAction() = 0;
 
-  // Change the value of a control, such as the text content of a text field.
-  // If |clear_first| is true, this replaces all text with the |new_value|.
-  // Otherwise this inserts |new_value| at the cursor position, replacing any
-  // selected text. The cursor is placed at the end of |new_value|.
-  virtual bool SetStringValue(const base::string16& new_value,
-                              bool clear_first) = 0;
-
-  // Whether SetStringValue() is callable, i.e. if the string value is not read
-  // only and if the callback exists.
-  virtual bool CanSetStringValue() = 0;
-
   // Focus or unfocus a View, checking if the View is focusable first.
   virtual bool SetFocused(bool focused) = 0;

[tapted, 2016/11/18 04:45:40]

Can this be removed at this level? (perhaps keep it as a non-virtual member on
NativeViewAccessibility)

[Patti Lor, 2016/11/21 01:52:35]

Done.

 };