Finish implementing ChromeVox Next active indicator.

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 f9037c5878f85f3214f68d1043a126e0fc590f90..8282409cc35e642f0fb6afd3bb3304b247ceeea4 100644
--- a/chrome/common/extensions/api/automation.idl
+++ b/chrome/common/extensions/api/automation.idl
@@ -337,6 +337,10 @@
     // The rendered location (as a bounding box) of this node within the frame.
     automation.Rect location;
 
+    // Computes the bounding box of a subrange of this node within the frame.

[Peter Lundblad, 2015/11/20 11:15:03]

Sorry for being ignorant about coordinate systems and stuff, but what does 'in
the frame' mean here? It looks to me like you're making the coordinate screen
absolute, but maybe I am missing something. Do you want to be clear that this
only works for inline text boxes?

[dmazzoni, 2015/11/20 23:35:14]

Good catch. Actually this returns global screen coordinates now, when I fixed it
a while ago I forgot to update the docs here. Fixed both.

I think doing the local to global conversion in C++ makes far more sense.

+    // Returns the same as |location| if range information is not available.
+    static automation.Rect boundsForRange(long startIndex, long endIndex);

[Peter Lundblad, 2015/11/20 11:15:03]

Document what the start and end indices index into.

[dmazzoni, 2015/11/20 23:35:14]

Done.

+
     // The purpose of the node, other than the role, if any.
     DOMString description;