Use ElementTraversal API in ContainerNode::checkForChildrenAdjacentRuleChanges()

Source/core/dom/ContainerNode.cpp

Index: Source/core/dom/ContainerNode.cpp
diff --git a/Source/core/dom/ContainerNode.cpp b/Source/core/dom/ContainerNode.cpp
index 5fdfa0c736a25f75642599581f2e6fcf84cf2eaa..4686036b207c006a215a7ebbbd0311170ae3ae8a 100644
--- a/Source/core/dom/ContainerNode.cpp
+++ b/Source/core/dom/ContainerNode.cpp
@@ -1116,20 +1116,16 @@ void ContainerNode::checkForChildrenAdjacentRuleChanges()
     bool forceCheckOfAnyElementSibling = false;
     Document& document = this->document();
 
-    for (Node* child = firstChild(); child; child = child->nextSibling()) {
-        if (!child->isElementNode())
-            continue;
-        Element* element = toElement(child);
-        bool childRulesChanged = element->needsStyleRecalc() && element->styleChangeType() >= SubtreeStyleChange;
+    for (Element* childElement = ElementTraversal::firstChild(*this); childElement; childElement = ElementTraversal::nextSibling(*childElement)) {

[esprehn, 2014/06/06 01:19:41]

Just call it child, you don't need to rename it.

[Inactive, 2014/06/06 01:27:59]

Done.

+        bool childRulesChanged = childElement->needsStyleRecalc() && childElement->styleChangeType() >= SubtreeStyleChange;
 
         if (forceCheckOfNextElementCount || forceCheckOfAnyElementSibling)
-            element->setNeedsStyleRecalc(SubtreeStyleChange);
-
-        if (forceCheckOfNextElementCount)
-            forceCheckOfNextElementCount--;
+            childElement->setNeedsStyleRecalc(SubtreeStyleChange);
 
         if (childRulesChanged && hasDirectAdjacentRules)
             forceCheckOfNextElementCount = document.styleEngine()->maxDirectAdjacentSelectors();
+        else if (forceCheckOfNextElementCount)
+            --forceCheckOfNextElementCount;
 
         forceCheckOfAnyElementSibling = forceCheckOfAnyElementSibling || (childRulesChanged && hasIndirectAdjacentRules);
     }