[webkit-reviews] review requested: [Bug 85856] [Chromium] Cannot show the left side of an RTL element with a vertical scrollbar : [Attachment 143519] Patch v2 (used reftests)

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed May 23 03:17:25 PDT 2012


Hironori Bono <hbono at chromium.org> has asked  for review:
Bug 85856: [Chromium] Cannot show the left side of an RTL element with a
vertical scrollbar
https://bugs.webkit.org/show_bug.cgi?id=85856

Attachment 143519: Patch v2 (used reftests)
https://bugs.webkit.org/attachment.cgi?id=143519&action=review

------- Additional Comments from Hironori Bono <hbono at chromium.org>
Greetings Tony,

Thanks for your review and comments. After uploading my previous change, I
noticed it might be better to fix the clientLeft attribute as written in
<http://crbug.com/128500> and use it instead of using scroll-bar width
directly. I have also changed its layout test to a reftest. (This test should
pass both whether USE(RTL_SCROLLBAR) is enabled.)

Regards,

Hironori Bono


More information about the webkit-reviews mailing list