Fix cursor positioning regression from r201136. GetCursorPos() shouldn't assume

ui/gfx/render_text.h

Index: ui/gfx/render_text.h
===================================================================
--- ui/gfx/render_text.h	(revision 201137)
+++ ui/gfx/render_text.h	(working copy)
@@ -368,11 +368,11 @@
   // Sets the selection model, the argument is assumed to be valid.
   virtual void SetSelectionModel(const SelectionModel& model);
 
-  // Get the height and horizontal bounds (relative to the left of the text, not
-  // the view) of the glyph starting at |index|. If the glyph is RTL then
+  // Get the horizontal bounds (relative to the left of the text, not the view)
+  // of the glyph starting at |index|. If the glyph is RTL then
   // xspan->is_reversed(). This does not return a Rect because a Rect can't have
   // a negative width.
-  virtual void GetGlyphBounds(size_t index, ui::Range* xspan, int* height) = 0;
+  virtual void GetGlyphBounds(size_t index, ui::Range* xspan) = 0;

[msw, 2013/05/23 19:54:26]

This should now return a ui::Range instead of using an out-param.

[Peter Kasting, 2013/05/23 21:01:30]

Good call.  Done.

 
   // Get the visual bounds containing the logical substring within the |range|.
   // If |range| is empty, the result is empty. These bounds could be visually