Restore the collapsed spaces of the text when we copy it

third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp

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 8750da2ab573b4b8fb0b13f107437bb25f87ef9b..3ebb9f12f3a17f28b21f0838ae7aef1d33cf8287 100644
--- a/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp
+++ b/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp
@@ -609,8 +609,15 @@ void TextIteratorAlgorithm<Strategy>::handleTextBox()
                     m_offset = runStart + 1;
                 } else {
                     size_t subrunEnd = str.find('\n', runStart);
-                    if (subrunEnd == kNotFound || subrunEnd > runEnd)
+                    if (subrunEnd == kNotFound || subrunEnd > runEnd) {
                         subrunEnd = runEnd;
+                        // Don't trim the space at the end of string because we need to
+                        // preserve it for copy & paste. See http://crbug.com/318925
+                        if (m_node->nextSibling() && isInline(Strategy::next(*m_node))) {

[esprehn, 2016/08/04 20:54:27]

What is this nextSibling check about? The next() thing with the Strategy is not
necessarily nextSibling, for example it might be display none, or a comment, or
ShadowDOM could be projecting something here.

+                            if (str[str.length() - 1] == ' ' && subrunEnd == str.length() - 1 && str[subrunEnd - 1] != ' ')
+                                ++subrunEnd;
+                        }
+                    }
 
                     m_offset = subrunEnd;
                     emitText(m_node, layoutObject, runStart, subrunEnd);