Remove PREVIOUS_GRAPHEME_TRAILING.

Remove const from GetIndexOfPreviousGrapheme() so the override function could
update local data.
Construct SelectionModel correctly in unittest.

BUG=90426
TEST=--use-pure-view.
Review URL: http://codereview.chromium.org/7607018

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97029 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 2 insertions, 4 deletions

diff --git a/ui/gfx/render_text.h b/ui/gfx/render_text.h
index c3f9104..cc4f4e5 100644
--- a/ui/gfx/render_text.h
+++ b/ui/gfx/render_text.h
@@ -90,15 +90,13 @@ enum BreakType {
 class UI_EXPORT SelectionModel {
  public:
   enum CaretPlacement {
-    // PREVIOUS_GRAPHEME_TRAILING means cursor is visually attached to the
-    // trailing edge of previous grapheme.
-    PREVIOUS_GRAPHEME_TRAILING,
     LEADING,
     TRAILING,
   };
 
   SelectionModel();
   explicit SelectionModel(size_t pos);
+  SelectionModel(size_t start, size_t end);
   SelectionModel(size_t end, size_t pos, CaretPlacement status);
   SelectionModel(size_t start, size_t end, size_t pos, CaretPlacement status);
 
@@ -277,7 +275,7 @@ class UI_EXPORT RenderText {
                                                 BreakType break_type);
 
   // Get the logical index of the grapheme preceeding the argument |position|.
-  virtual size_t GetIndexOfPreviousGrapheme(size_t position) const;
+  virtual size_t GetIndexOfPreviousGrapheme(size_t position);
 
   // Apply composition style (underline) to composition range and selection
   // style (foreground) to selection range.