Fix composited bounds of reflections.

Fix composited bounds of reflections.

Previously, they were not taking into account child stacking contexts of the
layer being reflected.

BUG=372837
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175432

[chrishtr, 2014-06-03 18:02:22]

This bug was revealed during refactoring of graphics layers to replace
anchorPoint with transformOrigin.

The layout tests compositing/reflections/nested-reflection-transformed.html and
compositing/reflections/nested-reflection-transformed2.html fail with the
transformOrigin
change but not this fix, so they are sufficient testing.

[ojan, 2014-06-03 23:59:38]

lgtm

https://codereview.chromium.org/313653004/diff/60001/Source/core/rendering/Re...
File Source/core/rendering/RenderLayer.cpp (right):

https://codereview.chromium.org/313653004/diff/60001/Source/core/rendering/Re...
Source/core/rendering/RenderLayer.cpp:3341: void
expandCompositingRectForStackingChildren(const RenderLayer* ancestorLayer,
RenderLayer::CalculateBoundsOptions options, LayoutRect& result)
Did you mean this to be static?

[chrishtr, 2014-06-04 00:01:39]

https://codereview.chromium.org/313653004/diff/60001/Source/core/rendering/Re...
File Source/core/rendering/RenderLayer.cpp (right):

https://codereview.chromium.org/313653004/diff/60001/Source/core/rendering/Re...
Source/core/rendering/RenderLayer.cpp:3341: void
expandCompositingRectForStackingChildren(const RenderLayer* ancestorLayer,
RenderLayer::CalculateBoundsOptions options, LayoutRect& result)
On 2014/06/03 23:59:38, ojan wrote:
> Did you mean this to be static?

Static but not a member function.

[chrishtr, 2014-06-04 00:04:02]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2014-06-04 00:04:20]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/chrishtr@chromium.org/313653004/70001

[commit-bot: I haz the power, 2014-06-04 03:03:10]

Change committed as 175432