Don't early return on SubtreeStyleChange for scheduling invalidations.

third_party/WebKit/Source/core/dom/Element.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Peter Kelly (pmk@post.com)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2007 David Smith (catfish.man@gmail.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012, 2013 Apple Inc. All rights reserved.
  *           (C) 2007 Eric Seidel (eric@webkit.org)
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 1190 matching lines @@
 
     document().incDOMTreeVersion();
 
     StyleResolver* styleResolver = document().styleResolver();
 
     if (name == HTMLNames::idAttr) {
         AtomicString oldId = elementData()->idForStyleResolution();
         AtomicString newId = makeIdForStyleResolution(newValue, document().inQuirksMode());
         if (newId != oldId) {
             elementData()->setIdForStyleResolution(newId);
-            if (inActiveDocument() && styleResolver && styleChangeType() < SubtreeStyleChange)
+            if (inActiveDocument() && styleResolver)
                 document().styleEngine().idChangedForElement(oldId, newId, *this);
         }
     } else if (name == classAttr) {
         classAttributeChanged(newValue);
     } else if (name == HTMLNames::nameAttr) {
         setHasName(!newValue.isNull());
     } else if (isStyledElement()) {
         if (name == styleAttr) {
             styleAttributeChanged(newValue, reason);
         } else if (isPresentationAttribute(name)) {
@@ skipping 59 matching lines @@
         return ClassStringContent::Empty;
 
     if (newClassString.is8Bit())
         return classStringHasClassName(newClassString.characters8(), length);
     return classStringHasClassName(newClassString.characters16(), length);
 }
 
 void Element::classAttributeChanged(const AtomicString& newClassString)
 {
     StyleResolver* styleResolver = document().styleResolver();
-    bool testShouldInvalidateStyle = inActiveDocument() && styleResolver && styleChangeType() < SubtreeStyleChange;
+    bool testShouldInvalidateStyle = inActiveDocument() && styleResolver;
 
     ASSERT(elementData());
     ClassStringContent classStringContentType = classStringHasClassName(newClassString);
     const bool shouldFoldCase = document().inQuirksMode();
     if (classStringContentType == ClassStringContent::HasClasses) {
         const SpaceSplitString oldClasses = elementData()->classNames();
         elementData()->setClass(newClassString, shouldFoldCase);
         const SpaceSplitString& newClasses = elementData()->classNames();
         if (testShouldInvalidateStyle)
             document().styleEngine().classChangedForElement(oldClasses, newClasses, *this);
@@ skipping 568 matching lines @@
 
 void Element::pseudoStateChanged(CSSSelector::PseudoType pseudo)
 {
     // We can't schedule invaliation sets from inside style recalc otherwise
     // we'd never process them.
     // TODO(esprehn): Make this an ASSERT and fix places that call into this
     // like HTMLSelectElement.
     if (document().inStyleRecalc())
         return;
     StyleResolver* styleResolver = document().styleResolver();
-    if (inActiveDocument() && styleResolver && styleChangeType() < SubtreeStyleChange)
+    if (inActiveDocument() && styleResolver)
         document().styleEngine().pseudoStateChangedForElement(pseudo, *this);
 }
 
 void Element::setAnimationStyleChange(bool animationStyleChange)
 {
     if (animationStyleChange && document().inStyleRecalc())
         return;
     if (!hasRareData())
         return;
     if (ElementAnimations* elementAnimations = elementRareData()->elementAnimations())
@@ skipping 176 matching lines @@
     }
     return false;
 }
 
 void Element::checkForEmptyStyleChange()
 {
     const ComputedStyle* style = computedStyle();
 
     if (!style && !styleAffectedByEmpty())
         return;
-    if (styleChangeType() >= SubtreeStyleChange)
-        return;
     if (!inActiveDocument())
         return;
     if (!document().styleResolver())
         return;
 
     if (!style || (styleAffectedByEmpty() && (!style->emptyState() || hasChildren())))
         pseudoStateChanged(CSSSelector::PseudoEmpty);
 }
 
 void Element::childrenChanged(const ChildrenChange& change)
@@ skipping 1037 matching lines @@
         updateExtraNamedItemRegistration(oldId, newId);
 }
 
 void Element::willModifyAttribute(const QualifiedName& name, const AtomicString& oldValue, const AtomicString& newValue)
 {
     if (name == HTMLNames::nameAttr) {
         updateName(oldValue, newValue);
     }
 
     if (oldValue != newValue) {
-        if (inActiveDocument() && document().styleResolver() && styleChangeType() < SubtreeStyleChange)
+        if (inActiveDocument() && document().styleResolver())
             document().styleEngine().attributeChangedForElement(name, *this);
 
         if (isUpgradedCustomElement())
             CustomElement::attributeDidChange(this, name.localName(), oldValue, newValue);
     }
 
     if (OwnPtrWillBeRawPtr<MutationObserverInterestGroup> recipients = MutationObserverInterestGroup::createForAttributesMutation(*this, name))
         recipients->enqueueMutationRecord(MutationRecord::createAttributes(this, name, oldValue));
 
     InspectorInstrumentation::willModifyDOMAttr(this, oldValue, newValue);
@@ skipping 499 matching lines @@
 {
 #if ENABLE(OILPAN)
     if (hasRareData())
         visitor->trace(elementRareData());
     visitor->trace(m_elementData);
 #endif
     ContainerNode::trace(visitor);
 }
 
 } // namespace blink