Issue 318925: Copy and paste sometimes removes spaces between words

Source/core/editing/markup.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2009, 2010, 2011 Google Inc. All rights reserved.
  * Copyright (C) 2011 Igalia S.L.
  * Copyright (C) 2011 Motorola Mobility. 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
@@ skipping 230 matching lines @@
 
 String StyledMarkupAccumulator::renderedText(Node& node, const Range* range)
 {
     if (!node.isTextNode())
         return String();
 
     const Text& textNode = toText(node);
     unsigned startOffset = 0;
     unsigned endOffset = textNode.length();
 
+    TextIteratorBehavior behavior = TextIteratorDefaultBehavior;

[tyoshino (SeeGerritForStatus), 2014/02/17 09:20:51]

Move to right above L254?

     if (range && node == range->startContainer())
         startOffset = range->startOffset();

[tyoshino (SeeGerritForStatus), 2014/02/17 09:20:51]

have one blank line here?

     if (range && node == range->endContainer())
         endOffset = range->endOffset();
+    else if (range)
+        behavior = TextIteratorBehavesAsIfNodesFollowing;

[tyoshino (SeeGerritForStatus), 2014/02/17 09:20:51]

let's factor out |range| checking into outer if-clause.

if (range) {
  if (node == ...)
    startOffSet = ...

  if (node == ...)
    endOffset = ...
  else
    behavior = ...
}

 
     Position start = createLegacyEditingPosition(&node, startOffset);
     Position end = createLegacyEditingPosition(&node, endOffset);
-    return plainText(Range::create(node.document(), start, end).get());
+    return plainText(Range::create(node.document(), start, end).get(), behavior);
 }
 
 String StyledMarkupAccumulator::stringValueForRange(const Node& node, const Range* range)
 {
     if (!range)
         return node.nodeValue();
 
     String str = node.nodeValue();
     if (node == range->endContainer())
         str.truncate(range->endOffset());
@@ skipping 775 matching lines @@
         return;
 
     RefPtr<Text> textNode = toText(node.get());
     RefPtr<Text> textNext = toText(next);
     textNode->appendData(textNext->data());
     if (textNext->parentNode()) // Might have been removed by mutation event.
         textNext->remove(exceptionState);
 }
 
 }