Skip rule matching during animations.

Source/core/animation/ActiveAnimations.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 15 matching lines @@
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #ifndef ActiveAnimations_h
 #define ActiveAnimations_h
 
 #include "core/animation/AnimationStack.h"
 #include "core/animation/css/CSSAnimations.h"
+#include "core/rendering/style/RenderStyle.h"
 #include "wtf/HashCountedSet.h"
 #include "wtf/HashMap.h"
 #include "wtf/RefPtr.h"
 #include "wtf/Vector.h"
 
 namespace blink {
 
 class CSSAnimations;
 class RenderObject;
 class Element;
@@ skipping 22 matching lines @@
 
     // AnimationPlayers which have animations targeting this element.
     AnimationPlayerCountedSet& players() { return m_players; }
 
     bool isEmpty() const { return m_defaultStack.isEmpty() && m_cssAnimations.isEmpty() && m_players.isEmpty(); }
 
     void cancelAnimationOnCompositor();
 
     void updateAnimationFlags(RenderStyle&);
     void setAnimationStyleChange(bool animationStyleChange) { m_animationStyleChange = animationStyleChange; }
+    bool isAnimationStyleChange() const { return m_animationStyleChange; }
+
+    const RenderStyle* baseRenderStyle() const { return m_baseRenderStyle.get(); }
+    void setBaseRenderStyle(PassRefPtr<RenderStyle> renderStyle)

[esprehn, 2014/10/13 18:19:33]

Can we make this out of line instead? Then we don't need to include
RenderStyle.h in this and all over more files.

[rune, 2014/10/13 21:31:48]

Done.

+    {
+#if ENABLE(ASSERT)
+        if (m_baseRenderStyle && renderStyle)
+            ASSERT(*m_baseRenderStyle == *renderStyle);
+#endif
+        m_baseRenderStyle = renderStyle;
+    }
 
 #if !ENABLE(OILPAN)
     void addAnimation(Animation* animation) { m_animations.append(animation); }
     void notifyAnimationDestroyed(Animation* animation) { m_animations.remove(m_animations.find(animation)); }
 #endif
 
     void trace(Visitor*);
 
 private:
-    bool isAnimationStyleChange() const { return m_animationStyleChange; }
-
     AnimationStack m_defaultStack;
     CSSAnimations m_cssAnimations;
     AnimationPlayerCountedSet m_players;
     bool m_animationStyleChange;
+    RefPtr<RenderStyle> m_baseRenderStyle;
 
 #if !ENABLE(OILPAN)
     // FIXME: Oilpan: This is to avoid a reference cycle that keeps Elements alive
     // and won't be needed once the Node hierarchy becomes traceable.
     Vector<Animation*> m_animations;
 #endif
 
     // CSSAnimations checks if a style change is due to animation.
     friend class CSSAnimations;
 };
 
 } // namespace blink
 
 #endif