Commit cf0547b6 authored by sullivan's avatar sullivan

The one word was typed by me, but only after debugging for a long time with Darin.

        - fixed <rdar://problem/4912002> REGRESSION: Series of words marked as a single misspelled word

        * editing/TextIterator.cpp:
        (WebCore::WordAwareIterator::WordAwareIterator):
        We were creating a TextIterator with the default kind, but WordAwareIterator is designed to work
        in concert with CharacterIterator, so they need to use the same kind. CharacterIterator uses
        kind RUNFINDER, so WordAwareIterator needs to do so also.



git-svn-id: svn://svn.chromium.org/blink/trunk@18923 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent d5bffc33
2007-01-17 John Sullivan <sullivan@apple.com>
The one word was typed by me, but only after debugging for a long time with Darin.
- fixed <rdar://problem/4912002> REGRESSION: Series of words marked as a single misspelled word
* editing/TextIterator.cpp:
(WebCore::WordAwareIterator::WordAwareIterator):
We were creating a TextIterator with the default kind, but WordAwareIterator is designed to work
in concert with CharacterIterator, so they need to use the same kind. CharacterIterator uses
kind RUNFINDER, so WordAwareIterator needs to do so also.
2007-01-17 Adele Peterson <adele@apple.com> 2007-01-17 Adele Peterson <adele@apple.com>
Reviewed by Brady. Reviewed by Brady.
......
...@@ -865,7 +865,7 @@ WordAwareIterator::WordAwareIterator() ...@@ -865,7 +865,7 @@ WordAwareIterator::WordAwareIterator()
} }
WordAwareIterator::WordAwareIterator(const Range *r) WordAwareIterator::WordAwareIterator(const Range *r)
: m_previousText(0), m_didLookAhead(false), m_textIterator(r) : m_previousText(0), m_didLookAhead(false), m_textIterator(r, RUNFINDER)
{ {
m_didLookAhead = true; // so we consider the first chunk from the text iterator m_didLookAhead = true; // so we consider the first chunk from the text iterator
advance(); // get in position over the first chunk of text advance(); // get in position over the first chunk of text
...@@ -878,7 +878,7 @@ WordAwareIterator::WordAwareIterator(const Range *r) ...@@ -878,7 +878,7 @@ WordAwareIterator::WordAwareIterator(const Range *r)
// (we looked ahead to the next chunk and found a word boundary) // (we looked ahead to the next chunk and found a word boundary)
// - We built up our own chunk of text from many chunks from the text iterator // - We built up our own chunk of text from many chunks from the text iterator
//FIXME: Perf could be bad for huge spans next to each other that don't fall on word boundaries // FIXME: Perf could be bad for huge spans next to each other that don't fall on word boundaries
void WordAwareIterator::advance() void WordAwareIterator::advance()
{ {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment