diff options
author | 2016-11-28 17:26:27 -0500 | |
---|---|---|
committer | 2016-11-28 23:01:45 +0000 | |
commit | 6f0286f0b4e61dd6373b693f7f1913cd66eba074 (patch) | |
tree | 31efad1e9e70e2eaa62c4a34209439ecbaaae9ba /tests/ImageTest.cpp | |
parent | 19671772c1a03bd973fae54dead4defc9ee1d4c3 (diff) |
use raster-pipeline in readPixels
1. tons of 565 diffs in gm, so skipping that for now
2. 32bit premul/unpremul conversions are slower than existing code, so for now use the pipeline after the existing special-cases.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=5152
CQ_INCLUDE_TRYBOTS=master.client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot;skia.primary:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD
Change-Id: I6ca43b6dd24434814f8f10cdaaabbaf396914d1a
Reviewed-on: https://skia-review.googlesource.com/5152
Commit-Queue: Mike Reed <reed@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Reviewed-by: Matt Sarett <msarett@google.com>
Reviewed-by: Mike Klein <mtklein@chromium.org>
Reviewed-by: Herb Derby <herb@google.com>
Diffstat (limited to 'tests/ImageTest.cpp')
-rw-r--r-- | tests/ImageTest.cpp | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/tests/ImageTest.cpp b/tests/ImageTest.cpp index 020e322f91..45681be648 100644 --- a/tests/ImageTest.cpp +++ b/tests/ImageTest.cpp @@ -1016,8 +1016,12 @@ static bool equal(const SkBitmap& a, const SkBitmap& b) { SkASSERT(a.width() == b.width()); SkASSERT(a.height() == b.height()); for (int y = 0; y < a.height(); ++y) { - if (0 != memcmp(a.getAddr32(0, y), b.getAddr32(0, y), a.width() * sizeof(SkPMColor))) { - return false; + for (int x = 0; x < a.width(); ++x) { + SkPMColor pa = *a.getAddr32(x, y); + SkPMColor pb = *b.getAddr32(x, y); + if (pa != pb) { + return false; + } } } return true; |