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

Unified Diff: tests/SkBlend_optsTest.cpp

Issue 1969803002: Revert "Add tests and benches to support the sRGB blitter for SkOpts" (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 4 years, 7 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/opts/SkOpts_sse41.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/SkBlend_optsTest.cpp
diff --git a/tests/SkBlend_optsTest.cpp b/tests/SkBlend_optsTest.cpp
deleted file mode 100644
index 10359002f4f0c77f385983ab369bb11750a78bd5..0000000000000000000000000000000000000000
--- a/tests/SkBlend_optsTest.cpp
+++ /dev/null
@@ -1,120 +0,0 @@
-/*
- * 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 <string>
-#include <tuple>
-#include <vector>
-#include "Resources.h"
-#include "SkCpu.h"
-#include "SkImage.h"
-#include "SkImage_Base.h"
-#include "SkOpts.h"
-#include "Test.h"
-#include "../include/core/SkImageInfo.h"
-
-typedef void (*Blender)(uint32_t* dst, const uint32_t* const srcStart, int ndst, const int nsrc);
-
-namespace sk_default {
-extern void brute_force_srcover_srgb_srgb(
- uint32_t* dst, const uint32_t* const srcStart, int ndst, const int nsrc);
-}
-
-namespace sk_default {
-extern void trivial_srcover_srgb_srgb(
- uint32_t* dst, const uint32_t* const srcStart, int ndst, const int nsrc);
-
-extern void best_non_simd_srcover_srgb_srgb(
- uint32_t* dst, const uint32_t* const srcStart, int ndst, const int nsrc);
-
-extern void srcover_srgb_srgb(
- uint32_t* dst, const uint32_t* const srcStart, int ndst, const int nsrc);
-}
-
-#if defined(SK_CPU_X86) && !defined(SK_BUILD_FOR_IOS)
-namespace sk_sse41 {
-extern void srcover_srgb_srgb(
- uint32_t* dst, const uint32_t* const srcStart, int ndst, const int nsrc);
-}
-#endif
-
-static SkString missmatch_message(std::string resourceName, std::string name, int x, int y,
- uint32_t src, uint32_t good, uint32_t bad) {
- return SkStringPrintf(
- "%s - %s missmatch at %d, %d src: %08x good: %08x bad: %08x",
- resourceName.c_str(), name.c_str(), x, y, src, good, bad);
-}
-
-using Spec = std::tuple<Blender, std::string>;
-
-static void test_blender(
- Spec spec,
- std::string resourceName,
- skiatest::Reporter* reporter)
-{
- Blender blender;
- std::string name;
- std::tie(blender, name) = spec;
-
- std::string fileName = resourceName + ".png";
- sk_sp<SkImage> image = GetResourceAsImage(fileName.c_str());
- SkASSERT(image != nullptr);
- if (image == nullptr) {
- SkFAIL("image is NULL");
- }
- SkBitmap bm;
- if (!as_IB(image)->getROPixels(&bm)) {
- SkFAIL("Could not read resource");
- }
-
- SkPixmap pixmap;
- bm.peekPixels(&pixmap);
- SkASSERTF(pixmap.colorType() == kN32_SkColorType, "colorType: %d", pixmap.colorType());
- SkASSERT(pixmap.alphaType() != kUnpremul_SkAlphaType);
- const uint32_t* src = pixmap.addr32();
- const int width = pixmap.rowBytesAsPixels();
- SkASSERT(width > 0);
- SkASSERT(width < 4000);
- SkAutoTArray<uint32_t> correctDst(width);
- SkAutoTArray<uint32_t> testDst(width);
-
- for (int y = 0; y < pixmap.height(); y++) {
- memset(correctDst.get(), 0, width * sizeof(uint32_t));
- memset(testDst.get(), 0, width * sizeof(uint32_t));
- sk_default::brute_force_srcover_srgb_srgb(correctDst.get(), src, width, width);
- blender(testDst.get(), src, width, width);
- for (int x = 0; x < width; x++) {
- REPORTER_ASSERT_MESSAGE(
- reporter, correctDst[x] == testDst[x],
- missmatch_message(resourceName, name, x, y, src[x], correctDst[x], testDst[x]));
- if (correctDst[x] != testDst[x]) break;
- }
- src += width;
- }
-}
-
-DEF_TEST(SkBlend_optsCheck, reporter) {
- std::vector<Spec> specs = {
- Spec{sk_default::trivial_srcover_srgb_srgb, "trivial"},
- Spec{sk_default::best_non_simd_srcover_srgb_srgb, "best_non_simd"},
- Spec{sk_default::srcover_srgb_srgb, "default"},
- };
- #if defined(SK_CPU_X86) && !defined(SK_BUILD_FOR_IOS)
- if (SkCpu::Supports(SkCpu::SSE41)) {
- specs.push_back(Spec{sk_sse41::srcover_srgb_srgb, "sse41", });
- }
- #endif
-
- std::vector<std::string> testResources = {
- "yellow_rose", "baby_tux", "plane", "mandrill_512", "iconstrip"
- };
-
- for (auto& spec : specs) {
- for (auto& resourceName : testResources) {
- test_blender(spec, resourceName, reporter);
- }
- }
-}
« no previous file with comments | « src/opts/SkOpts_sse41.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698