2016-03-04 21:27:35 +00:00
|
|
|
/*
|
|
|
|
* Copyright 2016 Google Inc.
|
|
|
|
*
|
|
|
|
* Use of this source code is governed by a BSD-style license that can be
|
|
|
|
* found in the LICENSE file.
|
|
|
|
*/
|
|
|
|
|
|
|
|
#include "Resources.h"
|
|
|
|
#include "SkCodec.h"
|
|
|
|
#include "SkColorSpace.h"
|
2016-05-23 16:29:29 +00:00
|
|
|
#include "SkColorSpace_Base.h"
|
2016-03-04 21:27:35 +00:00
|
|
|
#include "Test.h"
|
|
|
|
|
2016-03-08 01:25:12 +00:00
|
|
|
#include "png.h"
|
|
|
|
|
2016-05-03 19:13:21 +00:00
|
|
|
static bool almost_equal(float a, float b) {
|
|
|
|
return SkTAbs(a - b) < 0.001f;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void test_space(skiatest::Reporter* r, SkColorSpace* space,
|
|
|
|
const float red[], const float green[], const float blue[],
|
2016-06-16 14:37:41 +00:00
|
|
|
const SkColorSpace::GammaNamed expectedGamma) {
|
2016-05-17 16:31:20 +00:00
|
|
|
|
2016-06-16 14:37:41 +00:00
|
|
|
REPORTER_ASSERT(r, expectedGamma == space->gammaNamed());
|
2016-05-17 16:31:20 +00:00
|
|
|
|
2016-05-03 19:13:21 +00:00
|
|
|
|
|
|
|
SkMatrix44 mat = space->xyz();
|
|
|
|
const float src[] = {
|
|
|
|
1, 0, 0, 1,
|
|
|
|
0, 1, 0, 1,
|
|
|
|
0, 0, 1, 1,
|
|
|
|
};
|
|
|
|
float dst[4];
|
|
|
|
for (int i = 0; i < 3; ++i) {
|
|
|
|
mat.mapScalars(&src[i*4], dst);
|
|
|
|
REPORTER_ASSERT(r, almost_equal(red[i], dst[0]));
|
|
|
|
REPORTER_ASSERT(r, almost_equal(green[i], dst[1]));
|
|
|
|
REPORTER_ASSERT(r, almost_equal(blue[i], dst[2]));
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-05-03 21:24:47 +00:00
|
|
|
const float g_sRGB_XYZ[] = { 0.4358f, 0.2224f, 0.0139f, // R
|
|
|
|
0.3853f, 0.7170f, 0.0971f, // G
|
|
|
|
0.1430f, 0.0606f, 0.7139f }; // B
|
|
|
|
|
2016-05-03 19:13:21 +00:00
|
|
|
DEF_TEST(ColorSpace_sRGB, r) {
|
|
|
|
test_space(r, SkColorSpace::NewNamed(SkColorSpace::kSRGB_Named).get(),
|
2016-06-16 14:37:41 +00:00
|
|
|
g_sRGB_XYZ, &g_sRGB_XYZ[3], &g_sRGB_XYZ[6], SkColorSpace::kSRGB_GammaNamed);
|
2016-05-03 19:13:21 +00:00
|
|
|
|
|
|
|
}
|
|
|
|
|
2016-03-04 21:27:35 +00:00
|
|
|
static SkStreamAsset* resource(const char path[]) {
|
|
|
|
SkString fullPath = GetResourcePath(path);
|
|
|
|
return SkStream::NewFromFile(fullPath.c_str());
|
|
|
|
}
|
|
|
|
|
2016-03-21 15:04:40 +00:00
|
|
|
DEF_TEST(ColorSpaceParsePngICCProfile, r) {
|
2016-03-04 21:27:35 +00:00
|
|
|
SkAutoTDelete<SkStream> stream(resource("color_wheel_with_profile.png"));
|
|
|
|
REPORTER_ASSERT(r, nullptr != stream);
|
2016-03-26 01:38:09 +00:00
|
|
|
if (!stream) {
|
|
|
|
return;
|
|
|
|
}
|
2016-03-04 21:27:35 +00:00
|
|
|
|
2016-03-16 20:53:35 +00:00
|
|
|
SkAutoTDelete<SkCodec> codec(SkCodec::NewFromStream(stream.release()));
|
Revert of Make SkPngCodec decode progressively. (patchset #26 id:520001 of https://codereview.chromium.org/1997703003/ )
Reason for revert:
Still causing problems in Google3, e.g.
https://test.corp.google.com/ui#cl=124138817&flags=CAMQBQ==&id=OCL:124138817:BASE:124139560:1465227435491:219ffbdb&t=//third_party/skia/HEAD:dm
Original issue's description:
> Make SkPngCodec decode progressively.
>
> This is a step towards using SkCodec in Chromium, where progressive
> decoding is necessary.
>
> Switch from using png_read_row (which expects all the data to be
> available) to png_process_data, which uses callbacks when rows are
> available.
>
> Create a new API for SkCodec, which supports progressive decoding and
> scanline decoding. Future changes will switch the other clients off of
> startScanlineDecode and get/skip-Scanlines to the new API.
>
> Remove SkCodec::kNone_ScanlineOrder, which was only used for interlaced
> PNG images. In the new API, interlaced PNG fits kTopDown. Also remove
> updateCurrScanline(), which was only used by the old implementation for
> interlaced PNG.
>
> DMSrcSink:
> - In CodecSrc::kScanline_Mode, use the new method for scanline decoding
> for the supported formats (just PNG and PNG-in-ICO for now).
>
> fuzz.cpp:
> - Remove reference to kNone_ScanlineOrder
>
> SkCodec:
> - Add new APIs:
> - startIncrementalDecode
> - incrementalDecode
> - Remove kNone_SkScanlineOrder and updateCurrScanline()
>
> SkPngCodec:
> - Implement new APIs
> - Switch from sk_read_fn/png_read_row etc to png_process_data
> - Expand AutoCleanPng's role to decode the header and create the
> SkPngCodec
> - Make the interlaced PNG decoder report how many lines were
> initialized during an incomplete decode
> - Make initializeSwizzler return a bool instead of an SkCodec::Result
> (It only returned kSuccess or kInvalidInput anyway)
>
> SkIcoCodec:
> - Implement the new APIs; supported for PNG in ICO
>
> SkSampledCodec:
> - Call the new method for decoding scanlines, and fall back to the old
> method if the new version is unimplemented
> - Remove references to kNone_SkScanlineOrder
>
> tests/CodecPartial:
> - Add a test which decodes part of an image, then finishes the decode,
> and compares it to the straightforward method
>
> tests/CodecTest:
> - Add a test which decodes all scanlines using the new method
> - Repurpose the Codec_stripes test to decode using the new method in
> sections rather than all at once
> - In the method check(), add a parameter for whether the image supports
> the new method of scanline decoding, and be explicit about whether an
> image supports incomplete
> - Test incomplete PNG decodes. We should have been doing it anyway for
> non-interlaced (except for an image that is too small - one row), but
> the new method supports interlaced incomplete as well
> - Make test_invalid_parameters test the new method
> - Add a test to ensure that it's safe to fall back to scanline decoding without
> rewinding
>
> BUG=skia:4211
>
> The new version was generally faster than the old version (but not significantly so).
>
> Some raw performance differences can be found at https://docs.google.com/a/google.com/spreadsheets/d/1Gis3aRCEa72qBNDRMgGDg3jD-pMgO-FXldlNF9ejo4o/
>
> Design doc can be found at https://docs.google.com/a/google.com/document/d/11Mn8-ePDKwVEMCjs3nWwSjxcSpJ_Cu8DF57KNtUmgLM/
>
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1997703003
>
> Committed: https://skia.googlesource.com/skia/+/a4b09a117d4d1ba5dda372e6a2323e653766539e
>
> Committed: https://skia.googlesource.com/skia/+/30e78c9737ff4861dc4e3fa1e4cd010680ed6965
>
> Committed: https://skia.googlesource.com/skia/+/6fb2391b2cc83ee2160b4e994faa8128975acc1f
TBR=reed@google.com,msarett@google.com,scroggo@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=skia:4211
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2044573002
Review-Url: https://codereview.chromium.org/2044573002
2016-06-06 18:26:17 +00:00
|
|
|
REPORTER_ASSERT(r, nullptr != codec);
|
2016-03-04 21:27:35 +00:00
|
|
|
|
2016-03-08 01:25:12 +00:00
|
|
|
#if (PNG_LIBPNG_VER_MAJOR > 1) || (PNG_LIBPNG_VER_MAJOR == 1 && PNG_LIBPNG_VER_MINOR >= 6)
|
2016-07-21 18:57:49 +00:00
|
|
|
SkColorSpace* colorSpace = codec->getInfo().colorSpace();
|
2016-03-04 21:27:35 +00:00
|
|
|
REPORTER_ASSERT(r, nullptr != colorSpace);
|
|
|
|
|
2016-06-16 14:37:41 +00:00
|
|
|
test_space(r, colorSpace, &g_sRGB_XYZ[0], &g_sRGB_XYZ[3], &g_sRGB_XYZ[6],
|
|
|
|
SkColorSpace::kSRGB_GammaNamed);
|
2016-03-08 01:25:12 +00:00
|
|
|
#endif
|
2016-03-04 21:27:35 +00:00
|
|
|
}
|
2016-03-21 15:04:40 +00:00
|
|
|
|
|
|
|
DEF_TEST(ColorSpaceParseJpegICCProfile, r) {
|
|
|
|
SkAutoTDelete<SkStream> stream(resource("icc-v2-gbr.jpg"));
|
|
|
|
REPORTER_ASSERT(r, nullptr != stream);
|
2016-03-25 21:02:20 +00:00
|
|
|
if (!stream) {
|
|
|
|
return;
|
|
|
|
}
|
2016-03-21 15:04:40 +00:00
|
|
|
|
|
|
|
SkAutoTDelete<SkCodec> codec(SkCodec::NewFromStream(stream.release()));
|
|
|
|
REPORTER_ASSERT(r, nullptr != codec);
|
2016-03-25 21:02:20 +00:00
|
|
|
if (!codec) {
|
|
|
|
return;
|
|
|
|
}
|
2016-03-21 15:04:40 +00:00
|
|
|
|
2016-07-21 18:57:49 +00:00
|
|
|
SkColorSpace* colorSpace = codec->getInfo().colorSpace();
|
2016-03-21 15:04:40 +00:00
|
|
|
REPORTER_ASSERT(r, nullptr != colorSpace);
|
|
|
|
|
2016-05-03 19:13:21 +00:00
|
|
|
const float red[] = { 0.385117f, 0.716904f, 0.0970612f };
|
|
|
|
const float green[] = { 0.143051f, 0.0606079f, 0.713913f };
|
|
|
|
const float blue[] = { 0.436035f, 0.222488f, 0.013916f };
|
2016-06-16 14:37:41 +00:00
|
|
|
test_space(r, colorSpace, red, green, blue, SkColorSpace::k2Dot2Curve_GammaNamed);
|
2016-03-21 15:04:40 +00:00
|
|
|
}
|
2016-05-03 21:24:47 +00:00
|
|
|
|
|
|
|
DEF_TEST(ColorSpaceSRGBCompare, r) {
|
|
|
|
// Create an sRGB color space by name
|
|
|
|
sk_sp<SkColorSpace> namedColorSpace = SkColorSpace::NewNamed(SkColorSpace::kSRGB_Named);
|
|
|
|
|
|
|
|
// Create an sRGB color space by value
|
|
|
|
SkMatrix44 srgbToxyzD50(SkMatrix44::kUninitialized_Constructor);
|
2016-06-23 19:42:29 +00:00
|
|
|
srgbToxyzD50.set3x3RowMajorf(g_sRGB_XYZ);
|
2016-06-16 14:37:41 +00:00
|
|
|
sk_sp<SkColorSpace> rgbColorSpace = SkColorSpace::NewRGB(SkColorSpace::kSRGB_GammaNamed,
|
|
|
|
srgbToxyzD50);
|
2016-05-25 15:53:40 +00:00
|
|
|
REPORTER_ASSERT(r, rgbColorSpace == namedColorSpace);
|
2016-05-03 21:24:47 +00:00
|
|
|
|
|
|
|
// Change a single value from the sRGB matrix
|
|
|
|
srgbToxyzD50.set(2, 2, 0.5f);
|
2016-06-16 14:37:41 +00:00
|
|
|
sk_sp<SkColorSpace> strangeColorSpace = SkColorSpace::NewRGB(SkColorSpace::kSRGB_GammaNamed,
|
|
|
|
srgbToxyzD50);
|
2016-05-03 21:24:47 +00:00
|
|
|
REPORTER_ASSERT(r, strangeColorSpace != namedColorSpace);
|
|
|
|
}
|
2016-05-25 15:53:40 +00:00
|
|
|
|
|
|
|
DEF_TEST(ColorSpaceWriteICC, r) {
|
|
|
|
// Test writing a new ICC profile
|
|
|
|
sk_sp<SkColorSpace> namedColorSpace = SkColorSpace::NewNamed(SkColorSpace::kSRGB_Named);
|
|
|
|
sk_sp<SkData> namedData = as_CSB(namedColorSpace)->writeToICC();
|
|
|
|
sk_sp<SkColorSpace> iccColorSpace = SkColorSpace::NewICC(namedData->data(), namedData->size());
|
2016-06-16 14:37:41 +00:00
|
|
|
test_space(r, iccColorSpace.get(), g_sRGB_XYZ, &g_sRGB_XYZ[3], &g_sRGB_XYZ[6],
|
|
|
|
SkColorSpace::k2Dot2Curve_GammaNamed);
|
|
|
|
// FIXME (msarett): Test disabled. sRGB profiles are written approximately as 2.2f curves.
|
|
|
|
// REPORTER_ASSERT(r, iccColorSpace == namedColorSpace);
|
2016-05-25 15:53:40 +00:00
|
|
|
|
|
|
|
// Test saving the original ICC data
|
|
|
|
sk_sp<SkData> monitorData = SkData::MakeFromFileName(
|
|
|
|
GetResourcePath("monitor_profiles/HP_ZR30w.icc").c_str());
|
|
|
|
REPORTER_ASSERT(r, monitorData);
|
|
|
|
if (!monitorData) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
sk_sp<SkColorSpace> monitorSpace = SkColorSpace::NewICC(monitorData->data(),
|
|
|
|
monitorData->size());
|
|
|
|
sk_sp<SkData> newMonitorData = as_CSB(monitorSpace)->writeToICC();
|
|
|
|
sk_sp<SkColorSpace> newMonitorSpace = SkColorSpace::NewICC(newMonitorData->data(),
|
|
|
|
newMonitorData->size());
|
|
|
|
REPORTER_ASSERT(r, monitorSpace->xyz() == newMonitorSpace->xyz());
|
2016-06-16 14:37:41 +00:00
|
|
|
REPORTER_ASSERT(r, monitorSpace->gammaNamed() == newMonitorSpace->gammaNamed());
|
2016-05-25 15:53:40 +00:00
|
|
|
}
|
2016-06-02 19:40:22 +00:00
|
|
|
|
|
|
|
DEF_TEST(ColorSpace_Named, r) {
|
|
|
|
const struct {
|
|
|
|
SkColorSpace::Named fNamed;
|
|
|
|
bool fExpectedToSucceed;
|
2016-06-16 14:37:41 +00:00
|
|
|
bool fIsSRGB;
|
2016-06-02 19:40:22 +00:00
|
|
|
} recs[] {
|
2016-06-16 14:37:41 +00:00
|
|
|
{ SkColorSpace::kUnknown_Named, false, false },
|
|
|
|
{ SkColorSpace::kSRGB_Named, true, true },
|
|
|
|
{ SkColorSpace::kAdobeRGB_Named, true, false },
|
2016-06-02 19:40:22 +00:00
|
|
|
};
|
|
|
|
|
|
|
|
for (auto rec : recs) {
|
|
|
|
auto cs = SkColorSpace::NewNamed(rec.fNamed);
|
|
|
|
REPORTER_ASSERT(r, !cs == !rec.fExpectedToSucceed);
|
|
|
|
if (cs) {
|
2016-06-16 14:37:41 +00:00
|
|
|
if (rec.fIsSRGB) {
|
|
|
|
REPORTER_ASSERT(r, SkColorSpace::kSRGB_GammaNamed == cs->gammaNamed());
|
|
|
|
} else {
|
|
|
|
REPORTER_ASSERT(r, SkColorSpace::k2Dot2Curve_GammaNamed == cs->gammaNamed());
|
|
|
|
}
|
2016-06-02 19:40:22 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
SkImageInfo info = SkImageInfo::MakeS32(10, 10, kPremul_SkAlphaType);
|
2016-06-21 17:28:14 +00:00
|
|
|
REPORTER_ASSERT(r, info.gammaCloseToSRGB());
|
2016-06-02 19:40:22 +00:00
|
|
|
}
|
2016-06-22 15:18:54 +00:00
|
|
|
|
|
|
|
static void test_serialize(skiatest::Reporter* r, SkColorSpace* space, bool isNamed) {
|
|
|
|
sk_sp<SkData> data = space->serialize();
|
|
|
|
sk_sp<SkColorSpace> newSpace = SkColorSpace::Deserialize(data->data(), data->size());
|
|
|
|
|
|
|
|
if (isNamed) {
|
|
|
|
REPORTER_ASSERT(r, space == newSpace.get());
|
|
|
|
} else {
|
|
|
|
REPORTER_ASSERT(r, space->gammaNamed() == newSpace->gammaNamed());
|
|
|
|
|
|
|
|
REPORTER_ASSERT(r, space->xyz().getFloat(0, 0) == newSpace->xyz().getFloat(0, 0));
|
|
|
|
REPORTER_ASSERT(r, space->xyz().getFloat(0, 1) == newSpace->xyz().getFloat(0, 1));
|
|
|
|
REPORTER_ASSERT(r, space->xyz().getFloat(0, 2) == newSpace->xyz().getFloat(0, 2));
|
|
|
|
REPORTER_ASSERT(r, space->xyz().getFloat(0, 3) == newSpace->xyz().getFloat(0, 3));
|
|
|
|
REPORTER_ASSERT(r, space->xyz().getFloat(1, 0) == newSpace->xyz().getFloat(1, 0));
|
|
|
|
REPORTER_ASSERT(r, space->xyz().getFloat(1, 1) == newSpace->xyz().getFloat(1, 1));
|
|
|
|
REPORTER_ASSERT(r, space->xyz().getFloat(1, 2) == newSpace->xyz().getFloat(1, 2));
|
|
|
|
REPORTER_ASSERT(r, space->xyz().getFloat(1, 3) == newSpace->xyz().getFloat(1, 3));
|
|
|
|
REPORTER_ASSERT(r, space->xyz().getFloat(2, 0) == newSpace->xyz().getFloat(2, 0));
|
|
|
|
REPORTER_ASSERT(r, space->xyz().getFloat(2, 1) == newSpace->xyz().getFloat(2, 1));
|
|
|
|
REPORTER_ASSERT(r, space->xyz().getFloat(2, 2) == newSpace->xyz().getFloat(2, 2));
|
|
|
|
REPORTER_ASSERT(r, space->xyz().getFloat(2, 3) == newSpace->xyz().getFloat(2, 3));
|
|
|
|
REPORTER_ASSERT(r, space->xyz().getFloat(3, 0) == newSpace->xyz().getFloat(3, 0));
|
|
|
|
REPORTER_ASSERT(r, space->xyz().getFloat(3, 1) == newSpace->xyz().getFloat(3, 1));
|
|
|
|
REPORTER_ASSERT(r, space->xyz().getFloat(3, 2) == newSpace->xyz().getFloat(3, 2));
|
|
|
|
REPORTER_ASSERT(r, space->xyz().getFloat(3, 3) == newSpace->xyz().getFloat(3, 3));
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
DEF_TEST(ColorSpace_Serialize, r) {
|
|
|
|
test_serialize(r, SkColorSpace::NewNamed(SkColorSpace::kSRGB_Named).get(), true);
|
|
|
|
test_serialize(r, SkColorSpace::NewNamed(SkColorSpace::kAdobeRGB_Named).get(), true);
|
|
|
|
|
|
|
|
sk_sp<SkData> monitorData = SkData::MakeFromFileName(
|
|
|
|
GetResourcePath("monitor_profiles/HP_ZR30w.icc").c_str());
|
|
|
|
test_serialize(r, SkColorSpace::NewICC(monitorData->data(), monitorData->size()).get(), false);
|
|
|
|
}
|
|
|
|
|