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

[Alexei Svitkine (slow), 2013/05/23 21:03:47]

Please update this comment to not mention xpan anymore.

[Peter Kasting, 2013/05/23 21:06:58]

Oops.  Fixed (but not re-uploaded).

   // a negative width.
-  virtual void GetGlyphBounds(size_t index, ui::Range* xspan, int* height) = 0;
+  virtual ui::Range GetGlyphBounds(size_t index) = 0;
 
   // Get the visual bounds containing the logical substring within the |range|.
   // If |range| is empty, the result is empty. These bounds could be visually