Restore a collapsed leading space of text used for line break

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

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2005 Alexey Proskuryakov.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 601 matching lines @@
                     if (subrunEnd == kNotFound || subrunEnd > runEnd) {
                         subrunEnd = runEnd;
                         // Restore the collapsed space for copy & paste.
                         // See http://crbug.com/318925
                         // For trailing space.
                         if (!nextTextBox && m_textBox->root().nextRootBox() && m_textBox->root().lastChild() == m_textBox) {
                             if (str.endsWith(' ') && subrunEnd == str.length() - 1 && str[subrunEnd - 1] != ' ')
                                 ++subrunEnd;
                         }
                         // For leading space.
-                        if (m_textBox->root().prevRootBox() && m_textBox->root().firstChild() == m_textBox) {
+                        if (!(m_behavior & TextIteratorEmitsImageAltText) && !(m_behavior & TextIteratorDoesNotBreakAtReplacedElement) && !(m_behavior & TextIteratorForInnerText)

[yosin_UTC9, 2016/09/13 02:00:30]

Could you use |emitsImageAltText()| and introduce private inline functions for
others like others?

[joone, 2016/09/14 04:04:35]

Done.

+                            && m_textBox->root().prevRootBox() && m_textBox->root().firstChild() == m_textBox) {
                             InlineBox* lastChildOfPrevRoot = m_textBox->root().prevRootBox()->lastChild();
-                            if (!lastChildOfPrevRoot->isText() && !lastChildOfPrevRoot->getLineLayoutItem().isBR()
+                            if (m_textBox->getLineLayoutItem() != lastChildOfPrevRoot->getLineLayoutItem() && !lastChildOfPrevRoot->getLineLayoutItem().isBR()
                                 && !lastChildOfPrevRoot->isInlineFlowBox()) {
-                                if (runStart > 0 && str[0] == ' ')
+                                if (runStart > 0 && str[0] == ' ' && str[1] != ' ')
                                     --runStart;
                             }
                         }
                     }
 
                     m_offset = subrunEnd;
                     emitText(m_node, layoutObject, runStart, subrunEnd);
                 }
 
                 // If we are doing a subrun that doesn't go to the end of the text box,
@@ skipping 579 matching lines @@
 
 String plainText(const EphemeralRangeInFlatTree& range, TextIteratorBehaviorFlags behavior)
 {
     return createPlainText<EditingInFlatTreeStrategy>(range, behavior);
 }
 
 template class CORE_TEMPLATE_EXPORT TextIteratorAlgorithm<EditingStrategy>;
 template class CORE_TEMPLATE_EXPORT TextIteratorAlgorithm<EditingInFlatTreeStrategy>;
 
 } // namespace blink