Don't schedule invalidations when attributes changed if not needed or incorrect.

Source/core/css/invalidation/StyleInvalidator.cpp

Index: Source/core/css/invalidation/StyleInvalidator.cpp
diff --git a/Source/core/css/invalidation/StyleInvalidator.cpp b/Source/core/css/invalidation/StyleInvalidator.cpp
index 4b9a24cec661e254a1a8deafe63cbcaf000422e2..e130e08eeba9727895cffc6bdfdd0362bd4674de 100644
--- a/Source/core/css/invalidation/StyleInvalidator.cpp
+++ b/Source/core/css/invalidation/StyleInvalidator.cpp
@@ -28,6 +28,7 @@ void StyleInvalidator::invalidate(Document& document)
 
 void StyleInvalidator::scheduleInvalidation(PassRefPtr<DescendantInvalidationSet> invalidationSet, Element& element)
 {
+    ASSERT(element.inActiveDocument() && element.styleChangeType() < SubtreeStyleChange);

[esprehn, 2014/05/08 05:19:06]

Lets split this into two ASSERTs, one per line. When you combine asserts of
totally different properties like this it's hard to know which one is hit in a
stack trace.

[chrishtr, 2014/05/08 05:29:18]

Done.

     InvalidationList& list = ensurePendingInvalidationList(element);
     // If we're already going to invalidate the whole subtree we don't need to store any new sets.
     if (!list.isEmpty() && list.last()->wholeSubtreeInvalid())
@@ -51,8 +52,6 @@ void StyleInvalidator::clearInvalidation(Node& node)
 {
     if (node.isElementNode() && node.needsStyleInvalidation())
         m_pendingInvalidationMap.remove(toElement(&node));
-    node.clearChildNeedsStyleInvalidation();
-    node.clearNeedsStyleInvalidation();
 }
 
 void StyleInvalidator::clearPendingInvalidations()