WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Emil A Eklund
Comment 1
2012-03-02 11:37:18 PST
Created
attachment 129931
[details]
Patch
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.
Top of Page
Format For Printing
XML
Clone This Bug