Add a plain space instead of   between text nodes

third_party/WebKit/Source/core/editing/EditingUtilities.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2007 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 833 matching lines @@
     for (const LayoutObject* r = node.layoutObject(); r; r = r->parent()) {
         if (r->isLayoutBlockFlow()) {
             primaryDirection = r->style()->direction();
             break;
         }
     }
 
     return primaryDirection;
 }
 
-String stringWithRebalancedWhitespace(const String& string, bool startIsStartOfParagraph, bool endIsEndOfParagraph)
+String stringWithRebalancedWhitespace(const String& string, bool startIsStartOfParagraph, bool endIsEndOfParagraph, bool nextTextSibling)
 {
     unsigned length = string.length();
 
     StringBuilder rebalancedString;
     rebalancedString.reserveCapacity(length);
 
     bool previousCharacterWasSpace = false;
     for (size_t i = 0; i < length; i++) {
         UChar c = string[i];
         if (!isWhitespace(c)) {
             rebalancedString.append(c);
             previousCharacterWasSpace = false;
             continue;
         }
 
-        if (previousCharacterWasSpace || (!i && startIsStartOfParagraph) || (i + 1 == length && endIsEndOfParagraph)) {
+        // We need to ensure there is no next sibling text node. See http://crbug.com/310149
+        if (previousCharacterWasSpace || (!i && startIsStartOfParagraph) || (i + 1 == length && endIsEndOfParagraph && !nextTextSibling)) {

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

Should we insert U+0020 instead of U+00A0 when previous sibling is Text node?

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

I'm not sure. In this case, the space is inserted at the end of the first text
node. Anyway, we need to find such a case.

             rebalancedString.append(noBreakSpaceCharacter);
             previousCharacterWasSpace = false;
         } else {
             rebalancedString.append(' ');
             previousCharacterWasSpace = true;
         }
     }
 
     DCHECK_EQ(rebalancedString.length(), length);
 
@@ skipping 935 matching lines @@
 {
     if (!RuntimeEnabledFeatures::inputEventEnabled())
         return DispatchEventResult::NotCanceled;
     if (!target)
         return DispatchEventResult::NotCanceled;
     InputEvent* beforeInputEvent = InputEvent::createBeforeInput(inputType, data, InputEvent::EventCancelable::IsCancelable, InputEvent::EventIsComposing::NotComposing, ranges);
     return target->dispatchEvent(beforeInputEvent);
 }
 
 } // namespace blink