Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(180)

Issue 7248046: Improve dithering in 32bit gradients (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 7 months ago by reed1
Modified:
11 years, 7 months ago
CC:
skia-review_googlegroups.com
Base URL:
http://skia.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Go from a 1x2 to a 2x2 dither cell, and change/simplify the logic for how we compute the "dithered" version of a color to just a graduated fixed-point-round. Also, add this new dither to conical and sweep, which before had no dithering. Disabled for now using SK_IGNORE_GRADIENT_DITHER_FIX. Will enable this and and rebaseline skia. http://code.google.com/p/skia/issues/detail?id=1098 Committed: https://code.google.com/p/skia/source/detail?r=7549

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Patch Set 4 : #

Patch Set 5 : #

Patch Set 6 : #

Patch Set 7 : #

Patch Set 8 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+101 lines, -20 lines) Patch
M src/effects/gradients/SkGradientShader.cpp View 1 2 3 4 5 6 7 4 chunks +41 lines, -2 lines 0 comments Download
M src/effects/gradients/SkGradientShaderPriv.h View 1 2 3 4 5 6 3 chunks +7 lines, -2 lines 0 comments Download
M src/effects/gradients/SkLinearGradient.cpp View 1 2 3 4 5 2 chunks +4 lines, -1 line 0 comments Download
M src/effects/gradients/SkRadialGradient.cpp View 1 2 3 4 5 1 chunk +1 line, -2 lines 0 comments Download
M src/effects/gradients/SkSweepGradient.cpp View 1 2 3 4 5 6 2 chunks +13 lines, -2 lines 0 comments Download
M src/effects/gradients/SkTwoPointConicalGradient.cpp View 1 2 3 4 5 6 8 chunks +35 lines, -11 lines 0 comments Download

Messages

Total messages: 9
reed1
I think this CL is better than the existing code, but having now see what ...
11 years, 7 months ago (2013-01-30 22:44:03 UTC) #1
junov
Do you have any ideas or leads for improving quality?
11 years, 7 months ago (2013-01-31 14:42:17 UTC) #2
junov
If you have to reduce the amplitude by half to make the problem go away, ...
11 years, 7 months ago (2013-01-31 14:45:59 UTC) #3
caryclark1
lgtm but I agree with the CG quality sentiment.
11 years, 7 months ago (2013-01-31 14:55:41 UTC) #4
reed1
I have some experiments in play to try to improve quality, though they all involve ...
11 years, 7 months ago (2013-01-31 15:08:17 UTC) #5
reed1
ptal
11 years, 7 months ago (2013-02-01 21:39:56 UTC) #6
reed1
#5 fixes typo in hex-offsets, found by cary
11 years, 7 months ago (2013-02-01 21:50:32 UTC) #7
junov
On 2013/02/01 21:39:56, reed1 wrote: > ptal Yes! lgtm How does quality compare?
11 years, 7 months ago (2013-02-01 21:54:03 UTC) #8
reed1
11 years, 7 months ago (2013-02-01 21:59:03 UTC) #9
On 2013/02/01 21:54:03, junov wrote:
> On 2013/02/01 21:39:56, reed1 wrote:
> > ptal
> 
> Yes! lgtm
> How does quality compare?

http://code.google.com/p/skia/issues/detail?id=1098

The images attached to this bug show some screenshots.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b