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

Unified Diff: tests/JpegTest.cpp

Issue 1549473003: Add getYUV8Planes() API to SkCodec (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/codec/SkJpegCodec.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/JpegTest.cpp
diff --git a/tests/JpegTest.cpp b/tests/JpegTest.cpp
index d61b0bd3111cd89f62930c3ed9cff2d81b3abea9..1e7499a513f4fab99bc8b9ce44a1d56b85680e77 100644
--- a/tests/JpegTest.cpp
+++ b/tests/JpegTest.cpp
@@ -6,10 +6,12 @@
*/
#include "SkBitmap.h"
+#include "SkCodec.h"
#include "SkDecodingImageGenerator.h"
#include "SkForceLinking.h"
#include "SkImageDecoder.h"
#include "SkPixelRef.h"
+#include "Resources.h"
#include "SkStream.h"
#include "SkTemplates.h"
#include "Test.h"
@@ -491,3 +493,112 @@ DEF_TEST(Jpeg_YUV, reporter) {
// Get the YUV planes
REPORTER_ASSERT(reporter, gen->getYUV8Planes(yuvSizes, planes, rowBytes, nullptr));
}
+
+static SkStreamAsset* resource(const char path[]) {
+ SkString fullPath = GetResourcePath(path);
+ return SkStream::NewFromFile(fullPath.c_str());
+}
+
+static void codec_yuv(skiatest::Reporter* reporter,
+ const char path[],
+ SkISize expectedSizes[3]) {
+ SkAutoTDelete<SkStream> stream(resource(path));
+ if (!stream) {
+ SkDebugf("Missing resource '%s'\n", path);
+ return;
+ }
+ SkAutoTDelete<SkCodec> codec(SkCodec::NewFromStream(stream.detach()));
+ REPORTER_ASSERT(reporter, codec);
+ if (!codec) {
+ return;
+ }
+
+ // Test queryYUV8()
+ SkCodec::YUVSizeInfo info;
+ bool success = codec->queryYUV8(nullptr, nullptr);
+ REPORTER_ASSERT(reporter, !success);
+ success = codec->queryYUV8(&info, nullptr);
+ REPORTER_ASSERT(reporter, (expectedSizes == nullptr) == !success);
+ if (!success) {
+ return;
+ }
+ REPORTER_ASSERT(reporter,
+ 0 == memcmp((const void*) &info, (const void*) expectedSizes, 3 * sizeof(SkISize)));
+ REPORTER_ASSERT(reporter, info.fYWidthBytes == (uint32_t) SkAlign8(info.fYSize.width()));
+ REPORTER_ASSERT(reporter, info.fUWidthBytes == (uint32_t) SkAlign8(info.fUSize.width()));
+ REPORTER_ASSERT(reporter, info.fVWidthBytes == (uint32_t) SkAlign8(info.fVSize.width()));
+ SkYUVColorSpace colorSpace;
+ success = codec->queryYUV8(&info, &colorSpace);
+ REPORTER_ASSERT(reporter,
+ 0 == memcmp((const void*) &info, (const void*) expectedSizes, 3 * sizeof(SkISize)));
+ REPORTER_ASSERT(reporter, info.fYWidthBytes == (uint32_t) SkAlign8(info.fYSize.width()));
+ REPORTER_ASSERT(reporter, info.fUWidthBytes == (uint32_t) SkAlign8(info.fUSize.width()));
+ REPORTER_ASSERT(reporter, info.fVWidthBytes == (uint32_t) SkAlign8(info.fVSize.width()));
+ REPORTER_ASSERT(reporter, kJPEG_SkYUVColorSpace == colorSpace);
+
+ // Allocate the memory for the YUV decode
+ size_t totalBytes = info.fYWidthBytes * info.fYSize.height() +
+ info.fUWidthBytes * info.fUSize.height() +
+ info.fVWidthBytes * info.fVSize.height();
+ SkAutoMalloc storage(totalBytes);
+ void* planes[3];
+ planes[0] = storage.get();
+ planes[1] = SkTAddOffset<void>(planes[0], info.fYWidthBytes * info.fYSize.height());
+ planes[2] = SkTAddOffset<void>(planes[1], info.fUWidthBytes * info.fUSize.height());
+
+ // Test getYUV8Planes()
+ REPORTER_ASSERT(reporter, SkCodec::kInvalidInput ==
+ codec->getYUV8Planes(info, nullptr));
+ REPORTER_ASSERT(reporter, SkCodec::kSuccess ==
+ codec->getYUV8Planes(info, planes));
+}
+
+DEF_TEST(Jpeg_YUV_Codec, r) {
+ SkISize sizes[3];
+
+ sizes[0].set(128, 128);
+ sizes[1].set(64, 64);
+ sizes[2].set(64, 64);
+ codec_yuv(r, "color_wheel.jpg", sizes);
+
+ // H2V2
+ sizes[0].set(512, 512);
+ sizes[1].set(256, 256);
+ sizes[2].set(256, 256);
+ codec_yuv(r, "mandrill_512_q075.jpg", sizes);
+
+ // H1V1
+ sizes[1].set(512, 512);
+ sizes[2].set(512, 512);
+ codec_yuv(r, "mandrill_h1v1.jpg", sizes);
+
+ // H2V1
+ sizes[1].set(256, 512);
+ sizes[2].set(256, 512);
+ codec_yuv(r, "mandrill_h2v1.jpg", sizes);
+
+ // Non-power of two dimensions
+ sizes[0].set(439, 154);
+ sizes[1].set(220, 77);
+ sizes[2].set(220, 77);
+ codec_yuv(r, "cropped_mandrill.jpg", sizes);
+
+ sizes[0].set(8, 8);
+ sizes[1].set(4, 4);
+ sizes[2].set(4, 4);
+ codec_yuv(r, "randPixels.jpg", sizes);
+
+ // Progressive images
+ sizes[0].set(512, 512);
+ sizes[1].set(512, 512);
+ sizes[2].set(512, 512);
+ codec_yuv(r, "brickwork-texture.jpg", sizes);
+ codec_yuv(r, "brickwork_normal-map.jpg", sizes);
+
+ // A CMYK encoded image should fail.
+ codec_yuv(r, "CMYK.jpg", nullptr);
+ // A grayscale encoded image should fail.
+ codec_yuv(r, "grayscale.jpg", nullptr);
+ // A PNG should fail.
+ codec_yuv(r, "arrow.png", nullptr);
+}
« no previous file with comments | « src/codec/SkJpegCodec.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698