Add an option on SkImageDecoder to skip writing 0s.

bench/SkipZeroesBench.cpp

Index: bench/SkipZeroesBench.cpp
diff --git a/bench/ImageDecodeBench.cpp b/bench/SkipZeroesBench.cpp
similarity index 69%
copy from bench/ImageDecodeBench.cpp
copy to bench/SkipZeroesBench.cpp
index 3a61163862082830bfcbcc7b3d5f8d421ff9b335..90abf6a91d1a16e25dc2d16e35477e166e3f36b0 100644
--- a/bench/ImageDecodeBench.cpp
+++ b/bench/SkipZeroesBench.cpp
@@ -18,17 +18,25 @@ __SK_FORCE_IMAGE_DECODER_LINKING;
 
 class SkCanvas;
 
-class ImageDecodeBench : public SkBenchmark {
+class SkipZeroesBench : public SkBenchmark {
 public:
-    ImageDecodeBench(void* p, const char* filename)
-    : fName("image_decode_")
+    SkipZeroesBench(const char* filename, bool skipZeroes)
+    : fName("SkipZeroes_")
+    , fDecoder(NULL)
     , fFilename(filename)
     , fStream()
+    , fSkipZeroes(skipZeroes)
     , fValid(false) {
         fName.append(SkOSPath::SkBasename(filename));
+        if (skipZeroes) {
+            fName.append("_skip_zeroes");
+        }
         fIsRendering = false;
     }
 
+    ~SkipZeroesBench() {
+        SkDELETE(fDecoder);
+    }
 protected:
     virtual const char* onGetName() SK_OVERRIDE {
         return fName.c_str();
@@ -45,6 +53,12 @@ protected:
             } else {
                 SkAutoTUnref<SkData> skdata(SkData::NewFromMalloc(data, size));
                 fStream.setData(skdata.get());
+                fDecoder = SkImageDecoder::Factory(&fStream);
+                if (fDecoder) {
+                    fDecoder->setSkipWritingZeroes(fSkipZeroes);
+                } else {
+                    fValid = false;
+                }
             }
         }
     }
@@ -59,7 +73,7 @@ protected:
         // Decode a bunch of times
         SkBitmap bm;
         for (int i = 0; i < this->getLoops(); ++i) {
-            SkDEBUGCODE(bool success =) SkImageDecoder::DecodeStream(&fStream, &bm);
+            SkDEBUGCODE(bool success =) fDecoder->decode(&fStream, &bm, SkImageDecoder::kDecodePixels_Mode);
 #ifdef SK_DEBUG
             if (!success) {
                 SkDebugf("failed to decode %s\n", fName.c_str());
@@ -78,13 +92,14 @@ protected:
 
 private:
     SkString        fName;
+    SkImageDecoder* fDecoder;
     const SkString  fFilename;
     SkMemoryStream  fStream;
+    bool            fSkipZeroes;
     bool            fValid;
 
     typedef SkBenchmark INHERITED;
 };
 
-// These are files which call decodePalette
-//DEF_BENCH( return SkNEW_ARGS(ImageDecodeBench, ("/usr/local/google/home/scroggo/Downloads/images/hal_163x90.png")); )
-//DEF_BENCH( return SkNEW_ARGS(ImageDecodeBench, ("/usr/local/google/home/scroggo/Downloads/images/box_19_top-left.png")); )
+//DEF_BENCH( return SkNEW_ARGS(SkipZeroesBench, ("/sdcard/skia/images/arrow.png", true)));

[djsollen, 2013/09/19 19:41:17]

seems like we should be able to supply a resources directory as an arg to bench
so that we can enable this test if the resources are present.

[scroggo, 2013/09/19 21:07:21]

I can add this next week. Also as a note to myself, reed suggested I add the
benchmark before the change. On my local device, the benchmark runs in about the
same amount of time (as the false case, where we're not skipping zeroes).

+//DEF_BENCH( return SkNEW_ARGS(SkipZeroesBench, ("/sdcard/skia/images/arrow.png", false)));