Fixed position elements should layout relative to transformed container

Fixed position elements should layout relative to transformed container

The original layer coordinates conversion code contains a bug that it forgets
to return for the simple case where the fixed position container found is
already the ancestor layer wanted, and the code falls through to the special
path intended for RenderFlowThread.

R=jchaffraix@chromium.org
BUG=243967
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=153254

[trchen, 2013-05-25 03:50:05]

Hello Julien, this one-line patch fixes a fixed position layout bug. PTAL.
Thanks!

[Julien - ping for review, 2013-05-29 19:53:16]

Sorry for the delay, was gardening and didn't check my review queue.

LGTM!

https://codereview.chromium.org/16057002/diff/1/LayoutTests/fast/block/positi...
File
LayoutTests/fast/block/positioning/fixed-position-transformed-container.html
(right):

https://codereview.chromium.org/16057002/diff/1/LayoutTests/fast/block/positi...
LayoutTests/fast/block/positioning/fixed-position-transformed-container.html:5:
<div style="position:relative; left:500px; top:500px;">
Nit: I would indent the nested <div> to highlight the nested level.

[commit-bot: I haz the power, 2013-06-28 19:18:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/trchen@chromium.org/16057002/11001

[commit-bot: I haz the power, 2013-06-28 19:18:18]

Can't process patch for file
LayoutTests/compositing/gestures/gesture-tapHighlight-simple-div-boxshadow-fixed-nested2-expected.png.
Binary file support is temporarilly disabled due to a bug. Please commit blindly
the binary files first then commit the source change as a separate CL. Sorry for
the annoyance.

[trchen, 2013-06-28 19:22:44]

Committed patchset #3 manually as r153254 (presubmit successful).