Make CompositingReasonFinder::requiresCompositingForAnimation take RenderStyle*

Source/core/rendering/compositing/CompositingReasonFinder.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "core/rendering/compositing/CompositingReasonFinder.h"
 
 #include "CSSPropertyNames.h"
 #include "HTMLNames.h"
 #include "core/frame/FrameView.h"
@@ skipping 92 matching lines @@
 
     if (requiresCompositingForTransform(renderer))
         directReasons |= CompositingReason3DTransform;
 
     if (requiresCompositingForFilters(renderer))
         directReasons |= CompositingReasonFilters;
 
     if (style->backfaceVisibility() == BackfaceVisibilityHidden)
         directReasons |= CompositingReasonBackfaceVisibilityHidden;
 
-    if (requiresCompositingForAnimation(renderer))
+    if (requiresCompositingForAnimation(renderer->style()))

[abarth-chromium, 2014/06/06 18:10:53]

There's already a |style| local variable in this function you can use.

[ajuma, 2014/06/06 18:13:31]

Done.

         directReasons |= CompositingReasonActiveAnimation;
 
     if (style->hasWillChangeCompositingHint() && !style->subtreeWillChangeContents())
         directReasons |= CompositingReasonWillChangeCompositingHint;
 
     ASSERT(!(directReasons & ~CompositingReasonComboAllStyleDeterminedReasons));
     return directReasons;
 }
 
 bool CompositingReasonFinder::requiresCompositingForTransform(RenderObject* renderer) const
@@ skipping 35 matching lines @@
 
     if (requiresCompositingForPositionFixed(renderer, layer, 0))
         directReasons |= CompositingReasonPositionFixed;
 
     directReasons |= renderer->additionalCompositingReasons(m_compositingTriggers);
 
     ASSERT(!(directReasons & CompositingReasonComboAllStyleDeterminedReasons));
     return directReasons;
 }
 
-bool CompositingReasonFinder::requiresCompositingForAnimation(RenderObject* renderer) const
+bool CompositingReasonFinder::requiresCompositingForAnimation(RenderStyle* style) const
 {
-    if (renderer->style()->subtreeWillChangeContents())
-        return renderer->style()->isRunningAnimationOnCompositor();
+    if (style->subtreeWillChangeContents())
+        return style->isRunningAnimationOnCompositor();
 
-    return renderer->style()->shouldCompositeForCurrentAnimations();
+    return style->shouldCompositeForCurrentAnimations();
 }
 
 bool CompositingReasonFinder::requiresCompositingForPosition(RenderObject* renderer, const RenderLayer* layer, RenderLayer::ViewportConstrainedNotCompositedReason* viewportConstrainedNotCompositedReason) const
 {
     return requiresCompositingForPositionSticky(renderer, layer) || requiresCompositingForPositionFixed(renderer, layer, viewportConstrainedNotCompositedReason);
 }
 
 bool CompositingReasonFinder::requiresCompositingForPositionSticky(RenderObject* renderer, const RenderLayer* layer) const
 {
     if (!(m_compositingTriggers & ViewportConstrainedPositionedTrigger))
@@ skipping 79 matching lines @@
             if (viewportConstrainedNotCompositedReason)
                 *viewportConstrainedNotCompositedReason = RenderLayer::NotCompositedForBoundsOutOfView;
             return false;
         }
     }
 
     return true;
 }
 
 }