CL for perf tryjob on win

third_party/WebKit/Source/core/css/invalidation/StyleInvalidator.cpp

Index: third_party/WebKit/Source/core/css/invalidation/StyleInvalidator.cpp
diff --git a/third_party/WebKit/Source/core/css/invalidation/StyleInvalidator.cpp b/third_party/WebKit/Source/core/css/invalidation/StyleInvalidator.cpp
index c53ebeee71f4992e6e4eb27775e1267cdd1eaf20..bdd725367b52362c8e8b1467a80624cda458dd85 100644
--- a/third_party/WebKit/Source/core/css/invalidation/StyleInvalidator.cpp
+++ b/third_party/WebKit/Source/core/css/invalidation/StyleInvalidator.cpp
@@ -41,34 +41,34 @@ void StyleInvalidator::invalidate(Document& document)
     m_pendingInvalidationMap.clear();
 }
 
-void StyleInvalidator::scheduleInvalidationSetsForNode(const InvalidationLists& invalidationLists, ContainerNode& node)
+void StyleInvalidator::scheduleInvalidationSetsForElement(const InvalidationLists& invalidationLists, Element& element)
 {
-    DCHECK(node.inActiveDocument());
+    ASSERT(element.inActiveDocument());
     bool requiresDescendantInvalidation = false;
 
-    if (node.getStyleChangeType() < SubtreeStyleChange) {
+    if (element.getStyleChangeType() < SubtreeStyleChange) {
         for (auto& invalidationSet : invalidationLists.descendants) {
             if (invalidationSet->wholeSubtreeInvalid()) {
-                node.setNeedsStyleRecalc(SubtreeStyleChange, StyleChangeReasonForTracing::create(StyleChangeReason::StyleInvalidator));
+                element.setNeedsStyleRecalc(SubtreeStyleChange, StyleChangeReasonForTracing::create(StyleChangeReason::StyleInvalidator));
                 requiresDescendantInvalidation = false;
                 break;
             }
 
             if (invalidationSet->invalidatesSelf())
-                node.setNeedsStyleRecalc(LocalStyleChange, StyleChangeReasonForTracing::create(StyleChangeReason::StyleInvalidator));
+                element.setNeedsStyleRecalc(LocalStyleChange, StyleChangeReasonForTracing::create(StyleChangeReason::StyleInvalidator));
 
             if (!invalidationSet->isEmpty())
                 requiresDescendantInvalidation = true;
         }
     }
 
-    if (!requiresDescendantInvalidation && (invalidationLists.siblings.isEmpty() || !node.nextSibling()))
+    if (!requiresDescendantInvalidation && (invalidationLists.siblings.isEmpty() || !element.nextSibling()))
         return;
 
-    node.setNeedsStyleInvalidation();
+    element.setNeedsStyleInvalidation();
 
-    PendingInvalidations& pendingInvalidations = ensurePendingInvalidations(node);
-    if (node.nextSibling()) {
+    PendingInvalidations& pendingInvalidations = ensurePendingInvalidations(element);
+    if (element.nextSibling()) {
         for (auto& invalidationSet : invalidationLists.siblings) {
             if (pendingInvalidations.siblings().contains(invalidationSet))
                 continue;
@@ -80,7 +80,7 @@ void StyleInvalidator::scheduleInvalidationSetsForNode(const InvalidationLists&
         return;
 
     for (auto& invalidationSet : invalidationLists.descendants) {
-        DCHECK(!invalidationSet->wholeSubtreeInvalid());
+        ASSERT(!invalidationSet->wholeSubtreeInvalid());
         if (invalidationSet->isEmpty())
             continue;
         if (pendingInvalidations.descendants().contains(invalidationSet))
@@ -91,8 +91,6 @@ void StyleInvalidator::scheduleInvalidationSetsForNode(const InvalidationLists&
 
 void StyleInvalidator::scheduleSiblingInvalidationsAsDescendants(const InvalidationLists& invalidationLists, ContainerNode& schedulingParent)
 {
-    DCHECK(invalidationLists.descendants.isEmpty());
-
     if (invalidationLists.siblings.isEmpty())
         return;
 
@@ -147,9 +145,9 @@ StyleInvalidator::~StyleInvalidator()
 
 void StyleInvalidator::RecursionData::pushInvalidationSet(const InvalidationSet& invalidationSet)
 {
-    DCHECK(!m_wholeSubtreeInvalid);
-    DCHECK(!invalidationSet.wholeSubtreeInvalid());
-    DCHECK(!invalidationSet.isEmpty());
+    ASSERT(!m_wholeSubtreeInvalid);
+    ASSERT(!invalidationSet.wholeSubtreeInvalid());
+    ASSERT(!invalidationSet.isEmpty());
     if (invalidationSet.treeBoundaryCrossing())
         m_treeBoundaryCrossing = true;
     if (invalidationSet.insertionPointCrossing())
@@ -180,7 +178,7 @@ ALWAYS_INLINE bool StyleInvalidator::RecursionData::matchesCurrentInvalidationSe
 
 bool StyleInvalidator::RecursionData::matchesCurrentInvalidationSetsAsSlotted(Element& element) const
 {
-    DCHECK(m_invalidatesSlotted);
+    ASSERT(m_invalidatesSlotted);
 
     for (const auto& invalidationSet : m_invalidationSets) {
         if (!invalidationSet->invalidatesSlotted())
@@ -204,7 +202,7 @@ void StyleInvalidator::SiblingData::pushInvalidationSet(const SiblingInvalidatio
 bool StyleInvalidator::SiblingData::matchCurrentInvalidationSets(Element& element, RecursionData& recursionData)
 {
     bool thisElementNeedsStyleRecalc = false;
-    DCHECK(!recursionData.wholeSubtreeInvalid());
+    ASSERT(!recursionData.wholeSubtreeInvalid());
 
     unsigned index = 0;
     while (index < m_invalidationEntries.size()) {
@@ -240,7 +238,7 @@ bool StyleInvalidator::SiblingData::matchCurrentInvalidationSets(Element& elemen
 void StyleInvalidator::pushInvalidationSetsForContainerNode(ContainerNode& node, RecursionData& recursionData, SiblingData& siblingData)
 {
     PendingInvalidations* pendingInvalidations = m_pendingInvalidationMap.get(&node);
-    DCHECK(pendingInvalidations);
+    ASSERT(pendingInvalidations);
 
     for (const auto& invalidationSet : pendingInvalidations->siblings())
         siblingData.pushInvalidationSet(toSiblingInvalidationSet(*invalidationSet));
@@ -327,7 +325,7 @@ bool StyleInvalidator::invalidate(Element& element, RecursionData& recursionData
         someChildrenNeedStyleRecalc = invalidateChildren(element, recursionData);
 
     if (thisElementNeedsStyleRecalc) {
-        DCHECK(!recursionData.wholeSubtreeInvalid());
+        ASSERT(!recursionData.wholeSubtreeInvalid());
         element.setNeedsStyleRecalc(LocalStyleChange, StyleChangeReasonForTracing::create(StyleChangeReason::StyleInvalidator));
     } else if (recursionData.hasInvalidationSets() && someChildrenNeedStyleRecalc) {
         // Clone the ComputedStyle in order to preserve correct style sharing, if possible. Otherwise recalc style.