Add setSelection function to automation API.

chrome/common/extensions/api/automation.idl

Index: chrome/common/extensions/api/automation.idl
diff --git a/chrome/common/extensions/api/automation.idl b/chrome/common/extensions/api/automation.idl
index b49c3a5e179c58bd70ee76efc89f179bfb0eda68..71f3bbeb1dacdb6f19e8b954c76512e5a9dc350a 100644
--- a/chrome/common/extensions/api/automation.idl
+++ b/chrome/common/extensions/api/automation.idl
@@ -312,7 +312,7 @@
     // The root node of the tree containing this AutomationNode.
     AutomationNode root;
 
-    // Whether this AutomationNode is root node.
+    // Whether this AutomationNode is a root node.
     boolean isRootNode;
 
     // The role of this node.
@@ -422,6 +422,15 @@
     DOMString textInputType;
 
     //
+    // Tree selection attributes (available on root nodes only)
+    //
+
+    AutomationNode? anchorObject;
+    long? anchorOffset;
+    AutomationNode? focusObject;
+    long? focusOffset;

[David Tseng, 2015/09/23 17:57:41]

Please document all of these -- they get picked up as part of the public
documentation for chrome.automation.

+
+    //
     // Range attributes.
     //
 
@@ -553,5 +562,12 @@
     // Remove a tree change observer.
     [nocompile] static void removeTreeChangeObserver(
         TreeChangeObserver observer);
+
+    // Sets the selection in a tree.

[David Tseng, 2015/09/23 17:57:41]

Needs more detail. The anchor and focus nodes need to come from the same tree
right? (same tree here means node1.root === node2.root). Also, this doesn't work
for nodes where root.role == 'desktop'.

+    [nocompile] static void setSelection(

[David Tseng, 2015/09/23 17:57:41]

This is now a little confusing as there's two versions of setSelection (one on
chrome.automation and one on the AutomationNode). The AutomationNode version
needs its documentation updated as it's no longer restricted to text fields.
Maybe call the former chrome.automation.setDocumentSelection or something along
those lines.

+        [instanceOf=AutomationNode] object anchorObject,

[David Tseng, 2015/09/23 17:57:41]

Does the instanceOf property do anything here?

+        long anchorOffset,

[David Tseng, 2015/09/23 17:57:41]

Rather than this list of params, use a dictionary (similar to find).

+        [instanceOf=AutomationNode] object focusObject,
+        long focusOffset);
   };
 };