|Summary:||Text with simple line layout not getting pushed below float when there is not enough space for it|
|Product:||WebKit||Reporter:||Ralph T <ralpht+bugs>|
|Component:||Layout and Rendering||Assignee:||Nobody <webkit-unassigned>|
|Severity:||Normal||CC:||commit-queue, esprehn+autocc, glenn, koivisto, kondapallykalyan, mmaxfield, ralpht+bugs, zalan|
|Version:||528+ (Nightly build)|
Description Ralph T 2014-01-14 10:26:34 PST
This page uses an unaccelerated CSS animation (of width, with height set to auto) to grow "float: right" (class "zm right zoom") boxes containing images on hover. If you scroll to "SAPPHIRE" and hover the image on the right then at the end of the animation the text overlaps the image. With simple line layout disabled the text does not overlap the image.
Comment 1 Ralph T 2014-01-14 10:31:57 PST
Created attachment 221177 [details] Screenshot of bad overlapping layout I see this with a build I made using r161903 from yesterday on an elderly macbook air.
Comment 2 Ralph T 2014-01-20 17:16:20 PST
I thought this might have been fixed after Zalan's change r162340, but it still persists. Layout is fine with SimpleLineLayout disabled (I just change canUse() to always return false), so I'm pretty sure this is a SimpleLineLayout bug.
Comment 3 zalan 2014-01-20 20:47:42 PST
Created attachment 221718 [details] reduced test case This is definitely a layout issue and not paint coordinates snapping (what my patch addressed)
Comment 4 Antti Koivisto 2014-01-21 07:19:41 PST
SimpeLineLayout::canUseFor had code to disable the path for constrained float cases like this. I suspect the cached canUseFor is just not cleared after dynamic style change here. Or the existing test is just insufficient.