[Webkit-unassigned] [Bug 85856] [Chromium] Cannot show the left side of an RTL element with a vertical scrollbar

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=85856





--- Comment #39 from Hironori Bono <hbono at chromium.org>  2012-08-15 23:31:55 PST ---
Greetings Tony,

Thank you for your comment.
Yes, this is exactly the dirty part of this fix. To describe the background of this code, RenderBlock::paintChildren() cannot render to the place referred by the offset of the RanderObject class somehow when we move the top-left origin. (This function assumes static-positioned children always starts from (borderLeft + paddingLeft, borderTop + paddingTop)?) (That is, this change sets the offset correctly.) On the other hand, localToAbsolute() returns the expected position without this code. So, without this code, RenderBlock::paintChildren() paints RTL children to the different places from the ones returned by localToAbsolute() and it causes the rendering problem <http://crbug.com/139486>. (It is easy to show this inconsistency with DevTool and select an element.)

Regards,

Hironori Bono

(In reply to comment #38)
> It feels wrong that we're moving the paint rectangles.  Why is it better to move the paint offset rather than changing the RenderObject's offset?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.