RenderText API Outline.

views/controls/textfield/textfield_views_model.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef VIEWS_CONTROLS_TEXTFIELD_TEXTFIELD_VIEWS_MODEL_H_
 #define VIEWS_CONTROLS_TEXTFIELD_TEXTFIELD_VIEWS_MODEL_H_
 #pragma once
 
 #include <list>
 #include <vector>
@@ skipping 33 matching lines @@
   // Does the edit have to be merged with previous edit?
   // This forces the merge even if the previous edit is marked
   // as DO_NOT_MERGE.
   MERGE_WITH_PREVIOUS,
 };
 
 }  // namespace internal
 
 typedef std::vector<internal::TextStyleRange*> TextStyleRanges;
 
+// TODO(msw): Divide responsibility with RenderText; revise API and fix impls.
+// TODO(msw): Make RenderText a member of TextfieldViewsModel?

[xji, 2011/07/01 18:29:44]

Maybe make RenderText a member of TextfieldViewsModel instead of
NativeTextfieldViews? because the model contains the text itself plus those UI
operations, and you can access model from view, but not the other way around.

[msw, 2011/07/01 21:52:15]

Yeah, I'm leaning in that direction now, thanks for the feedback!

+
 // A model that represents a text content for TextfieldViews.
 // It supports editing, selection and cursor manipulation.
 class TextfieldViewsModel {
  public:
 
   // Delegate interface implemented by the textfield view class to provided
   // additional functionalities required by the model.
   class Delegate {
    public:
     // Called when the current composition text is confirmed or cleared.
@@ skipping 331 matching lines @@
   // List of style ranges. Elements in the list never overlap each other.
   // Elements are not sorted at the time of insertion, and gets sorted
   // when it's painted (if necessary).
   TextStyleRanges style_ranges_;
   // True if the style_ranges_ needs to be sorted.
   bool sort_style_ranges_;
 
   // List of style ranges for composition text.
   TextStyleRanges composition_style_ranges_;
 
+  // TODO(msw): Make RenderText a member of TextfieldViewsModel?
+  //// The stylized text, cursor, selection, and the visual layout model.
+  //gfx::RenderText* render_text_;
+
   DISALLOW_COPY_AND_ASSIGN(TextfieldViewsModel);
 };
 
 }  // namespace views
 
 #endif  // VIEWS_CONTROLS_TEXTFIELD_TEXTFIELD_VIEWS_MODEL_H_