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

Unified Diff: tests/CodexTest.cpp

Issue 1048423003: Make SkPngCodec support rewinding properly. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 5 years, 9 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/SkCodec_libpng.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/CodexTest.cpp
diff --git a/tests/CodexTest.cpp b/tests/CodexTest.cpp
index ec597b60434e31dbd0088f30f9281b320a9fc156..2671b1422992a97428e4754cb07708b195af4d36 100644
--- a/tests/CodexTest.cpp
+++ b/tests/CodexTest.cpp
@@ -82,18 +82,17 @@ DEF_TEST(Codec, r) {
check(r, "color_wheel.ico", SkISize::Make(128, 128), false);
// PNG
- // TODO (scroggo): SkPngCodec should be able to rewind.
- check(r, "arrow.png", SkISize::Make(187, 312), false);
- check(r, "baby_tux.png", SkISize::Make(240, 246), false);
- check(r, "color_wheel.png", SkISize::Make(128, 128), false);
- check(r, "half-transparent-white-pixel.png", SkISize::Make(1, 1), false);
- check(r, "mandrill_128.png", SkISize::Make(128, 128), false);
- check(r, "mandrill_16.png", SkISize::Make(16, 16), false);
- check(r, "mandrill_256.png", SkISize::Make(256, 256), false);
- check(r, "mandrill_32.png", SkISize::Make(32, 32), false);
- check(r, "mandrill_512.png", SkISize::Make(512, 512), false);
- check(r, "mandrill_64.png", SkISize::Make(64, 64), false);
- check(r, "plane.png", SkISize::Make(250, 126), false);
- check(r, "randPixels.png", SkISize::Make(8, 8), false);
- check(r, "yellow_rose.png", SkISize::Make(400, 301), false);
+ check(r, "arrow.png", SkISize::Make(187, 312), true);
+ check(r, "baby_tux.png", SkISize::Make(240, 246), true);
+ check(r, "color_wheel.png", SkISize::Make(128, 128), true);
+ check(r, "half-transparent-white-pixel.png", SkISize::Make(1, 1), true);
+ check(r, "mandrill_128.png", SkISize::Make(128, 128), true);
+ check(r, "mandrill_16.png", SkISize::Make(16, 16), true);
+ check(r, "mandrill_256.png", SkISize::Make(256, 256), true);
+ check(r, "mandrill_32.png", SkISize::Make(32, 32), true);
+ check(r, "mandrill_512.png", SkISize::Make(512, 512), true);
+ check(r, "mandrill_64.png", SkISize::Make(64, 64), true);
+ check(r, "plane.png", SkISize::Make(250, 126), true);
+ check(r, "randPixels.png", SkISize::Make(8, 8), true);
+ check(r, "yellow_rose.png", SkISize::Make(400, 301), true);
}
« no previous file with comments | « src/codec/SkCodec_libpng.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698