Summary: | Subpixel rendering: Ensure consistent border painting width. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | zalan <zalan> | ||||||
Component: | Layout and Rendering | Assignee: | zalan <zalan> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, esprehn+autocc, glenn, jonlee, kondapallykalyan, simon.fraser | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
zalan
2014-02-24 09:55:16 PST
(In reply to comment #0) > class BorderEdge should return width in device pixels so that when the border rect for painting is calculated, the position does not impact border width, in other words, border widths are consistent across edges. position + fractional border width. Created attachment 225089 [details]
Patch
Comment on attachment 225089 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=225089&action=review > Source/WebCore/rendering/RenderBoxModelObject.cpp:1452 > + float widthForPainting() const { return isPresent ? flooredToDevicePixelWidth : 0.f; } Is the .f required? (In reply to comment #3) > (From update of attachment 225089 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=225089&action=review > > > Source/WebCore/rendering/RenderBoxModelObject.cpp:1452 > > + float widthForPainting() const { return isPresent ? flooredToDevicePixelWidth : 0.f; } > > Is the .f required? Probably just a leftover from my previous attempts. Will remove it before committing. Thanks! Created attachment 225215 [details]
Patch
Comment on attachment 225215 [details]
Patch
EWS testing
Comment on attachment 225215 [details] Patch Clearing flags on attachment: 225215 Committed r164698: <http://trac.webkit.org/changeset/164698> All reviewed patches have been landed. Closing bug. |