Swizzler changes Index8 and 565

dm/DM.cpp

 /*
  * Copyright 2013 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "CrashHandler.h"
 #include "DMJsonWriter.h"
 #include "DMSrcSink.h"
 #include "DMSrcSinkAndroid.h"
 #include "OverwriteLine.h"
 #include "ProcStats.h"
 #include "SkBBHFactory.h"
 #include "SkChecksum.h"
+#include "SkCodecPriv.h"
 #include "SkCommonFlags.h"
 #include "SkForceLinking.h"
 #include "SkGraphics.h"
 #include "SkInstCnt.h"
 #include "SkMD5.h"
 #include "SkOSFile.h"
 #include "SkTHash.h"
 #include "SkTaskGroup.h"
 #include "SkThreadUtils.h"
 #include "Test.h"
@@ skipping 145 matching lines @@
     SkAutoTDelete<Src> src(s);
     if (FLAGS_src.contains(tag) &&
         !SkCommandLineFlags::ShouldSkip(FLAGS_match, src->name().c_str())) {
         Tagged<Src>& s = gSrcs.push_back();
         s.reset(src.detach());
         s.tag = tag;
         s.options = options;
     }
 }
 
+static SkCodec* create_codec(Path path) {
+    SkAutoTUnref<SkData> encoded(SkData::NewFromFileName(path.c_str()));
+    if (!encoded) {
+        SkCodecPrintf(return SkStringPrintf("Couldn't read %s.", path.c_str()));

[scroggo, 2015/04/08 17:21:27]

SkCodecPrintf (which I think is defined in src/codec?), should only be used in
src/codec.

Also I do not think we should use a return statement inside a print statement
like this. I think what you want is

SkDebugf("Couldn't read %s.", path.c_str());

[msarett, 2015/04/08 19:35:40]

Yeah that is what I meant.

+        return NULL;
+    }
+
+    return SkCodec::NewFromData(encoded);
+}
+
+static void push_codec_srcs(Path path) {
+    // Create a codec
+    SkCodec* codec = create_codec(path);
+    if (NULL == codec) {
+        SkCodecPrintf(return SkStringPrintf("Couldn't create codec for %s.", path.c_str()));
+        return;
+    }
+
+    // Build additional test cases for images that decode natively to non-canvas types
+    switch(codec->getInfo().colorType()) {
+      case kIndex_8_SkColorType:
+          push_src("image", "codec kIndex8", new CodecSrc(path, CodecSrc::kNormal_Mode,

[scroggo, 2015/04/08 17:21:27]

We should probably add a src for kScanline_Mode as well.

[msarett, 2015/04/08 19:35:40]

Done.

+                  CodecSrc::kIndex8_Always_DstColorType, create_codec(path)));

[msarett, 2015/04/08 13:59:10]

Can we assume that if create_codec succeeds once, it will always succeed?

[scroggo, 2015/04/08 17:21:27]

If it does not, that is a bug (or a disk read error).

[msarett, 2015/04/08 19:35:40]

Acknowledged.

+        break;
+      case kGray_8_SkColorType:
+          push_src("image", "codec kGray8", new CodecSrc(path, CodecSrc::kNormal_Mode,
+                  CodecSrc::kGrayscale_Always_DstColorType, create_codec(path)));
+        break;
+      default:
+        // Do nothing
+        break;
+    }
+
+    // Decode all images to the canvas color type
+    push_src("image", "codec", new CodecSrc(path, CodecSrc::kNormal_Mode,
+            CodecSrc::kGetFromCanvas_DstColorType, create_codec(path)));
+    push_src("image", "scanline", new CodecSrc(
+            path, CodecSrc::kScanline_Mode, CodecSrc::kGetFromCanvas_DstColorType, codec));
+}
+
 static bool codec_supported(const char* ext) {
     // FIXME: Once other versions of SkCodec are available, we can add them to this
     // list (and eventually we can remove this check once they are all supported).
     static const char* const exts[] = {
         "bmp", "gif", "png", "ico", "wbmp",
         "BMP", "GIF", "PNG", "ICO", "WBMP"
     };
 
     for (uint32_t i = 0; i < SK_ARRAY_COUNT(exts); i++) {
         if (0 == strcmp(exts[i], ext)) {
@@ skipping 25 matching lines @@
     for (int i = 0; i < FLAGS_images.count(); i++) {
         const char* flag = FLAGS_images[i];
         if (sk_isdir(flag)) {
             for (size_t j = 0; j < SK_ARRAY_COUNT(exts); j++) {
                 SkOSFile::Iter it(flag, exts[j]);
                 for (SkString file; it.next(&file); ) {
                     SkString path = SkOSPath::Join(flag, file.c_str());
                     push_src("image", "decode", new ImageSrc(path)); // Decode entire image
                     push_src("image", "subset", new ImageSrc(path, 2)); // Decode into 2x2 subsets
                     if (codec_supported(exts[j])) {
-                        push_src("image", "codec", new CodecSrc(path, CodecSrc::kNormal_Mode));
-                        push_src("image", "scanline", new CodecSrc(path, CodecSrc::kScanline_Mode));
+                        push_codec_srcs(path);
                     }
                 }
             }
         } else if (sk_exists(flag)) {
             // assume that FLAGS_images[i] is a valid image if it is a file.
             push_src("image", "decode", new ImageSrc(flag)); // Decode entire image.
             push_src("image", "subset", new ImageSrc(flag, 2)); // Decode into 2 x 2 subsets
-            push_src("image", "codec", new CodecSrc(flag, CodecSrc::kNormal_Mode));
-            push_src("image", "scanline", new CodecSrc(flag, CodecSrc::kScanline_Mode));
+            push_codec_srcs(flag);
         }
     }
 }
 
 static GrGLStandard get_gpu_api() {
     if (FLAGS_gpuAPI.contains("gl"))   { return kGL_GrGLStandard; }
     if (FLAGS_gpuAPI.contains("gles")) { return kGLES_GrGLStandard; }
     return kNone_GrGLStandard;
 }
 
@@ skipping 452 matching lines @@
     }
     return 0;
 }
 
 #if !defined(SK_BUILD_FOR_IOS)
 int main(int argc, char** argv) {
     SkCommandLineFlags::Parse(argc, argv);
     return dm_main();
 }
 #endif