Avoid duplicate functions/code in core/editing: endTag

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 13539b5bda7fc20fc738591c0044097ff6190ac3..314999c2a87a81600cf54862e74f59e23ec7fd72 100644
--- a/third_party/WebKit/Source/core/editing/EditingUtilities.cpp
+++ b/third_party/WebKit/Source/core/editing/EditingUtilities.cpp
@@ -1834,6 +1834,13 @@ bool lineBreakExistsAtPosition(const Position& position) {
   return offset < textNode->length() && textNode->data()[offset] == '\n';
 }
 
+bool elementCannotHaveEndTag(const Node& node) {
+  if (!node.isHTMLElement())
+    return false;
+
+  return !toHTMLElement(node).shouldSerializeEndTag();

[Peter Kasting, 2017/04/06 10:03:35]

Drive-by: Possibly clearer:

  return node.isHTMLElement() && !toHTMLElement(node).shouldSerializeEndTag();

+}
+
 // Modifies selections that have an end point at the edge of a table
 // that contains the other endpoint so that they don't confuse
 // code that iterates over selected paragraphs.