New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 25365 link

Starred by 11 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2009
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug

Restricted
  • Only users with Commit permission may comment.



Sign in to add a comment

REGRESSION: Font rendering blurrified

Reported by evan@chromium.org, Oct 21 2009

Issue description

See text in screenshot.
agl, maybe due to your rounded corners change?
 
blurred.png
8.6 KB View Download

Comment 1 by evan@chromium.org, Oct 21 2009

Bisected down to
http://build.chromium.org/buildbot/perf/dashboard/ui/changelog.html?
url=/trunk/src&range=29271:29319

Comment 2 by agl@chromium.org, Oct 21 2009

 Issue 25173  has been merged into this issue.

Comment 3 by agl@chromium.org, Oct 21 2009

 Issue 25167  has been merged into this issue.

Comment 4 by agl@chromium.org, Oct 21 2009

Status: Assigned

Comment 5 by agl@chromium.org, Oct 21 2009

Confirmed to be r49641.
aw crap.

Comment 7 by agl@chromium.org, Oct 21 2009

Once again, I'm an idiot.

r49641 adds code which puts a layer over the canvas when doing anti-aliased clipping. 
We need this because Skia only supports 1bpp clip masks (see [1]). When drawing 
subpixel text, we merge with the underlying pixel value. In a layer, this is black. 
With anti-aliased text, this is fine because the alpha value means that we'll 
correctly composite the layer when popping the stack, but with subpixel text we need 
three alpha channels, but we only have one. So the blurring is caused by compositing 
without the subpixel information.

1) We could disable drawing subpixel text on a layer.
2) We could copy the contents under the layer into the layer when we create it. I 
think this will work.

I'm going to chat with Brett tomorrow about the options.


[1] http://www.imperialviolet.org/2009/09/02/anti-aliased-clipping.html

Comment 8 by evan@chromium.org, Oct 21 2009

So that means this bug only affects text drawn atop rounded rects.  Which isn't such a 
serious thing, and maybe helps explain why few noticed this.

Comment 10 by evan@chromium.org, Oct 21 2009

Why this wasn't caught by layout tests: our layout tests (and baselines) are run 
without subpixel antialiasing.

Comment 12 by agl@chromium.org, Oct 21 2009

Status: Fixed
Workaround landed in WebKit:

http://trac.webkit.org/changeset/49909

Comment 13 by e...@chromium.org, Oct 23 2009

Did this get released on the linux dev channel? I just got updated and there's blurry 
fonts on the NTP on top of the rounded "Recently closed" tab.

Comment 14 by evan@chromium.org, Oct 23 2009

Yep.  It was fixed in the morning but the afternoon release was branched from just 
before.  *shrug*  The suckiest one is that it gets reddit. ;)

Comment 15 by evan@chromium.org, Oct 25 2009

Keywords: blurred blur smeared bold

Comment 16 by derat@chromium.org, Oct 26 2009

 Issue 25797  has been merged into this issue.

Comment 17 by agl@chromium.org, Oct 26 2009

 Issue 25814  has been merged into this issue.

Comment 18 by evan@chromium.org, Oct 26 2009

Clearly, the proper fix is to go back to aliased rounded corners.  ;)
Project Member

Comment 19 by bugdroid1@chromium.org, Oct 12 2012

Labels: Restrict-AddIssueComment-Commit
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.

Sign in to add a comment