Keep formatting tags included when it is cut or copied.

third_party/WebKit/Source/core/editing/serializers/StyledMarkupSerializer.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 171 matching lines @@
             return markupAccumulator.takeResults();
 
         firstNode = nextPositionOf(visibleStart).deepEquivalent().anchorNode();
 
         if (pastEnd && PositionTemplate<Strategy>::beforeNode(firstNode).compareTo(PositionTemplate<Strategy>::beforeNode(pastEnd)) >= 0) {
             // This condition hits in editing/pasteboard/copy-display-none.html.
             return markupAccumulator.takeResults();
         }
     }
 
-    if (!m_lastClosed)
+    // If there is no the highest node in the selected nodes, |m_lastClosed| can be #text
+    // when its parent is a formatting tag. In this case, #text is wrapped by <span> tag,
+    // but this text should be wrapped by the formatting tag. See http://crbug.com/634482
+    bool shouldAppendParentTag = false;
+    if (!m_lastClosed) {
         m_lastClosed = StyledMarkupTraverser<Strategy>().traverse(firstNode, pastEnd);
+        if (m_lastClosed && m_lastClosed->isTextNode() && isPresentationalHTMLElement(m_lastClosed->parentNode())) {
+            m_lastClosed = m_lastClosed->parentElement();
+            shouldAppendParentTag = true;
+        }
+    }
+
     StyledMarkupTraverser<Strategy> traverser(&markupAccumulator, m_lastClosed);
     Node* lastClosed = traverser.traverse(firstNode, pastEnd);
 
     if (m_highestNodeToBeSerialized && lastClosed) {
         // TODO(hajimehoshi): This is calculated at createMarkupInternal too.
         Node* commonAncestor = Strategy::commonAncestor(*m_start.computeContainerNode(), *m_end.computeContainerNode());
         DCHECK(commonAncestor);
         HTMLBodyElement* body = toHTMLBodyElement(enclosingElementWithTag(Position::firstPositionInNode(commonAncestor), bodyTag));
         HTMLBodyElement* fullySelectedRoot = nullptr;
         // FIXME: Do this for all fully selected blocks, not just the body.
@@ skipping 29 matching lines @@
                 // If the node is not fully selected by the range, then we don't want to keep styles that affect its relationship to the nodes around it
                 // only the ones that affect it and the nodes within it.
                 if (style && style->style())
                     style->style()->removeProperty(CSSPropertyFloat);
                 traverser.wrapWithNode(*ancestor, style);
             }
 
             if (ancestor == m_highestNodeToBeSerialized)
                 break;
         }
+    } else if (shouldAppendParentTag) {
+        EditingStyle* style = traverser.createInlineStyleIfNeeded(*m_lastClosed);
+        traverser.wrapWithNode(*toContainerNode(m_lastClosed), style);
     }
 
     // FIXME: The interchange newline should be placed in the block that it's in, not after all of the content, unconditionally.
     if (shouldAnnotate() && needInterchangeNewlineAt(visibleEnd))
         markupAccumulator.appendInterchangeNewline();
 
     return markupAccumulator.takeResults();
 }
 
 template<typename Strategy>
@@ skipping 221 matching lines @@
     if (element.isHTMLElement() && shouldAnnotate())
         inlineStyle->mergeStyleFromRulesForSerialization(&toHTMLElement(element));
 
     return inlineStyle;
 }
 
 template class StyledMarkupSerializer<EditingStrategy>;
 template class StyledMarkupSerializer<EditingInFlatTreeStrategy>;
 
 } // namespace blink