Fix issue in which removing a class does not always invalidate style

Source/core/css/RuleFeature.cpp

Index: Source/core/css/RuleFeature.cpp
diff --git a/Source/core/css/RuleFeature.cpp b/Source/core/css/RuleFeature.cpp
index b52569a0034cea47262ee54d626699f31eaf38c5..ea0edf74cc9cbe28cbadd82f0fc6e94542f40c6f 100644
--- a/Source/core/css/RuleFeature.cpp
+++ b/Source/core/css/RuleFeature.cpp
@@ -380,9 +380,12 @@ bool RuleFeatureSet::invalidateStyleForClassChangeOnChildren(Element* element, V
 
 bool RuleFeatureSet::invalidateStyleForClassChange(Element* element, Vector<AtomicString>& invalidationClasses, bool foundInvalidationSet)
 {
+    bool thisElementNeedsStyleRecalc = false;
     int oldSize = invalidationClasses.size();
+
     if (element->needsStyleInvalidation()) {
         if (InvalidationList* invalidationList = m_pendingInvalidationMap.get(element)) {

[ojan, 2014/02/18 22:53:39]

Aside: How can this be null? Instead of being an "if" should this just be
followed by an ASSERT(invalidationList)?

[chrishtr, 2014/02/18 23:19:13]

Done.

+            thisElementNeedsStyleRecalc = true;
             foundInvalidationSet = true;
             for (InvalidationList::const_iterator it = invalidationList->begin(); it != invalidationList->end(); ++it) {
                 if ((*it)->wholeSubtreeInvalid()) {
@@ -396,8 +399,6 @@ bool RuleFeatureSet::invalidateStyleForClassChange(Element* element, Vector<Atom
         }
     }
 
-    bool thisElementNeedsStyleRecalc = false;
-
     if (element->hasClass()) {
         const SpaceSplitString& classNames = element->classNames();
         for (Vector<AtomicString>::const_iterator it = invalidationClasses.begin(); it != invalidationClasses.end(); ++it) {