Bug 129259 – Subpixel rendering: Ensure consistent border painting width.

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[alan baradlay]

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.

[alan baradlay]

(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.

[alan baradlay]

Created attachment 225089 [details]
Patch

[Simon Fraser (smfr)]

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?

[alan baradlay]

(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!

[alan baradlay]

Created attachment 225215 [details]
Patch

[alan baradlay]

Comment on attachment 225215 [details]
Patch

EWS testing

[WebKit Commit Bot]

Comment on attachment 225215 [details]
Patch

Clearing flags on attachment: 225215

Committed r164698: <http://trac.webkit.org/changeset/164698>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.