Ico test with embedded png

tests/CodexTest.cpp

Index: tests/CodexTest.cpp
diff --git a/tests/CodexTest.cpp b/tests/CodexTest.cpp
index f4ed0e8b02858bceb343fbe5092573c4643a9a83..1689bafd33f1e119fb42fa854fde1ce04372e6c1 100644
--- a/tests/CodexTest.cpp
+++ b/tests/CodexTest.cpp
@@ -77,7 +77,11 @@ DEF_TEST(Codec, r) {
     check(r, "randPixels.bmp", SkISize::Make(8, 8), true);
 
     // ICO
+    // These two tests examine interestingly different behavior:
+    // Decodes an embedded BMP image
     check(r, "color_wheel.ico", SkISize::Make(128, 128), true);

[scroggo, 2015/04/01 18:28:21]

Sorry; you had a conflict with my change that just went in.

canRewind is now always true, so I removed it. Now the boolean variable is
supportsScanlineDecoding, which is false for ICO, so these should both be false.

+    // Decodes an embedded PNG image
+    check(r, "google_chrome.ico", SkISize::Make(256, 256), true);
 
     // PNG
     check(r, "arrow.png", SkISize::Make(187, 312), true);