2015-07-30 22:34:56 +00:00
|
|
|
/*
|
|
|
|
* Copyright 2015 Google Inc.
|
|
|
|
*
|
|
|
|
* Use of this source code is governed by a BSD-style license that can be
|
|
|
|
* found in the LICENSE file.
|
|
|
|
*/
|
|
|
|
|
2019-04-23 17:05:21 +00:00
|
|
|
#include "include/core/SkCanvas.h"
|
|
|
|
#include "include/core/SkSurface.h"
|
2020-07-06 14:56:46 +00:00
|
|
|
#include "include/gpu/GrDirectContext.h"
|
2019-04-23 17:05:21 +00:00
|
|
|
#include "src/gpu/GrCaps.h"
|
2020-10-14 15:23:11 +00:00
|
|
|
#include "src/gpu/GrDirectContextPriv.h"
|
2019-09-30 16:15:30 +00:00
|
|
|
#include "src/gpu/GrImageInfo.h"
|
2019-04-23 17:05:21 +00:00
|
|
|
#include "src/gpu/SkGr.h"
|
2021-08-10 17:49:51 +00:00
|
|
|
#include "src/gpu/SurfaceContext.h"
|
2019-04-23 17:05:21 +00:00
|
|
|
#include "tests/Test.h"
|
2021-08-16 15:41:27 +00:00
|
|
|
#include "tests/TestUtils.h"
|
2015-07-30 22:34:56 +00:00
|
|
|
|
|
|
|
// using anonymous namespace because these functions are used as template params.
|
|
|
|
namespace {
|
|
|
|
/** convert 0..1 srgb value to 0..1 linear */
|
|
|
|
float srgb_to_linear(float srgb) {
|
|
|
|
if (srgb <= 0.04045f) {
|
|
|
|
return srgb / 12.92f;
|
|
|
|
} else {
|
|
|
|
return powf((srgb + 0.055f) / 1.055f, 2.4f);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/** convert 0..1 linear value to 0..1 srgb */
|
|
|
|
float linear_to_srgb(float linear) {
|
|
|
|
if (linear <= 0.0031308) {
|
|
|
|
return linear * 12.92f;
|
|
|
|
} else {
|
|
|
|
return 1.055f * powf(linear, 1.f / 2.4f) - 0.055f;
|
|
|
|
}
|
|
|
|
}
|
2020-08-06 18:11:56 +00:00
|
|
|
} // namespace
|
2015-07-30 22:34:56 +00:00
|
|
|
|
|
|
|
/** tests a conversion with an error tolerance */
|
|
|
|
template <float (*CONVERT)(float)> static bool check_conversion(uint32_t input, uint32_t output,
|
|
|
|
float error) {
|
|
|
|
// alpha should always be exactly preserved.
|
|
|
|
if ((input & 0xff000000) != (output & 0xff000000)) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
for (int c = 0; c < 3; ++c) {
|
|
|
|
uint8_t inputComponent = (uint8_t) ((input & (0xff << (c*8))) >> (c*8));
|
2020-02-07 15:36:46 +00:00
|
|
|
float lower = std::max(0.f, (float) inputComponent - error);
|
|
|
|
float upper = std::min(255.f, (float) inputComponent + error);
|
2015-07-30 22:34:56 +00:00
|
|
|
lower = CONVERT(lower / 255.f);
|
|
|
|
upper = CONVERT(upper / 255.f);
|
|
|
|
SkASSERT(lower >= 0.f && lower <= 255.f);
|
|
|
|
SkASSERT(upper >= 0.f && upper <= 255.f);
|
|
|
|
uint8_t outputComponent = (output & (0xff << (c*8))) >> (c*8);
|
|
|
|
if (outputComponent < SkScalarFloorToInt(lower * 255.f) ||
|
|
|
|
outputComponent > SkScalarCeilToInt(upper * 255.f)) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
/** tests a forward and backward conversion with an error tolerance */
|
|
|
|
template <float (*FORWARD)(float), float (*BACKWARD)(float)>
|
|
|
|
static bool check_double_conversion(uint32_t input, uint32_t output, float error) {
|
|
|
|
// alpha should always be exactly preserved.
|
|
|
|
if ((input & 0xff000000) != (output & 0xff000000)) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
for (int c = 0; c < 3; ++c) {
|
|
|
|
uint8_t inputComponent = (uint8_t) ((input & (0xff << (c*8))) >> (c*8));
|
2020-02-07 15:36:46 +00:00
|
|
|
float lower = std::max(0.f, (float) inputComponent - error);
|
|
|
|
float upper = std::min(255.f, (float) inputComponent + error);
|
2015-07-30 22:34:56 +00:00
|
|
|
lower = FORWARD(lower / 255.f);
|
|
|
|
upper = FORWARD(upper / 255.f);
|
|
|
|
SkASSERT(lower >= 0.f && lower <= 255.f);
|
|
|
|
SkASSERT(upper >= 0.f && upper <= 255.f);
|
|
|
|
uint8_t upperComponent = SkScalarCeilToInt(upper * 255.f);
|
|
|
|
uint8_t lowerComponent = SkScalarFloorToInt(lower * 255.f);
|
2020-02-07 15:36:46 +00:00
|
|
|
lower = std::max(0.f, (float) lowerComponent - error);
|
|
|
|
upper = std::min(255.f, (float) upperComponent + error);
|
2015-07-30 22:34:56 +00:00
|
|
|
lower = BACKWARD(lowerComponent / 255.f);
|
|
|
|
upper = BACKWARD(upperComponent / 255.f);
|
|
|
|
SkASSERT(lower >= 0.f && lower <= 255.f);
|
|
|
|
SkASSERT(upper >= 0.f && upper <= 255.f);
|
|
|
|
upperComponent = SkScalarCeilToInt(upper * 255.f);
|
|
|
|
lowerComponent = SkScalarFloorToInt(lower * 255.f);
|
|
|
|
|
|
|
|
uint8_t outputComponent = (output & (0xff << (c*8))) >> (c*8);
|
|
|
|
if (outputComponent < lowerComponent || outputComponent > upperComponent) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
static bool check_srgb_to_linear_conversion(uint32_t srgb, uint32_t linear, float error) {
|
|
|
|
return check_conversion<srgb_to_linear>(srgb, linear, error);
|
|
|
|
}
|
|
|
|
|
|
|
|
static bool check_linear_to_srgb_conversion(uint32_t linear, uint32_t srgb, float error) {
|
|
|
|
return check_conversion<linear_to_srgb>(linear, srgb, error);
|
|
|
|
}
|
|
|
|
|
|
|
|
static bool check_linear_to_srgb_to_linear_conversion(uint32_t input, uint32_t output, float error) {
|
|
|
|
return check_double_conversion<linear_to_srgb, srgb_to_linear>(input, output, error);
|
|
|
|
}
|
|
|
|
|
|
|
|
static bool check_srgb_to_linear_to_srgb_conversion(uint32_t input, uint32_t output, float error) {
|
|
|
|
return check_double_conversion<srgb_to_linear, linear_to_srgb>(input, output, error);
|
|
|
|
}
|
|
|
|
|
2018-02-13 22:13:31 +00:00
|
|
|
static bool check_no_conversion(uint32_t input, uint32_t output, float error) {
|
|
|
|
// This is a bit of a hack to check identity transformations that may lose precision.
|
|
|
|
return check_srgb_to_linear_to_srgb_conversion(input, output, error);
|
|
|
|
}
|
|
|
|
|
2015-07-30 22:34:56 +00:00
|
|
|
typedef bool (*CheckFn) (uint32_t orig, uint32_t actual, float error);
|
|
|
|
|
2020-08-11 16:02:22 +00:00
|
|
|
void read_and_check_pixels(skiatest::Reporter* reporter,
|
|
|
|
GrDirectContext* dContext,
|
2021-08-11 13:31:00 +00:00
|
|
|
skgpu::SurfaceContext* sc,
|
2017-03-29 16:08:49 +00:00
|
|
|
uint32_t* origData,
|
|
|
|
const SkImageInfo& dstInfo, CheckFn checker, float error,
|
2015-07-30 22:34:56 +00:00
|
|
|
const char* subtestName) {
|
2020-12-24 01:36:44 +00:00
|
|
|
auto [w, h] = dstInfo.dimensions();
|
2021-01-26 18:29:30 +00:00
|
|
|
GrPixmap readPM = GrPixmap::Allocate(dstInfo);
|
2020-12-24 01:36:44 +00:00
|
|
|
memset(readPM.addr(), 0, sizeof(uint32_t)*w*h);
|
2017-03-29 16:08:49 +00:00
|
|
|
|
2021-08-11 13:31:00 +00:00
|
|
|
if (!sc->readPixels(dContext, readPM, {0, 0})) {
|
2015-07-30 22:34:56 +00:00
|
|
|
ERRORF(reporter, "Could not read pixels for %s.", subtestName);
|
|
|
|
return;
|
|
|
|
}
|
2017-03-29 16:08:49 +00:00
|
|
|
|
2015-07-30 22:34:56 +00:00
|
|
|
for (int j = 0; j < h; ++j) {
|
|
|
|
for (int i = 0; i < w; ++i) {
|
|
|
|
uint32_t orig = origData[j * w + i];
|
2020-12-24 01:36:44 +00:00
|
|
|
uint32_t read = static_cast<uint32_t*>(readPM.addr())[j * w + i];
|
2015-07-30 22:34:56 +00:00
|
|
|
|
|
|
|
if (!checker(orig, read, error)) {
|
2018-03-19 20:06:44 +00:00
|
|
|
ERRORF(reporter, "Original 0x%08x, read back as 0x%08x in %s at %d, %d).", orig,
|
|
|
|
read, subtestName, i, j);
|
2015-07-30 22:34:56 +00:00
|
|
|
return;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2018-02-13 22:13:31 +00:00
|
|
|
namespace {
|
|
|
|
enum class Encoding {
|
|
|
|
kUntagged,
|
|
|
|
kLinear,
|
|
|
|
kSRGB,
|
|
|
|
};
|
2020-08-06 18:11:56 +00:00
|
|
|
} // namespace
|
2018-02-13 22:13:31 +00:00
|
|
|
|
|
|
|
static sk_sp<SkColorSpace> encoding_as_color_space(Encoding encoding) {
|
|
|
|
switch (encoding) {
|
|
|
|
case Encoding::kUntagged: return nullptr;
|
|
|
|
case Encoding::kLinear: return SkColorSpace::MakeSRGBLinear();
|
|
|
|
case Encoding::kSRGB: return SkColorSpace::MakeSRGB();
|
|
|
|
}
|
|
|
|
return nullptr;
|
|
|
|
}
|
|
|
|
|
|
|
|
static const char* encoding_as_str(Encoding encoding) {
|
|
|
|
switch (encoding) {
|
|
|
|
case Encoding::kUntagged: return "untagged";
|
|
|
|
case Encoding::kLinear: return "linear";
|
|
|
|
case Encoding::kSRGB: return "sRGB";
|
|
|
|
}
|
|
|
|
return nullptr;
|
|
|
|
}
|
|
|
|
|
2018-02-26 19:32:39 +00:00
|
|
|
static constexpr int kW = 255;
|
|
|
|
static constexpr int kH = 255;
|
|
|
|
|
|
|
|
static std::unique_ptr<uint32_t[]> make_data() {
|
|
|
|
std::unique_ptr<uint32_t[]> data(new uint32_t[kW * kH]);
|
2015-07-30 22:34:56 +00:00
|
|
|
for (int j = 0; j < kH; ++j) {
|
|
|
|
for (int i = 0; i < kW; ++i) {
|
2018-07-12 14:47:01 +00:00
|
|
|
data[j * kW + i] = (0xFF << 24) | (i << 16) | (i << 8) | i;
|
2015-07-30 22:34:56 +00:00
|
|
|
}
|
|
|
|
}
|
2018-02-26 19:32:39 +00:00
|
|
|
return data;
|
|
|
|
}
|
2015-07-30 22:34:56 +00:00
|
|
|
|
2021-08-10 17:49:51 +00:00
|
|
|
static std::unique_ptr<skgpu::SurfaceContext> make_surface_context(Encoding contextEncoding,
|
|
|
|
GrRecordingContext* rContext,
|
|
|
|
skiatest::Reporter* reporter) {
|
2021-07-29 15:23:48 +00:00
|
|
|
GrImageInfo info(GrColorType::kRGBA_8888,
|
|
|
|
kPremul_SkAlphaType,
|
|
|
|
encoding_as_color_space(contextEncoding),
|
|
|
|
kW, kH);
|
|
|
|
|
2021-08-16 15:41:27 +00:00
|
|
|
auto sc = CreateSurfaceContext(rContext,
|
|
|
|
info,
|
|
|
|
SkBackingFit::kExact,
|
|
|
|
kBottomLeft_GrSurfaceOrigin,
|
|
|
|
GrRenderable::kYes);
|
2021-07-29 15:23:48 +00:00
|
|
|
if (!sc) {
|
2018-02-13 22:13:31 +00:00
|
|
|
ERRORF(reporter, "Could not create %s surface context.", encoding_as_str(contextEncoding));
|
2018-02-26 19:32:39 +00:00
|
|
|
}
|
2021-07-29 15:23:48 +00:00
|
|
|
return sc;
|
2018-02-26 19:32:39 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
static void test_write_read(Encoding contextEncoding, Encoding writeEncoding, Encoding readEncoding,
|
2020-08-11 16:02:22 +00:00
|
|
|
float error, CheckFn check, GrDirectContext* dContext,
|
2018-02-26 19:32:39 +00:00
|
|
|
skiatest::Reporter* reporter) {
|
2020-08-11 16:02:22 +00:00
|
|
|
auto surfaceContext = make_surface_context(contextEncoding, dContext, reporter);
|
2018-02-26 19:32:39 +00:00
|
|
|
if (!surfaceContext) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
auto writeII = SkImageInfo::Make(kW, kH, kRGBA_8888_SkColorType, kPremul_SkAlphaType,
|
|
|
|
encoding_as_color_space(writeEncoding));
|
|
|
|
auto data = make_data();
|
2021-03-30 20:14:37 +00:00
|
|
|
GrCPixmap dataPM(writeII, data.get(), kW*sizeof(uint32_t));
|
2020-12-24 01:36:44 +00:00
|
|
|
if (!surfaceContext->writePixels(dContext, dataPM, {0, 0})) {
|
2018-02-13 22:13:31 +00:00
|
|
|
ERRORF(reporter, "Could not write %s to %s surface context.",
|
|
|
|
encoding_as_str(writeEncoding), encoding_as_str(contextEncoding));
|
|
|
|
return;
|
|
|
|
}
|
2018-02-13 20:51:26 +00:00
|
|
|
|
2018-02-13 22:13:31 +00:00
|
|
|
auto readII = SkImageInfo::Make(kW, kH, kRGBA_8888_SkColorType, kPremul_SkAlphaType,
|
|
|
|
encoding_as_color_space(readEncoding));
|
|
|
|
SkString testName;
|
|
|
|
testName.printf("write %s data to a %s context and read as %s.", encoding_as_str(writeEncoding),
|
|
|
|
encoding_as_str(contextEncoding), encoding_as_str(readEncoding));
|
2020-08-11 16:02:22 +00:00
|
|
|
read_and_check_pixels(reporter, dContext, surfaceContext.get(), data.get(), readII, check,
|
|
|
|
error, testName.c_str());
|
2018-02-13 22:13:31 +00:00
|
|
|
}
|
2018-02-13 20:51:26 +00:00
|
|
|
|
2018-02-13 22:13:31 +00:00
|
|
|
// Test all combinations of writePixels/readPixels where the surface context/write source/read dst
|
|
|
|
// are sRGB, linear, or untagged RGBA_8888.
|
|
|
|
DEF_GPUTEST_FOR_RENDERING_CONTEXTS(SRGBReadWritePixels, reporter, ctxInfo) {
|
2020-07-06 14:56:46 +00:00
|
|
|
auto context = ctxInfo.directContext();
|
2019-08-01 17:08:33 +00:00
|
|
|
if (!context->priv().caps()->getDefaultBackendFormat(GrColorType::kRGBA_8888_SRGB,
|
|
|
|
GrRenderable::kNo).isValid()) {
|
2018-02-13 22:13:31 +00:00
|
|
|
return;
|
|
|
|
}
|
|
|
|
// We allow more error on GPUs with lower precision shader variables.
|
2019-02-04 18:26:26 +00:00
|
|
|
float error = context->priv().caps()->shaderCaps()->halfIs32Bits() ? 0.5f : 1.2f;
|
2018-02-13 22:13:31 +00:00
|
|
|
// For the all-sRGB case, we allow a small error only for devices that have
|
|
|
|
// precision variation because the sRGB data gets converted to linear and back in
|
|
|
|
// the shader.
|
2019-02-04 18:26:26 +00:00
|
|
|
float smallError = context->priv().caps()->shaderCaps()->halfIs32Bits() ? 0.0f : 1.f;
|
2018-02-13 20:51:26 +00:00
|
|
|
|
2018-02-13 22:13:31 +00:00
|
|
|
///////////////////////////////////////////////////////////////////////////////////////////////
|
|
|
|
// Write sRGB data to a sRGB context - no conversion on the write.
|
|
|
|
|
2018-03-19 20:06:44 +00:00
|
|
|
// back to sRGB - no conversion.
|
2018-02-26 19:32:39 +00:00
|
|
|
test_write_read(Encoding::kSRGB, Encoding::kSRGB, Encoding::kSRGB, smallError,
|
|
|
|
check_no_conversion, context, reporter);
|
2018-03-19 20:06:44 +00:00
|
|
|
// Reading back to untagged should be a pass through with no conversion.
|
|
|
|
test_write_read(Encoding::kSRGB, Encoding::kSRGB, Encoding::kUntagged, error,
|
|
|
|
check_no_conversion, context, reporter);
|
|
|
|
|
2018-02-13 22:13:31 +00:00
|
|
|
// Converts back to linear
|
2018-02-26 19:32:39 +00:00
|
|
|
test_write_read(Encoding::kSRGB, Encoding::kSRGB, Encoding::kLinear, error,
|
|
|
|
check_srgb_to_linear_conversion, context, reporter);
|
2018-02-13 22:13:31 +00:00
|
|
|
|
let's like, chill out about all these rules, man
There's really no reason to prevent any of these conversions;
they all have somewhat reasonable behavior:
- converting between grey in different color spaces
should probably work just fine
- we'll convert color to gray using a fixed set of
luminance coefficients, but that's better than failing
- we'll invent {r,g,b} = {0,0,0} if we convert alpha
to something with color
- converting to opaque formats without thinking about
premul/unpremul is probably fine, better than just
not working at all
- a missing src color space can always be assumed to be sRGB
Updates to ReadPixelsTest:
- skip more supported test cases in test_conversion(),
each with a TODO
- conversions from non-opaque to opaque should now work
- conversion from A8 to non-A8 should sometimes now
work on GPUs, and the test needed a little bit of
a tweak to not expect A8 to carry around color somehow.
Updates to SRGBReadWritePixelsTest:
- writing untagged pixels shouldn't fail anymore;
instead, it should behave like it was tagged sRGB
Change-Id: I19e78f3a6c89ef74fbcbc985d3fbd77fa984b1c2
Reviewed-on: https://skia-review.googlesource.com/147815
Commit-Queue: Mike Klein <mtklein@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
2018-08-17 18:09:55 +00:00
|
|
|
// Untagged source data should be interpreted as sRGB.
|
|
|
|
test_write_read(Encoding::kSRGB, Encoding::kUntagged, Encoding::kSRGB, smallError,
|
|
|
|
check_no_conversion, context, reporter);
|
2018-02-13 22:13:31 +00:00
|
|
|
|
|
|
|
///////////////////////////////////////////////////////////////////////////////////////////////
|
|
|
|
// Write linear data to a sRGB context. It gets converted to sRGB on write. The reads
|
|
|
|
// are all the same as the above cases where the original data was untagged.
|
2018-02-26 19:32:39 +00:00
|
|
|
test_write_read(Encoding::kSRGB, Encoding::kLinear, Encoding::kSRGB, error,
|
|
|
|
check_linear_to_srgb_conversion, context, reporter);
|
2018-03-19 20:06:44 +00:00
|
|
|
// When the dst buffer is untagged there should be no conversion on the read.
|
|
|
|
test_write_read(Encoding::kSRGB, Encoding::kLinear, Encoding::kUntagged, error,
|
|
|
|
check_linear_to_srgb_conversion, context, reporter);
|
2018-02-26 19:32:39 +00:00
|
|
|
test_write_read(Encoding::kSRGB, Encoding::kLinear, Encoding::kLinear, error,
|
|
|
|
check_linear_to_srgb_to_linear_conversion, context, reporter);
|
2018-02-13 22:13:31 +00:00
|
|
|
|
|
|
|
///////////////////////////////////////////////////////////////////////////////////////////////
|
|
|
|
// Write data to an untagged context. The write does no conversion no matter what encoding the
|
|
|
|
// src data has.
|
|
|
|
for (auto writeEncoding : {Encoding::kSRGB, Encoding::kUntagged, Encoding::kLinear}) {
|
2018-03-19 20:06:44 +00:00
|
|
|
// The read from untagged to sRGB also does no conversion.
|
2018-02-26 19:32:39 +00:00
|
|
|
test_write_read(Encoding::kUntagged, writeEncoding, Encoding::kSRGB, error,
|
|
|
|
check_no_conversion, context, reporter);
|
2018-02-13 22:13:31 +00:00
|
|
|
// Reading untagged back as untagged should do no conversion.
|
2018-02-26 19:32:39 +00:00
|
|
|
test_write_read(Encoding::kUntagged, writeEncoding, Encoding::kUntagged, error,
|
|
|
|
check_no_conversion, context, reporter);
|
2019-03-28 15:49:01 +00:00
|
|
|
// Reading untagged back as linear does convert (context is source, so treated as sRGB),
|
|
|
|
// dst is tagged.
|
2018-02-26 19:32:39 +00:00
|
|
|
test_write_read(Encoding::kUntagged, writeEncoding, Encoding::kLinear, error,
|
2019-03-28 15:49:01 +00:00
|
|
|
check_srgb_to_linear_conversion, context, reporter);
|
2015-07-30 22:34:56 +00:00
|
|
|
}
|
2018-02-13 22:13:31 +00:00
|
|
|
|
|
|
|
///////////////////////////////////////////////////////////////////////////////////////////////
|
|
|
|
// Write sRGB data to a linear context - converts to sRGB on the write.
|
|
|
|
|
|
|
|
// converts back to sRGB on read.
|
2018-02-26 19:32:39 +00:00
|
|
|
test_write_read(Encoding::kLinear, Encoding::kSRGB, Encoding::kSRGB, error,
|
|
|
|
check_srgb_to_linear_to_srgb_conversion, context, reporter);
|
2018-03-19 20:06:44 +00:00
|
|
|
// Reading untagged data from linear currently does no conversion.
|
2018-02-26 19:32:39 +00:00
|
|
|
test_write_read(Encoding::kLinear, Encoding::kSRGB, Encoding::kUntagged, error,
|
|
|
|
check_srgb_to_linear_conversion, context, reporter);
|
2018-02-13 22:13:31 +00:00
|
|
|
// Stays linear when read.
|
2018-02-26 19:32:39 +00:00
|
|
|
test_write_read(Encoding::kLinear, Encoding::kSRGB, Encoding::kLinear, error,
|
|
|
|
check_srgb_to_linear_conversion, context, reporter);
|
2018-02-13 22:13:31 +00:00
|
|
|
|
let's like, chill out about all these rules, man
There's really no reason to prevent any of these conversions;
they all have somewhat reasonable behavior:
- converting between grey in different color spaces
should probably work just fine
- we'll convert color to gray using a fixed set of
luminance coefficients, but that's better than failing
- we'll invent {r,g,b} = {0,0,0} if we convert alpha
to something with color
- converting to opaque formats without thinking about
premul/unpremul is probably fine, better than just
not working at all
- a missing src color space can always be assumed to be sRGB
Updates to ReadPixelsTest:
- skip more supported test cases in test_conversion(),
each with a TODO
- conversions from non-opaque to opaque should now work
- conversion from A8 to non-A8 should sometimes now
work on GPUs, and the test needed a little bit of
a tweak to not expect A8 to carry around color somehow.
Updates to SRGBReadWritePixelsTest:
- writing untagged pixels shouldn't fail anymore;
instead, it should behave like it was tagged sRGB
Change-Id: I19e78f3a6c89ef74fbcbc985d3fbd77fa984b1c2
Reviewed-on: https://skia-review.googlesource.com/147815
Commit-Queue: Mike Klein <mtklein@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
2018-08-17 18:09:55 +00:00
|
|
|
// Untagged source data should be interpreted as sRGB.
|
|
|
|
test_write_read(Encoding::kLinear, Encoding::kUntagged, Encoding::kSRGB, error,
|
|
|
|
check_srgb_to_linear_to_srgb_conversion, context, reporter);
|
2018-02-13 22:13:31 +00:00
|
|
|
|
|
|
|
///////////////////////////////////////////////////////////////////////////////////////////////
|
|
|
|
// Write linear data to a linear context. Does no conversion.
|
|
|
|
|
|
|
|
// Reading to sRGB does a conversion.
|
2018-02-26 19:32:39 +00:00
|
|
|
test_write_read(Encoding::kLinear, Encoding::kLinear, Encoding::kSRGB, error,
|
|
|
|
check_linear_to_srgb_conversion, context, reporter);
|
2018-03-19 20:06:44 +00:00
|
|
|
// Reading to untagged does no conversion.
|
2018-02-26 19:32:39 +00:00
|
|
|
test_write_read(Encoding::kLinear, Encoding::kLinear, Encoding::kUntagged, error,
|
|
|
|
check_no_conversion, context, reporter);
|
2018-02-13 22:13:31 +00:00
|
|
|
// Stays linear when read.
|
2018-02-26 19:32:39 +00:00
|
|
|
test_write_read(Encoding::kLinear, Encoding::kLinear, Encoding::kLinear, error,
|
|
|
|
check_no_conversion, context, reporter);
|
2015-07-30 22:34:56 +00:00
|
|
|
}
|