Avoid duplicate functions/code in core/editing: endTag

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 1816 matching lines @@
 
   if (!position.anchorNode()->isTextNode() ||
       !position.anchorNode()->layoutObject()->style()->preserveNewline())
     return false;
 
   Text* textNode = toText(position.anchorNode());
   unsigned offset = position.offsetInContainerNode();
   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.
 VisibleSelection selectionForParagraphIteration(
     const VisibleSelection& original) {
   VisibleSelection newSelection(original);
   VisiblePosition startOfSelection(newSelection.visibleStart());
   VisiblePosition endOfSelection(newSelection.visibleEnd());
 
   // If the end of the selection to modify is just after a table, and if the
@@ skipping 313 matching lines @@
         return InputType::DeleteSoftLineBackward;
       if (granularity == ParagraphBoundary)
         return InputType::DeleteHardLineBackward;
       return InputType::DeleteContentBackward;
     default:
       return InputType::None;
   }
 }
 
 }  // namespace blink