Implement actions for Automation API.

chrome/common/extensions/api/automation_internal.idl

 // 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.
 
 // This is the implementation layer of the chrome.automation API, and is
 // essentially a translation of the internal accessibility tree update system
 // into an extension API.
 namespace automationInternal {
   // A compact representation of the accessibility information for a
   // single web object, in a form that can be serialized and sent from
@@ skipping 10 matching lines @@
     object? int_attributes;
     object? intlist_attributes;
     object? string_attributes;
     long[] child_ids;
   };
 
   // Data for an accessibility event and/or an atomic change to an accessibility
   // tree. See ui/accessibility/ax_tree_update.h for an extended explanation of
   // the tree update format.
   dictionary AXEventParams {
+    // The process id of the renderer host.
+    long process_id;
+
     // The routing id of the web contents that this update is for.
     long routing_id;
 
     // The type of event that this update represents.
     DOMString event_type;
 
     // A vector of nodes to update according to the rules described in
     // ui/ax_tree_update.h.
     AXNodeData[] nodes;
   };
 
-  // Returns the routing_id of the tab whose accessibility was enabled using
-  // enable().
-  callback EnableCallback = void(long routing_id);
+  // All possible actions that can be performed on automation nodes.
+  // TODO(dtseng): Refer to
+  // content/browser/accessibility/browser_accessibility_manager.h:BrowserAccessibilityDelegate

[dmazzoni, 2014/04/01 17:05:25]

nit: wrap to 80 chars

[aboxhall, 2014/04/01 17:49:51]

Difficult to do. I think this could stay as-is, or alternatively replace
content/browser/accessibility with c/b/a or even just leave out?

[David Tseng, 2014/04/01 21:27:15]

I removed this TODO. I initially thought I could reuse the
BrowserAccessibilityDelegate interface for actions, but it was easier to stick
with defining everything on RWH.

+  // (which should live in ui/accessibility).
+  enum ActionType {
+    focus,
+    do_default,
+    make_visible,
+    set_selection
+  };
+
+  // Returns the process_id and routing_id of the tab whose accessibility was
+  // enabled using enable().
+  callback EnableCallback = void(long process_id, long routing_id);
 
   interface Functions {
     // Enable automation of the active tab and retrieves its routing id for use
     // in future updates.
     static void enableCurrentTab(EnableCallback callback);
+
+    static void performAction(long processId,
+                              long routingId,
+                              long automationId,
+                              ActionType actionType,
+                              long[] opt_args);

[dmazzoni, 2014/04/01 17:05:25]

What about args that aren't a long, is there a chance we'll have some in the
future?

[aboxhall, 2014/04/01 17:49:51]

Suggestion: use an object for opt_args and pass by name (e.g. {"start_index": 0,
"end_index": 10}). I think this pattern is used elsewhere in extension APIs.

[David Tseng, 2014/04/01 21:27:15]

Updated with the object approach.

   };
 
   interface Events {
     // Fired when an accessibility event occurs
     static void onAccessibilityEvent(AXEventParams update);
   };
 };