135453 2014-07-30 23:13:53 -0700 REGRESSION: Animations inside RTL multicolumn content is broken 2014-07-31 19:22:47 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Unspecified Unspecified RESOLVED DUPLICATE 135452 P2 Normal --- 1 rniwa webkit-unassigned abucur dino hyatt koivisto mihnea simon.fraser oldest_to_newest 1025918 0 rniwa 2014-07-30 23:13:53 -0700 When a CSS animation is added to an element inside a RTL multicolumn content, the accelerated layer is shown at a wrong location. 1025919 1 235809 rniwa 2014-07-30 23:15:41 -0700 Created attachment 235809 Test case See how "hello" moves between two locations but it shouldn't. It should always show up on the right of "world". 1025920 2 rniwa 2014-07-30 23:22:20 -0700 This one appears to be caused by accumulateOffsetTowardsAncestor adding the offset in: RenderRegion* region = toRenderMultiColumnFlowThread(parentLayer->renderer()).physicalTranslationFromFlowToRegion(location); if (region) location.moveBy(region->topLeftLocation() + -parentLayer->renderBox()->topLeftLocation()); Removing the last two lines here fixes the bug. It's curious how RenderBoxModelObject::adjustedPositionRelativeToOffsetParent has a very similar code: RenderRegion* region = toRenderMultiColumnFlowThread(curr)->physicalTranslationFromFlowToRegion(referencePoint); if (region) referencePoint.moveBy(region->topLeftLocation()); 1025980 3 simon.fraser 2014-07-31 08:40:09 -0700 Isn't this just all RenderLayer content, even without animation? I thought you had an example with opacity. 1025987 4 abucur 2014-07-31 09:32:50 -0700 This bug has the same cause as https://bugs.webkit.org/show_bug.cgi?id=135452. 1026169 5 rniwa 2014-07-31 19:22:47 -0700 Confirmed fixed with http://trac.webkit.org/changeset/171882. Thanks! *** This bug has been marked as a duplicate of bug 135452 *** 235809 2014-07-30 23:15:41 -0700 2014-07-30 23:15:41 -0700 Test case column-bug-layer.html text/html 490 rniwa PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8aGVhZD4KPHN0eWxlPgpALXdlYmtpdC1rZXlmcmFtZXMg YmxpbmtlciB7ICAKICBmcm9tIHsgYmFja2dyb3VuZDogcmVkOyBvcGFjaXR5OiAxOyB9CiAgdG8g eyBiYWNrZ3JvdW5kOiBncmVlbjsgb3BhY2l0eTogMTsgfQp9CgpzcGFuIHsKICAtd2Via2l0LWFu aW1hdGlvbi1uYW1lOiBibGlua2VyOwogIC13ZWJraXQtYW5pbWF0aW9uLWl0ZXJhdGlvbi1jb3Vu dDogaW5maW5pdGU7CiAgLXdlYmtpdC1hbmltYXRpb24tZHVyYXRpb246IDFzOwp9Cjwvc3R5bGU+ CjwvaGVhZD4KPGJvZHk+CjxkaXYgZGlyPSJydGwiIHN0eWxlPSJwb3NpdGlvbjphYnNvbHV0ZTsg bGVmdDogMDsgdG9wOiAwcHg7IHdpZHRoOiA2MDBweDsgaGVpZ2h0OiA2MHB4OwogICAgLXdlYmtp dC1jb2x1bW4tY291bnQ6IDM7IGZvbnQtc2l6ZTogNTBweDsiPgo8c3Bhbj5oZWxsbzwvc3Bhbj4K PGJyPgp3b3JsZAo8L2Rpdj4KPC9ib2R5Pgo8L2h0bWw+Cg==