Bug 31778 - Chromium: make rounded borders heavier.
Summary: Chromium: make rounded borders heavier.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P2 Minor
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-11-21 16:53 PST by Adam Langley
Modified: 2009-12-04 16:39 PST (History)
4 users (show)

See Also:


Attachments
patch (2.20 KB, patch)
2009-11-21 16:55 PST, Adam Langley
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Langley 2009-11-21 16:53:33 PST
Skia draws its rounded corners differently from the other ports.
Whereas they have anti-aliased clipping regions, Skia only has 1-bit
regions. (Which is technically more correct, but somewhat unhelpful
for us.) Instead, with Skia we use a layer and collect all the
clipping paths in effect. When the state is popped, we paint
transparency outside the clipping paths and merge the layer down.

This appears to cause rounded borders to look a little thin, which is
addressed in this patch.

This is well covered by existing tests but will require new baselines
in the Chromium tree.
Comment 1 Adam Langley 2009-11-21 16:55:52 PST
Created attachment 43668 [details]
patch

(Note: please don't cq+ this patch.)
Comment 2 Eric Seidel (no email) 2009-11-23 20:09:07 PST
Comment on attachment 43668 [details]
patch

Marking cq- per Adam's note above.
Comment 3 Eric Seidel (no email) 2009-11-23 20:10:07 PST
Comment on attachment 43668 [details]
patch

LGTM.  CC'd brett so he sees this go by.
Comment 4 Julie Parent 2009-12-04 15:35:12 PST
Comment on attachment 43668 [details]
patch

Setting commit-queue+ as we are ready on Chromium side for this to land.
Comment 5 WebKit Commit Bot 2009-12-04 16:39:34 PST
Comment on attachment 43668 [details]
patch

Clearing flags on attachment: 43668

Committed r51722: <http://trac.webkit.org/changeset/51722>
Comment 6 WebKit Commit Bot 2009-12-04 16:39:39 PST
All reviewed patches have been landed.  Closing bug.