Summary: | [chromium] Win mock scrollbars appear to overflow by a pixel to the right and bottom | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Tony Chang <tony> | ||||||
Component: | Tools / Tests | Assignee: | Tony Chang <tony> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | dpranke, ojan, tkent, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Windows 7 | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 77789 | ||||||||
Attachments: |
|
Description
Tony Chang
2012-01-30 14:01:50 PST
Created attachment 124596 [details]
pixel result with scrollbars
In the attach image, the scrollbars shouldn't overflow the green boxes.
dpranke said he's hoping to work on this this quarter. I tracked this down since I had another ref test fail. The bug is here: http://trac.webkit.org/browser/trunk/Tools/DumpRenderTree/chromium/WebThemeEngineDRTWin.cpp#L51 WebRect is defined as a point + width/height. SkIRect is a left, top, right and bottom edge. The conversion needs to subtract 1 from the width and the height. For example, suppose WebRect is at 0, 0 and has a width and height of 10. This converts to a SkIRect with points at 0, 0, 10, 10, but that's actually a width and height of 11. This also explains why we don't have right/bottom edges on any of our scrollbars, e.g.: http://trac.webkit.org/browser/trunk/LayoutTests/platform/chromium-win/css1/basic/class_as_selector-expected.png We'll need to rebaseline many tests, e.g., all textareas and all scrollbars are different. Created attachment 146351 [details]
Patch
Committed r119748: <http://trac.webkit.org/changeset/119748> Probably all of rectangles drawn by the mock theme have this issue. e.g. LayoutTests/platform/chromium-win/fast/forms/number/input-appearance-spinbutton-layer-expected.png |