Add a plain space instead of   between text nodes

third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.cpp

 /*
  * Copyright (C) 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
  *
  * 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
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 717 matching lines @@
     VisiblePosition visibleUpstreamPos = createVisiblePosition(Position(textNode, upstream));
     VisiblePosition visibleDownstreamPos = createVisiblePosition(Position(textNode, downstream));
 
     String string = text.substring(upstream, length);
     String rebalancedString = stringWithRebalancedWhitespace(string,
     // FIXME: Because of the problem mentioned at the top of this function, we
     // must also use nbsps at the start/end of the string because this function
     // doesn't get all surrounding whitespace, just the whitespace in the
     // current text node.
         isStartOfParagraph(visibleUpstreamPos) || upstream == 0,
-        isEndOfParagraph(visibleDownstreamPos) || (unsigned)downstream == text.length());
+        isEndOfParagraph(visibleDownstreamPos) || (unsigned)downstream == text.length(),

[yosin_UTC9, 2016/07/25 07:42:23]

How about renaming |endIsEndOfParagraph| to |shouldEmitNBSPbeforeEnd|?
If so, we don't need to introduce |nextTextSibling| parameter.

[joone, 2016/07/26 04:16:34]

Done.

+        textNode->nextSibling() && textNode->nextSibling()->isTextNode());
 
     if (string != rebalancedString)
         replaceTextInNodePreservingMarkers(textNode, upstream, length, rebalancedString);
 }
 
 void CompositeEditCommand::prepareWhitespaceAtPositionForSplit(Position& position)
 {
     if (!isRichlyEditablePosition(position))
         return;
     Node* node = position.anchorNode();
@@ skipping 894 matching lines @@
 }
 
 DEFINE_TRACE(CompositeEditCommand)
 {
     visitor->trace(m_commands);
     visitor->trace(m_composition);
     EditCommand::trace(visitor);
 }
 
 } // namespace blink