Chromium Code Reviews| Index: third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp |
| diff --git a/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp b/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp |
| index d7c94cbae624651be48ff5adf40405ab76ca02c0..c73a289c5bef7325fcd1b881bd4b8d87dfe2f4e9 100644 |
| --- a/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp |
| +++ b/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp |
| @@ -458,8 +458,15 @@ bool TextIteratorAlgorithm<Strategy>::handleTextNode() |
| if (!layoutObject->style()->collapseWhiteSpace()) { |
| int runStart = m_offset; |
| if (m_lastTextNodeEndedWithCollapsedSpace && hasVisibleTextNode(layoutObject)) { |
| - emitCharacter(spaceCharacter, textNode, 0, runStart, runStart); |
| - return false; |
| + if (m_behavior & TextIteratorEmitExtraSpaceDeprecated) { |
| + if (runStart > 0 && str[runStart - 1] == ' ') { |
| + emitCharacter(spaceCharacter, textNode, 0, runStart, runStart); |
| + return false; |
| + } |
| + } else { |
| + emitCharacter(spaceCharacter, textNode, 0, runStart, runStart); |
| + return false; |
| + } |
| } |
| if (!m_handledFirstLetter && layoutObject->isTextFragment() && !m_offset) { |
| handleTextNodeFirstLetter(toLayoutTextFragment(layoutObject)); |
| @@ -671,7 +678,15 @@ bool TextIteratorAlgorithm<Strategy>::handleReplacedElement() |
| return true; |
| } |
| - if (m_lastTextNodeEndedWithCollapsedSpace) { |
| + if (m_behavior & TextIteratorEmitExtraSpaceDeprecated) { |
| + if (m_lastTextNode) { |
| + String str = m_lastTextNode->layoutObject()->text(); |
| + if (m_lastTextNodeEndedWithCollapsedSpace && m_offset > 0 && str[m_offset-1] == ' ') { |
|
yosin_UTC9
2016/01/20 01:40:23
nit: need spaces around '-'
ramya.v
2016/01/20 03:42:56
Done.
|
| + emitCharacter(spaceCharacter, Strategy::parent(*m_lastTextNode), m_lastTextNode, 1, 1); |
| + return false; |
| + } |
| + } |
| + } else if (m_lastTextNodeEndedWithCollapsedSpace) { |
| emitCharacter(spaceCharacter, Strategy::parent(*m_lastTextNode), m_lastTextNode, 1, 1); |
| return false; |
| } |