Accessible boundsForRange common impl

content/browser/accessibility/browser_accessibility.h

Index: content/browser/accessibility/browser_accessibility.h
diff --git a/content/browser/accessibility/browser_accessibility.h b/content/browser/accessibility/browser_accessibility.h
index fec4d292dea30b329bc81cac8fc7400fb6603862..d09a95695374f6818f9174eec6de15d80ec7ce00 100644
--- a/content/browser/accessibility/browser_accessibility.h
+++ b/content/browser/accessibility/browser_accessibility.h
@@ -112,6 +112,14 @@ class CONTENT_EXPORT BrowserAccessibility {
   // Returns the bounds of this object in screen coordinates.
   gfx::Rect GetGlobalBoundsRect() const;
 
+  // Returns the bounds of the given range in coordinates relative to the
+  // top-left corner of the overall web area. Only valid when the

[David Tseng, 2013/10/17 21:32:02]

Are you thinking about supporting this for non-static (AXStaticText) nodes (as
we discussed, indexing could follow the same model that Selection/Range does
today in the DOM).

[dmazzoni, 2013/10/21 17:25:38]

Sounds good - but for now, this is strictly only supposed to work for static
text. I'm pretty sure that's sufficient for it to support the native
boundsForRange apis on both Mac and Windows.

+  // role is WebAXRoleStaticText.

[aboxhall, 2013/10/04 18:53:07]

There's no chance this could be called on an inline text box?

[dmazzoni, 2013/10/21 17:25:38]

No. The inline text boxes are not exposed to the platform at all, so there's no
way that call could hever happen. There's a DCHECK at the top also.

+  gfx::Rect GetLocalBoundsForRange(int start, int len) const;

[David Tseng, 2013/10/17 21:32:02]

If the above were true, then perhaps this should be startIndex/endIndex (to
generalize indexing into children and text)..

[dmazzoni, 2013/10/21 17:25:38]

I think we'd want a higher-level function that takes a start node and offset,
and an end node and offset, and computes the bounds from that. A single start
and end within the same node like this wouldn't really handle that case anyway.

Let's think of this as a special case (that's useful now); later when we add the
general case, this one just becomes a private helper function.

+
+  // Same as GetLocalBoundsForRange, in screen coordinates.
+  gfx::Rect GetGlobalBoundsForRange(int start, int len) const;

[David Tseng, 2013/10/17 21:32:02]

Is this also only available for static text?

[dmazzoni, 2013/10/21 17:25:38]

Yes. Updated the comment.

+
   // Returns the deepest descendant that contains the specified point
   // (in global screen coordinates).
   BrowserAccessibility* BrowserAccessibilityForPoint(const gfx::Point& point);