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

Side by Side Diff: tests/SkBlend_optsTest.cpp

Issue 1939513002: Add specialized sRGB blitter for SkOpts (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Address comments. 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 unified diff | Download patch
« src/opts/SkBlend_opts.h ('K') | « src/opts/SkOpts_sse41.cpp ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 /*
2 * Copyright 2016 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
8 #include <string>
9 #include <tuple>
10 #include <vector>
11 #include "Resources.h"
12 #include "SkCpu.h"
13 #include "SkImage.h"
14 #include "SkImage_Base.h"
15 #include "SkOpts.h"
16 #include "Test.h"
17
18 typedef void (*Blender)(uint32_t* dst, const uint32_t* const srcStart, int ndst, const int nsrc);
19
20 namespace sk_default {
21 extern void brute_force_srcover_srgb_srgb(
22 uint32_t* dst, const uint32_t* const srcStart, int ndst, const int nsrc);
23 }
24
25 namespace sk_default {
26 extern void trivial_srcover_srgb_srgb(
27 uint32_t* dst, const uint32_t* const srcStart, int ndst, const int nsrc);
28
29 extern void best_non_simd_srcover_srgb_srgb(
30 uint32_t* dst, const uint32_t* const srcStart, int ndst, const int nsrc);
31
32 extern void srcover_srgb_srgb(
33 uint32_t* dst, const uint32_t* const srcStart, int ndst, const int nsrc);
34 }
35
36 #if defined(SK_CPU_X86) && !defined(SK_BUILD_FOR_IOS)
37 namespace sk_sse41 {
38 extern void srcover_srgb_srgb(
39 uint32_t* dst, const uint32_t* const srcStart, int ndst, const int nsrc);
40 }
41 #endif
42
43 static SkString missmatch_message(std::string resourceName, std::string name, in t x, int y,
44 uint32_t src, uint32_t good, uint32_t bad) {
45 return SkStringPrintf(
46 "%s - %s missmatch at %d, %d src: %08x good: %08x bad: %08x",
47 resourceName.c_str(), name.c_str(), x, y, src, good, bad);
48 }
49
50 static void test_blender(
51 std::tuple<Blender, std::string> spec,
52 std::string resourceName,
53 skiatest::Reporter* reporter)
54 {
55 Blender blender;
56 std::string name;
57 std::tie(blender, name) = spec;
58
59 std::string fileName = resourceName + ".png";
60 sk_sp<SkImage> image = GetResourceAsImage(fileName.c_str());
61 SkASSERT(image != nullptr);
62 if (image == nullptr) {
63 SkFAIL("image is NULL");
64 }
65 SkBitmap bm;
66 if (!as_IB(image)->getROPixels(&bm)) {
67 SkFAIL("Could not read resource");
68 }
69
70 SkPixmap pixmap;
71 bm.peekPixels(&pixmap);
72 SkASSERT(pixmap.colorType() == kRGBA_8888_SkColorType);
73 SkASSERT(pixmap.alphaType() == kPremul_SkAlphaType);
74 const int width = pixmap.rowBytesAsPixels();
75 const uint32_t* src = pixmap.addr32();
76 SkAutoTArray<uint32_t> correctDst(width);
77 SkAutoTArray<uint32_t> testDst(width);
78
79 for (int y = 0; y < pixmap.height(); y++) {
80 memset(correctDst.get(), 0, width * sizeof(uint32_t));
81 memset(testDst.get(), 0, width * sizeof(uint32_t));
82 sk_default::brute_force_srcover_srgb_srgb(correctDst.get(), src, width, width);
83 blender(testDst.get(), src, width, width);
84 for (int x = 0; x < width; x++) {
85 REPORTER_ASSERT_MESSAGE(
86 reporter, correctDst[x] == testDst[x],
87 missmatch_message(resourceName, name, x, y, src[x], correctDst[x ], testDst[x]));
88 break;
f(malita) 2016/05/06 21:04:00 I think we want to break only when correctDst[x] !
herb_g 2016/05/06 21:19:50 Done.
89 }
90 src += width;
91 }
92 }
93
94 DEF_TEST(SkBlend_optsCheck, reporter) {
95 std::vector<std::tuple<Blender, std::string>> specs = {
96 {sk_default::trivial_srcover_srgb_srgb, "trivial"},
97 {sk_default::best_non_simd_srcover_srgb_srgb, "best_non_simd"},
98 {sk_default::srcover_srgb_srgb, "default"},
99 };
100 #if defined(SK_CPU_X86) && !defined(SK_BUILD_FOR_IOS)
101 if (SkCpu::Supports(SkCpu::SSE41)) {
102 specs.push_back({sk_sse41::srcover_srgb_srgb, "sse41", });
103 }
104 #endif
105
106 std::vector<std::string> testResources = {
107 "yellow_rose", "baby_tux", "plane", "mandrill_512", "iconstrip"
108 };
109
110 for (auto& spec : specs) {
111 for (auto& resourceName : testResources) {
112 test_blender(spec, resourceName, reporter);
113 }
114 }
115 }
OLDNEW
« src/opts/SkBlend_opts.h ('K') | « src/opts/SkOpts_sse41.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698