Add AndroidCodecBench to time scaled decodes

bench/AndroidCodecBench.cpp

Index: bench/AndroidCodecBench.cpp
diff --git a/bench/AndroidCodecBench.cpp b/bench/AndroidCodecBench.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..01aae791990a4c0826695773bbbc8388a57a4b08
--- /dev/null
+++ b/bench/AndroidCodecBench.cpp
@@ -0,0 +1,56 @@
+/*
+ * Copyright 2015 Google Inc.

[scroggo, 2016/02/10 16:29:49]

nit 2016

[msarett, 2016/02/11 14:07:15]

Done.

+ *
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ */
+
+#include "AndroidCodecBench.h"
+#include "CodecBenchPriv.h"
+#include "SkBitmap.h"
+#include "SkAndroidCodec.h"
+#include "SkCommandLineFlags.h"
+#include "SkOSFile.h"
+
+AndroidCodecBench::AndroidCodecBench(SkString baseName, SkData* encoded, int sampleSize)
+    : fData(SkRef(encoded))
+    , fSampleSize(sampleSize)
+{
+    // Parse filename and the color type to give the benchmark a useful name
+    fName.printf("AndroidCodec_%s_SampleSize%d", baseName.c_str(), sampleSize);

[scroggo, 2016/02/10 16:29:49]

This just occurred to me - should we print the benchtype when print each line?
Then we could remove "AndroidCodec" from the name and rely on the type for that
info.

[msarett, 2016/02/10 20:06:45]

Do you think I should change the name to match the benchType or that we should
pass the benchType as a parameter to CodecBench, AndroidCodecBench, BRDBench
(and then use that to set fName)?

[scroggo, 2016/02/10 21:13:54]

My idea was that the name did not include the type at all - that would be stored
in the benchtype, which we would print on the same line as the results. 

Passing it as a parameter seems unnecessary (it'll always be set to the same
value).

But I'm also fine with leaving as is.

[msarett, 2016/02/11 14:07:15]

Ahh ok I understand.  I think it's a good idea, but I'd like to avoid making
that a part of this CL.

[scroggo, 2016/02/11 14:08:54]

sgtm

+}
+
+const char* AndroidCodecBench::onGetName() {
+    return fName.c_str();
+}
+
+bool AndroidCodecBench::isSuitableFor(Backend backend) {
+    return kNonRendering_Backend == backend;
+}
+
+void AndroidCodecBench::onDelayedSetup() {
+    SkAutoTDelete<SkAndroidCodec> codec(SkAndroidCodec::NewFromData(fData));
+    SkISize scaledSize = codec->getSampledDimensions(fSampleSize);
+
+    fInfo = codec->getInfo().makeWH(scaledSize.width(), scaledSize.height())
+            .makeColorType(kN32_SkColorType);

[scroggo, 2016/02/10 16:29:49]

why only n32, and only premul?

- edit: looking down below, it looks like this is to save time. sgt

[msarett, 2016/02/10 20:06:45]

Yep that's right

+    if (kUnpremul_SkAlphaType == fInfo.alphaType()) {
+        fInfo = fInfo.makeAlphaType(kPremul_SkAlphaType);
+    }
+
+    fPixelStorage.reset(fInfo.getSafeSize(fInfo.minRowBytes()));
+}
+
+void AndroidCodecBench::onDraw(int n, SkCanvas* canvas) {
+    SkAutoTDelete<SkAndroidCodec> codec;
+    SkAndroidCodec::AndroidOptions options;
+    options.fSampleSize = fSampleSize;
+    for (int i = 0; i < n; i++) {
+        codec.reset(SkAndroidCodec::NewFromData(fData));
+#ifdef SK_DEBUG
+        const SkCodec::Result result =
+#endif
+        codec->getAndroidPixels(fInfo, fPixelStorage.get(), fInfo.minRowBytes(), &options);
+        SkASSERT(result == SkCodec::kSuccess || result == SkCodec::kIncompleteInput);
+    }
+}