Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(134)

Issue 2674883002: Change SK_GAMMA_CONTRAST in Windows from 0.5 to 1.0. (Closed)

Created:
3 years, 10 months ago by WC Leung
Modified:
3 years, 7 months ago
CC:
chromium-reviews, kojii, eae, drott
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Change SK_GAMMA_CONTRAST in Windows from 0.5 to 1.0. Firefox applies an enhancedContrast of 1.0 instead of the documented 0.5 See https://hg.mozilla.org/mozilla-central/file/tip/gfx/thebes/gfxWindowsPlatform.cpp#l1224 for the actual value and https://hg.mozilla.org/mozilla-central/file/tip/modules/libpref/init/all.js#l3436 for the documented value. This CL changes SK_GAMMA_CONTRAST (windows) from 0.5 to 1.0 to follow the actual behavior of Firefox. Screenshots can be found in bug 534732. BUG=534732, 630917

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M skia/BUILD.gn View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 13 (10 generated)
WC Leung
bungeman@: PTAL. kojii@ and others. FYI. I've checked that this fixes bug 630917, and you ...
3 years, 10 months ago (2017-02-03 13:06:27 UTC) #6
bungeman-chromium
This does not lgtm. Your example in Firefox does not apply because SK_GAMMA_CONTRAST is not ...
3 years, 10 months ago (2017-02-03 14:56:24 UTC) #12
WC Leung
3 years, 10 months ago (2017-02-03 14:58:06 UTC) #13
On 2017/02/03 14:56:24, bungeman-chromium wrote:
> This does not lgtm. Your example in Firefox does not apply because
> SK_GAMMA_CONTRAST is not a thing which is passed to DirectWrite,
> SK_GAMMA_CONTRAST actually controls something else in our mask blitting.

Thanks. I'll try again.

Powered by Google App Engine
This is Rietveld 408576698