fix know issues in RenderText

base/i18n/break_iterator.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 BASE_I18N_BREAK_ITERATOR_H_
 #define BASE_I18N_BREAK_ITERATOR_H_
 #pragma once
 
 #include "base/basictypes.h"
 #include "base/string16.h"
@@ skipping 70 matching lines @@
   // character in the string, and when we advance to that position it's the
   // last time Advance() returns true.)
   bool Advance();
 
   // Under BREAK_WORD mode, returns true if the break we just hit is the
   // end of a word. (Otherwise, the break iterator just skipped over e.g.
   // whitespace or punctuation.)  Under BREAK_LINE and BREAK_NEWLINE modes,
   // this distinction doesn't apply and it always retuns false.
   bool IsWord() const;
 
+  // Under BREAK_WORD mode, returns true if |position| is at the end of word or
+  // at the start of word. It always retuns false Under BREAK_LINE and
+  // BREAK_NEWLINE modes,

[msw, 2011/09/09 23:03:24]

use a period instead of a comma at the end of this comment.

[xji, 2011/09/12 21:31:48]

Done.

+  bool IsEndOfWord(size_t position) const;
+  bool IsStartOfWord(size_t position) const;
+
   // Returns the string between prev() and pos().
   // Advance() must have been called successfully at least once for pos() to
   // have advanced to somewhere useful.
   string16 GetString() const;
 
   // Returns the value of pos() returned before Advance() was last called.
   size_t prev() const { return prev_; }
 
   // Returns the current break position within the string,
   // or BreakIterator::npos when done.
@@ skipping 15 matching lines @@
   // Previous and current iterator positions.
   size_t prev_, pos_;
 
   DISALLOW_COPY_AND_ASSIGN(BreakIterator);
 };
 
 }  // namespace i18n
 }  // namespace base
 
 #endif  // BASE_I18N_BREAK_ITERATOR_H_