RESOLVED FIXED Bug 80175
Revert borders to integers for subpixel layout
https://bugs.webkit.org/show_bug.cgi?id=80175
Summary Revert borders to integers for subpixel layout
Emil A Eklund
Reported 2012-03-02 11:33:14 PST
Change border representation and calculation back to integers to ensure that borders given the same width are rendered with the same actual width. This also avoids unnecessary type conversions.
Attachments
Patch (28.69 KB, patch)
2012-03-02 11:37 PST, Emil A Eklund
no flags
Emil A Eklund
Comment 1 2012-03-02 11:37:18 PST
Eric Seidel (no email)
Comment 2 2012-03-02 14:09:25 PST
Comment on attachment 129931 [details] Patch hmm... So even table borders are not sub-pixel precise? I know individual cell-layout for tables is done with integers... but I'm surprised that borders around said table would be?
Emil A Eklund
Comment 3 2012-03-02 14:14:45 PST
(In reply to comment #2) > (From update of attachment 129931 [details]) > hmm... So even table borders are not sub-pixel precise? I know individual cell-layout for tables is done with integers... but I'm surprised that borders around said table would be? Borders are aligned to device pixels to ensure that a box with border a style such as "0.3ex solid black" has all four borders rendered with the same width regardless of the offset. If we did this with subpixel precision the bottom/right border might be pained width a different width.
WebKit Review Bot
Comment 4 2012-03-02 14:31:46 PST
Comment on attachment 129931 [details] Patch Attachment 129931 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/11799129 New failing tests: editing/selection/select-line-break-with-opposite-directionality.html
WebKit Review Bot
Comment 5 2012-03-02 17:48:04 PST
Comment on attachment 129931 [details] Patch Clearing flags on attachment: 129931 Committed r109642: <http://trac.webkit.org/changeset/109642>
WebKit Review Bot
Comment 6 2012-03-02 17:48:15 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.