Get rid of inefficient uses of childNodeCount()

Source/core/editing/markup.cpp

Index: Source/core/editing/markup.cpp
diff --git a/Source/core/editing/markup.cpp b/Source/core/editing/markup.cpp
index ce7484641725632a31205dbabf8cb6e86c49a584..fa6886a32225592dee24363f1a2cef475db78ca0 100644
--- a/Source/core/editing/markup.cpp
+++ b/Source/core/editing/markup.cpp
@@ -369,7 +369,7 @@ Node* StyledMarkupAccumulator::traverseNodesForSerialization(Node* startNode, No
                 appendStartTag(*n);
 
             // If node has no children, close the tag now.
-            if (!n->childNodeCount()) {
+            if (!n->hasChildNodes()) {
                 if (shouldEmit)
                     appendEndTag(*n);
                 lastClosed = n;
@@ -782,15 +782,19 @@ static void fillContainerFromString(ContainerNode* paragraph, const String& stri
     }
 }
 
-bool isPlainTextMarkup(Node *node)
+bool isPlainTextMarkup(Node* node)
 {
-    if (!node->isElementNode() || !node->hasTagName(divTag) || toElement(node)->hasAttributes())
+    if (!node->isElementNode())
         return false;
 
-    if (node->childNodeCount() == 1 && (node->firstChild()->isTextNode() || (node->firstChild()->firstChild())))
+    Element* element = toElement(node);
+    if (!element->hasTagName(divTag) || !element->hasAttributes())
+        return false;
+
+    if (element->hasOneChild() && (element->firstChild()->isTextNode() || (element->firstChild()->firstChild())))
         return true;
 
-    return (node->childNodeCount() == 2 && isTabSpanTextNode(node->firstChild()->firstChild()) && node->firstChild()->nextSibling()->isTextNode());
+    return element->hasChildCount(2) && isTabSpanTextNode(element->firstChild()->firstChild()) && element->firstChild()->nextSibling()->isTextNode();

[eseidel, 2014/02/19 18:26:50]

The last one can be element->lastChild() instead of
element->firstChild()->nextSibling() if you rather.

[Inactive, 2014/02/19 18:36:32]

Better indeed thanks, Done.

 }
 
 static bool shouldPreserveNewline(const Range& range)