blob: 03c9e468acc533a3207e6de3c3a0002a825c86b9 [file] [log] [blame]
bsalomon16921ec2015-07-30 15:34:56 -07001/*
2 * Copyright 2015 Google Inc.
3 *
4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file.
6 */
7
Mike Kleinc0bd9f92019-04-23 12:05:21 -05008#include "include/core/SkCanvas.h"
9#include "include/core/SkSurface.h"
Robert Phillips6d344c32020-07-06 10:56:46 -040010#include "include/gpu/GrDirectContext.h"
Mike Kleinc0bd9f92019-04-23 12:05:21 -050011#include "src/gpu/GrCaps.h"
Adlai Hollera0693042020-10-14 11:23:11 -040012#include "src/gpu/GrDirectContextPriv.h"
Brian Salomonf2ebdd92019-09-30 12:15:30 -040013#include "src/gpu/GrImageInfo.h"
Mike Kleinc0bd9f92019-04-23 12:05:21 -050014#include "src/gpu/GrSurfaceContext.h"
15#include "src/gpu/SkGr.h"
Robert Phillips4dca8312021-07-28 15:13:20 -040016#include "src/gpu/v1/SurfaceDrawContext_v1.h"
Mike Kleinc0bd9f92019-04-23 12:05:21 -050017#include "tests/Test.h"
bsalomon16921ec2015-07-30 15:34:56 -070018
19// using anonymous namespace because these functions are used as template params.
20namespace {
21/** convert 0..1 srgb value to 0..1 linear */
22float srgb_to_linear(float srgb) {
23 if (srgb <= 0.04045f) {
24 return srgb / 12.92f;
25 } else {
26 return powf((srgb + 0.055f) / 1.055f, 2.4f);
27 }
28}
29
30/** convert 0..1 linear value to 0..1 srgb */
31float linear_to_srgb(float linear) {
32 if (linear <= 0.0031308) {
33 return linear * 12.92f;
34 } else {
35 return 1.055f * powf(linear, 1.f / 2.4f) - 0.055f;
36 }
37}
John Stilesa6841be2020-08-06 14:11:56 -040038} // namespace
bsalomon16921ec2015-07-30 15:34:56 -070039
40/** tests a conversion with an error tolerance */
41template <float (*CONVERT)(float)> static bool check_conversion(uint32_t input, uint32_t output,
42 float error) {
43 // alpha should always be exactly preserved.
44 if ((input & 0xff000000) != (output & 0xff000000)) {
45 return false;
46 }
47
48 for (int c = 0; c < 3; ++c) {
49 uint8_t inputComponent = (uint8_t) ((input & (0xff << (c*8))) >> (c*8));
Brian Osman788b9162020-02-07 10:36:46 -050050 float lower = std::max(0.f, (float) inputComponent - error);
51 float upper = std::min(255.f, (float) inputComponent + error);
bsalomon16921ec2015-07-30 15:34:56 -070052 lower = CONVERT(lower / 255.f);
53 upper = CONVERT(upper / 255.f);
54 SkASSERT(lower >= 0.f && lower <= 255.f);
55 SkASSERT(upper >= 0.f && upper <= 255.f);
56 uint8_t outputComponent = (output & (0xff << (c*8))) >> (c*8);
57 if (outputComponent < SkScalarFloorToInt(lower * 255.f) ||
58 outputComponent > SkScalarCeilToInt(upper * 255.f)) {
59 return false;
60 }
61 }
62 return true;
63}
64
65/** tests a forward and backward conversion with an error tolerance */
66template <float (*FORWARD)(float), float (*BACKWARD)(float)>
67static bool check_double_conversion(uint32_t input, uint32_t output, float error) {
68 // alpha should always be exactly preserved.
69 if ((input & 0xff000000) != (output & 0xff000000)) {
70 return false;
71 }
72
73 for (int c = 0; c < 3; ++c) {
74 uint8_t inputComponent = (uint8_t) ((input & (0xff << (c*8))) >> (c*8));
Brian Osman788b9162020-02-07 10:36:46 -050075 float lower = std::max(0.f, (float) inputComponent - error);
76 float upper = std::min(255.f, (float) inputComponent + error);
bsalomon16921ec2015-07-30 15:34:56 -070077 lower = FORWARD(lower / 255.f);
78 upper = FORWARD(upper / 255.f);
79 SkASSERT(lower >= 0.f && lower <= 255.f);
80 SkASSERT(upper >= 0.f && upper <= 255.f);
81 uint8_t upperComponent = SkScalarCeilToInt(upper * 255.f);
82 uint8_t lowerComponent = SkScalarFloorToInt(lower * 255.f);
Brian Osman788b9162020-02-07 10:36:46 -050083 lower = std::max(0.f, (float) lowerComponent - error);
84 upper = std::min(255.f, (float) upperComponent + error);
bsalomon16921ec2015-07-30 15:34:56 -070085 lower = BACKWARD(lowerComponent / 255.f);
86 upper = BACKWARD(upperComponent / 255.f);
87 SkASSERT(lower >= 0.f && lower <= 255.f);
88 SkASSERT(upper >= 0.f && upper <= 255.f);
89 upperComponent = SkScalarCeilToInt(upper * 255.f);
90 lowerComponent = SkScalarFloorToInt(lower * 255.f);
91
92 uint8_t outputComponent = (output & (0xff << (c*8))) >> (c*8);
93 if (outputComponent < lowerComponent || outputComponent > upperComponent) {
94 return false;
95 }
96 }
97 return true;
98}
99
100static bool check_srgb_to_linear_conversion(uint32_t srgb, uint32_t linear, float error) {
101 return check_conversion<srgb_to_linear>(srgb, linear, error);
102}
103
104static bool check_linear_to_srgb_conversion(uint32_t linear, uint32_t srgb, float error) {
105 return check_conversion<linear_to_srgb>(linear, srgb, error);
106}
107
108static bool check_linear_to_srgb_to_linear_conversion(uint32_t input, uint32_t output, float error) {
109 return check_double_conversion<linear_to_srgb, srgb_to_linear>(input, output, error);
110}
111
112static bool check_srgb_to_linear_to_srgb_conversion(uint32_t input, uint32_t output, float error) {
113 return check_double_conversion<srgb_to_linear, linear_to_srgb>(input, output, error);
114}
115
Brian Salomonf802e752018-02-13 17:13:31 -0500116static bool check_no_conversion(uint32_t input, uint32_t output, float error) {
117 // This is a bit of a hack to check identity transformations that may lose precision.
118 return check_srgb_to_linear_to_srgb_conversion(input, output, error);
119}
120
bsalomon16921ec2015-07-30 15:34:56 -0700121typedef bool (*CheckFn) (uint32_t orig, uint32_t actual, float error);
122
Adlai Hollerc95b5892020-08-11 12:02:22 -0400123void read_and_check_pixels(skiatest::Reporter* reporter,
124 GrDirectContext* dContext,
125 GrSurfaceContext* sContext,
Robert Phillips301431d2017-03-29 12:08:49 -0400126 uint32_t* origData,
127 const SkImageInfo& dstInfo, CheckFn checker, float error,
bsalomon16921ec2015-07-30 15:34:56 -0700128 const char* subtestName) {
Brian Salomondd4087d2020-12-23 20:36:44 -0500129 auto [w, h] = dstInfo.dimensions();
Brian Salomonbe1084b2021-01-26 13:29:30 -0500130 GrPixmap readPM = GrPixmap::Allocate(dstInfo);
Brian Salomondd4087d2020-12-23 20:36:44 -0500131 memset(readPM.addr(), 0, sizeof(uint32_t)*w*h);
Robert Phillips301431d2017-03-29 12:08:49 -0400132
Brian Salomondd4087d2020-12-23 20:36:44 -0500133 if (!sContext->readPixels(dContext, readPM, {0, 0})) {
bsalomon16921ec2015-07-30 15:34:56 -0700134 ERRORF(reporter, "Could not read pixels for %s.", subtestName);
135 return;
136 }
Robert Phillips301431d2017-03-29 12:08:49 -0400137
bsalomon16921ec2015-07-30 15:34:56 -0700138 for (int j = 0; j < h; ++j) {
139 for (int i = 0; i < w; ++i) {
140 uint32_t orig = origData[j * w + i];
Brian Salomondd4087d2020-12-23 20:36:44 -0500141 uint32_t read = static_cast<uint32_t*>(readPM.addr())[j * w + i];
bsalomon16921ec2015-07-30 15:34:56 -0700142
143 if (!checker(orig, read, error)) {
Brian Salomon19eaf2d2018-03-19 16:06:44 -0400144 ERRORF(reporter, "Original 0x%08x, read back as 0x%08x in %s at %d, %d).", orig,
145 read, subtestName, i, j);
bsalomon16921ec2015-07-30 15:34:56 -0700146 return;
147 }
148 }
149 }
150}
151
Brian Salomonf802e752018-02-13 17:13:31 -0500152namespace {
153enum class Encoding {
154 kUntagged,
155 kLinear,
156 kSRGB,
157};
John Stilesa6841be2020-08-06 14:11:56 -0400158} // namespace
Brian Salomonf802e752018-02-13 17:13:31 -0500159
160static sk_sp<SkColorSpace> encoding_as_color_space(Encoding encoding) {
161 switch (encoding) {
162 case Encoding::kUntagged: return nullptr;
163 case Encoding::kLinear: return SkColorSpace::MakeSRGBLinear();
164 case Encoding::kSRGB: return SkColorSpace::MakeSRGB();
165 }
166 return nullptr;
167}
168
Brian Salomonf802e752018-02-13 17:13:31 -0500169static const char* encoding_as_str(Encoding encoding) {
170 switch (encoding) {
171 case Encoding::kUntagged: return "untagged";
172 case Encoding::kLinear: return "linear";
173 case Encoding::kSRGB: return "sRGB";
174 }
175 return nullptr;
176}
177
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500178static constexpr int kW = 255;
179static constexpr int kH = 255;
180
181static std::unique_ptr<uint32_t[]> make_data() {
182 std::unique_ptr<uint32_t[]> data(new uint32_t[kW * kH]);
bsalomon16921ec2015-07-30 15:34:56 -0700183 for (int j = 0; j < kH; ++j) {
184 for (int i = 0; i < kW; ++i) {
Brian Osmanf494bd22018-07-12 10:47:01 -0400185 data[j * kW + i] = (0xFF << 24) | (i << 16) | (i << 8) | i;
bsalomon16921ec2015-07-30 15:34:56 -0700186 }
187 }
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500188 return data;
189}
bsalomon16921ec2015-07-30 15:34:56 -0700190
Brian Salomonbf6b9792019-08-21 09:38:10 -0400191static std::unique_ptr<GrSurfaceContext> make_surface_context(Encoding contextEncoding,
Robert Phillips0c5bb2f2020-07-17 15:40:13 -0400192 GrRecordingContext* rContext,
Brian Salomonbf6b9792019-08-21 09:38:10 -0400193 skiatest::Reporter* reporter) {
Robert Phillips4dca8312021-07-28 15:13:20 -0400194 auto surfaceContext = skgpu::v1::SurfaceDrawContext::Make(
Robert Phillips0c5bb2f2020-07-17 15:40:13 -0400195 rContext, GrColorType::kRGBA_8888, encoding_as_color_space(contextEncoding),
Chris Daltonf5b87f92021-04-19 17:27:09 -0600196 SkBackingFit::kExact, {kW, kH}, SkSurfaceProps(), 1, GrMipmapped::kNo, GrProtected::kNo,
Greg Daniele20fcad2020-01-08 11:52:34 -0500197 kBottomLeft_GrSurfaceOrigin, SkBudgeted::kNo);
Brian Salomonf802e752018-02-13 17:13:31 -0500198 if (!surfaceContext) {
199 ERRORF(reporter, "Could not create %s surface context.", encoding_as_str(contextEncoding));
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500200 }
Mike Kleina9609ea2020-02-18 13:33:23 -0600201 return std::move(surfaceContext);
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500202}
203
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500204static void test_write_read(Encoding contextEncoding, Encoding writeEncoding, Encoding readEncoding,
Adlai Hollerc95b5892020-08-11 12:02:22 -0400205 float error, CheckFn check, GrDirectContext* dContext,
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500206 skiatest::Reporter* reporter) {
Adlai Hollerc95b5892020-08-11 12:02:22 -0400207 auto surfaceContext = make_surface_context(contextEncoding, dContext, reporter);
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500208 if (!surfaceContext) {
209 return;
210 }
211 auto writeII = SkImageInfo::Make(kW, kH, kRGBA_8888_SkColorType, kPremul_SkAlphaType,
212 encoding_as_color_space(writeEncoding));
213 auto data = make_data();
Brian Salomon5392c942021-03-30 16:14:37 -0400214 GrCPixmap dataPM(writeII, data.get(), kW*sizeof(uint32_t));
Brian Salomondd4087d2020-12-23 20:36:44 -0500215 if (!surfaceContext->writePixels(dContext, dataPM, {0, 0})) {
Brian Salomonf802e752018-02-13 17:13:31 -0500216 ERRORF(reporter, "Could not write %s to %s surface context.",
217 encoding_as_str(writeEncoding), encoding_as_str(contextEncoding));
218 return;
219 }
220
221 auto readII = SkImageInfo::Make(kW, kH, kRGBA_8888_SkColorType, kPremul_SkAlphaType,
222 encoding_as_color_space(readEncoding));
223 SkString testName;
224 testName.printf("write %s data to a %s context and read as %s.", encoding_as_str(writeEncoding),
225 encoding_as_str(contextEncoding), encoding_as_str(readEncoding));
Adlai Hollerc95b5892020-08-11 12:02:22 -0400226 read_and_check_pixels(reporter, dContext, surfaceContext.get(), data.get(), readII, check,
227 error, testName.c_str());
Brian Salomonf802e752018-02-13 17:13:31 -0500228}
229
230// Test all combinations of writePixels/readPixels where the surface context/write source/read dst
231// are sRGB, linear, or untagged RGBA_8888.
232DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SRGBReadWritePixels, reporter, ctxInfo) {
Robert Phillips6d344c32020-07-06 10:56:46 -0400233 auto context = ctxInfo.directContext();
Greg Daniel0258c902019-08-01 13:08:33 -0400234 if (!context->priv().caps()->getDefaultBackendFormat(GrColorType::kRGBA_8888_SRGB,
235 GrRenderable::kNo).isValid()) {
Brian Salomonf802e752018-02-13 17:13:31 -0500236 return;
237 }
238 // We allow more error on GPUs with lower precision shader variables.
Robert Phillips9da87e02019-02-04 13:26:26 -0500239 float error = context->priv().caps()->shaderCaps()->halfIs32Bits() ? 0.5f : 1.2f;
Brian Salomonf802e752018-02-13 17:13:31 -0500240 // For the all-sRGB case, we allow a small error only for devices that have
241 // precision variation because the sRGB data gets converted to linear and back in
242 // the shader.
Robert Phillips9da87e02019-02-04 13:26:26 -0500243 float smallError = context->priv().caps()->shaderCaps()->halfIs32Bits() ? 0.0f : 1.f;
Brian Salomonf802e752018-02-13 17:13:31 -0500244
245 ///////////////////////////////////////////////////////////////////////////////////////////////
246 // Write sRGB data to a sRGB context - no conversion on the write.
247
Brian Salomon19eaf2d2018-03-19 16:06:44 -0400248 // back to sRGB - no conversion.
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500249 test_write_read(Encoding::kSRGB, Encoding::kSRGB, Encoding::kSRGB, smallError,
250 check_no_conversion, context, reporter);
Brian Salomon19eaf2d2018-03-19 16:06:44 -0400251 // Reading back to untagged should be a pass through with no conversion.
252 test_write_read(Encoding::kSRGB, Encoding::kSRGB, Encoding::kUntagged, error,
253 check_no_conversion, context, reporter);
Brian Salomon19eaf2d2018-03-19 16:06:44 -0400254
Brian Salomonf802e752018-02-13 17:13:31 -0500255 // Converts back to linear
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500256 test_write_read(Encoding::kSRGB, Encoding::kSRGB, Encoding::kLinear, error,
257 check_srgb_to_linear_conversion, context, reporter);
Brian Salomonf802e752018-02-13 17:13:31 -0500258
Mike Klein12d4b6e2018-08-17 14:09:55 -0400259 // Untagged source data should be interpreted as sRGB.
260 test_write_read(Encoding::kSRGB, Encoding::kUntagged, Encoding::kSRGB, smallError,
261 check_no_conversion, context, reporter);
Brian Salomonf802e752018-02-13 17:13:31 -0500262
263 ///////////////////////////////////////////////////////////////////////////////////////////////
264 // Write linear data to a sRGB context. It gets converted to sRGB on write. The reads
265 // are all the same as the above cases where the original data was untagged.
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500266 test_write_read(Encoding::kSRGB, Encoding::kLinear, Encoding::kSRGB, error,
267 check_linear_to_srgb_conversion, context, reporter);
Brian Salomon19eaf2d2018-03-19 16:06:44 -0400268 // When the dst buffer is untagged there should be no conversion on the read.
269 test_write_read(Encoding::kSRGB, Encoding::kLinear, Encoding::kUntagged, error,
270 check_linear_to_srgb_conversion, context, reporter);
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500271 test_write_read(Encoding::kSRGB, Encoding::kLinear, Encoding::kLinear, error,
272 check_linear_to_srgb_to_linear_conversion, context, reporter);
Brian Salomonf802e752018-02-13 17:13:31 -0500273
274 ///////////////////////////////////////////////////////////////////////////////////////////////
275 // Write data to an untagged context. The write does no conversion no matter what encoding the
276 // src data has.
277 for (auto writeEncoding : {Encoding::kSRGB, Encoding::kUntagged, Encoding::kLinear}) {
Brian Salomon19eaf2d2018-03-19 16:06:44 -0400278 // The read from untagged to sRGB also does no conversion.
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500279 test_write_read(Encoding::kUntagged, writeEncoding, Encoding::kSRGB, error,
280 check_no_conversion, context, reporter);
Brian Salomonf802e752018-02-13 17:13:31 -0500281 // Reading untagged back as untagged should do no conversion.
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500282 test_write_read(Encoding::kUntagged, writeEncoding, Encoding::kUntagged, error,
283 check_no_conversion, context, reporter);
Brian Osman4f98dbe2019-03-28 11:49:01 -0400284 // Reading untagged back as linear does convert (context is source, so treated as sRGB),
285 // dst is tagged.
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500286 test_write_read(Encoding::kUntagged, writeEncoding, Encoding::kLinear, error,
Brian Osman4f98dbe2019-03-28 11:49:01 -0400287 check_srgb_to_linear_conversion, context, reporter);
Brian Salomonf802e752018-02-13 17:13:31 -0500288 }
289
290 ///////////////////////////////////////////////////////////////////////////////////////////////
291 // Write sRGB data to a linear context - converts to sRGB on the write.
292
293 // converts back to sRGB on read.
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500294 test_write_read(Encoding::kLinear, Encoding::kSRGB, Encoding::kSRGB, error,
295 check_srgb_to_linear_to_srgb_conversion, context, reporter);
Brian Salomon19eaf2d2018-03-19 16:06:44 -0400296 // Reading untagged data from linear currently does no conversion.
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500297 test_write_read(Encoding::kLinear, Encoding::kSRGB, Encoding::kUntagged, error,
298 check_srgb_to_linear_conversion, context, reporter);
Brian Salomonf802e752018-02-13 17:13:31 -0500299 // Stays linear when read.
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500300 test_write_read(Encoding::kLinear, Encoding::kSRGB, Encoding::kLinear, error,
301 check_srgb_to_linear_conversion, context, reporter);
Brian Salomonf802e752018-02-13 17:13:31 -0500302
Mike Klein12d4b6e2018-08-17 14:09:55 -0400303 // Untagged source data should be interpreted as sRGB.
304 test_write_read(Encoding::kLinear, Encoding::kUntagged, Encoding::kSRGB, error,
305 check_srgb_to_linear_to_srgb_conversion, context, reporter);
Brian Salomonf802e752018-02-13 17:13:31 -0500306
307 ///////////////////////////////////////////////////////////////////////////////////////////////
308 // Write linear data to a linear context. Does no conversion.
309
310 // Reading to sRGB does a conversion.
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500311 test_write_read(Encoding::kLinear, Encoding::kLinear, Encoding::kSRGB, error,
312 check_linear_to_srgb_conversion, context, reporter);
Brian Salomon19eaf2d2018-03-19 16:06:44 -0400313 // Reading to untagged does no conversion.
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500314 test_write_read(Encoding::kLinear, Encoding::kLinear, Encoding::kUntagged, error,
315 check_no_conversion, context, reporter);
Brian Salomonf802e752018-02-13 17:13:31 -0500316 // Stays linear when read.
Brian Salomon5f33a8c2018-02-26 14:32:39 -0500317 test_write_read(Encoding::kLinear, Encoding::kLinear, Encoding::kLinear, error,
318 check_no_conversion, context, reporter);
bsalomon16921ec2015-07-30 15:34:56 -0700319}