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

Issue 2350003003: Add ability to pick a different gamut in GPU f16 or sRGB configs (Closed)

Created:
4 years, 3 months ago by Brian Osman
Modified:
4 years, 3 months ago
Reviewers:
msarett, bsalomon, mtklein, reed1
CC:
reviews_skia.org
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

Add ability to pick a different gamut in GPU f16 or sRGB configs For now, the only options are sRGB or WideGamutRGB Basically, the color option to the gpu config is now of the form: 8888|srgb[_gamut]|f16[_gamut] color=8888 still implies legacy behavior srgb implies 8-bit gamma-correct rendering (via sRGB format) f16 implies 16-bit gamma-correct rendering (via F16 format) Either of the last two options can then optionally include a gamut specifier, either _srgb or _wide. _srgb selects the (default) sRGB gamut _wide selects the Adobe Wide Gamut RGB gamut, which is nice for testing, in that it's significantly wider than sRGB, so rendering differences are obvious. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2350003003 Committed: https://skia.googlesource.com/skia/+/4562f6ecda4dd819561e3eee17defd525017b3e1

Patch Set 1 #

Total comments: 1

Patch Set 2 : Fix test code #

Patch Set 3 : Switched to glwide and vkwide as new configs #

Unified diffs Side-by-side diffs Delta from patch set Stats (+54 lines, -10 lines) Patch
M tests/TestConfigParsing.cpp View 1 2 2 chunks +6 lines, -1 line 0 comments Download
M tools/flags/SkCommonFlagsConfig.cpp View 1 2 8 chunks +48 lines, -9 lines 0 comments Download

Messages

Total messages: 23 (16 generated)
Brian Osman
https://codereview.chromium.org/2350003003/diff/1/tools/flags/SkCommonFlagsConfig.cpp File tools/flags/SkCommonFlagsConfig.cpp (right): https://codereview.chromium.org/2350003003/diff/1/tools/flags/SkCommonFlagsConfig.cpp#newcode317 tools/flags/SkCommonFlagsConfig.cpp:317: SkStrSplit(value.c_str(), "_", &commands); I realize this is unfortunate (introducing ...
4 years, 3 months ago (2016-09-19 19:39:19 UTC) #6
bsalomon
I think with a little more effort we will kill off the 'gpu' configs in ...
4 years, 3 months ago (2016-09-19 20:53:03 UTC) #13
Brian Osman
On 2016/09/19 20:53:03, bsalomon wrote: > I think with a little more effort we will ...
4 years, 3 months ago (2016-09-19 21:01:11 UTC) #14
Brian Osman
color option parsing remains the same. Pre-defined configs are now glwide and vkwide. I'm really ...
4 years, 3 months ago (2016-09-19 21:20:16 UTC) #17
bsalomon
lgtm On 2016/09/19 21:20:16, Brian Osman wrote: > color option parsing remains the same. Pre-defined ...
4 years, 3 months ago (2016-09-19 21:28:12 UTC) #18
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2350003003/40001
4 years, 3 months ago (2016-09-19 21:33:53 UTC) #21
commit-bot: I haz the power
4 years, 3 months ago (2016-09-19 21:42:07 UTC) #23
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/4562f6ecda4dd819561e3eee17defd525017b3e1

Powered by Google App Engine
This is Rietveld 408576698