Chromium Code Reviews| Index: Source/core/rendering/compositing/CompositingReasonFinder.cpp |
| diff --git a/Source/core/rendering/compositing/CompositingReasonFinder.cpp b/Source/core/rendering/compositing/CompositingReasonFinder.cpp |
| index 4eeca79afea82c6e1f5012f1bcd92344da58e867..3c87a3eda72aeeb599e7d049ce52d802f8c361f5 100644 |
| --- a/Source/core/rendering/compositing/CompositingReasonFinder.cpp |
| +++ b/Source/core/rendering/compositing/CompositingReasonFinder.cpp |
| @@ -110,7 +110,7 @@ CompositingReasons CompositingReasonFinder::styleDeterminedReasons(RenderObject* |
| 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 f
ajuma
2014/06/06 18:13:31
Done.
|
| directReasons |= CompositingReasonActiveAnimation; |
| if (style->hasWillChangeCompositingHint() && !style->subtreeWillChangeContents()) |
| @@ -166,12 +166,12 @@ CompositingReasons CompositingReasonFinder::nonStyleDeterminedDirectReasons(cons |
| 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 |