Keep formatting tags included when it is cut or copied.

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

Index: third_party/WebKit/Source/core/editing/EditingUtilities.cpp
diff --git a/third_party/WebKit/Source/core/editing/EditingUtilities.cpp b/third_party/WebKit/Source/core/editing/EditingUtilities.cpp
index f3f043b3d8b617389766cc076cbab5882b1091b5..03f26043df8c37e63090ab05dca7006359d81ab9 100644
--- a/third_party/WebKit/Source/core/editing/EditingUtilities.cpp
+++ b/third_party/WebKit/Source/core/editing/EditingUtilities.cpp
@@ -1136,6 +1136,16 @@ bool isListItem(const Node* n)
     return n && n->layoutObject() && n->layoutObject()->isListItem();
 }
 
+bool isPresentationalHTMLElement(const Node* node)

[yosin_UTC9, 2016/08/12 01:37:35]

Could you make |isPresentationalHTMLElement() to take |const Node&|,
since this function doesn't take |nullptr|?

[joone, 2016/08/12 22:12:21]

Maybe no, because we need to change other methods because they are used as
parameter of highestEnclosingNodeOfType().
=> isPresentationalHTMLElement, isMailHTMLBlockquoteElement, isInline,
isHTMLHeaderElement, isInlineHTMLElementWithStyle.
Other methods may take |nullptr| so we need to figure out if those methods
really take |nullptr|.

+{
+    if (!node->isHTMLElement())
+        return false;
+
+    const HTMLElement& element = toHTMLElement(*node);
+    return element.hasTagName(uTag) || element.hasTagName(sTag) || element.hasTagName(strikeTag)
+        || element.hasTagName(iTag) || element.hasTagName(emTag) || element.hasTagName(bTag) || element.hasTagName(strongTag);
+}
+
 Element* associatedElementOf(const Position& position)
 {
     Node* node = position.anchorNode();