Clean up of SelectionModel

ui/gfx/render_text.h

Index: ui/gfx/render_text.h
===================================================================
--- ui/gfx/render_text.h	(revision 103192)
+++ ui/gfx/render_text.h	(working copy)
@@ -100,10 +100,6 @@
   size_t GetCursorPosition() const;
   void SetCursorPosition(size_t position);
 
-  void SetCaretPlacement(SelectionModel::CaretPlacement placement) {
-      selection_model_.set_caret_placement(placement);
-  }
-
   // Moves the cursor left or right. Cursor movement is visual, meaning that
   // left and right are relative to screen, not the directionality of the text.
   // If |select| is false, the selection start is moved to the same position.
@@ -179,6 +175,10 @@
   // Get the logical index of the grapheme following the argument |position|.
   virtual size_t GetIndexOfNextGrapheme(size_t position);
 
+  // Returns the selection model of selection_start_.

[msw, 2011/09/29 02:20:21]

|selection_start_| isn't a member of this class, can you update the comment to
"Return a SelectionModel with the cursor at the current selection's start."

[xji, 2011/10/03 23:18:57]

Done.

+  // The returned value represents a cursor/caret position without a selection.
+  SelectionModel GetSelectionModelForSelectionStart();
+
  protected:
   RenderText();
 
@@ -252,10 +252,6 @@
   // cursor is within the visible display area.
   void UpdateCachedBoundsAndOffset();
 
-  // Returns the selection model of selection_start_.
-  // The returned value represents a cursor/caret position without a selection.
-  SelectionModel GetSelectionModelForSelectionStart();
-
   // Logical UTF-16 string data to be drawn.
   string16 text_;