Bug 86775

Summary: Floor y coordinate in Chromium's GraphicsContext::drawLineForText to avoid anti-aliasing
Product: WebKit Reporter: Levi Weintraub <leviw>
Component: Layout and RenderingAssignee: Levi Weintraub <leviw>
Status: RESOLVED FIXED    
Severity: Normal CC: eae, eric
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 85555    
Attachments:
Description Flags
Patch none

Description Levi Weintraub 2012-05-17 14:19:36 PDT
Currently we take floats but always paint at whole y coordinates. This patch will preserve that behavior with the sub-pixel conversion. Otherwise, some underlines and strike-throughs end up midway between two pixels.
Comment 1 Levi Weintraub 2012-05-17 14:26:35 PDT
Created attachment 142556 [details]
Patch
Comment 2 Eric Seidel (no email) 2012-05-17 14:28:18 PDT
Comment on attachment 142556 [details]
Patch

Does Skia floor the X for us?  I'm confused?
Comment 3 Levi Weintraub 2012-05-17 18:54:14 PDT
(In reply to comment #2)
> (From update of attachment 142556 [details])
> Does Skia floor the X for us?  I'm confused?

I'm going to add a comment. We always draw these lines horizontally. Sub-pixel x values are normal and acceptable.
Comment 4 Levi Weintraub 2012-05-24 10:23:55 PDT
Forgot to comment... landed in r117528 <http://trac.webkit.org/changeset/117528>