DM: fix failures when using -r by comparing unpremultiplied.

dm/DMWriteTask.cpp

Index: dm/DMWriteTask.cpp
diff --git a/dm/DMWriteTask.cpp b/dm/DMWriteTask.cpp
index 11b9fd3470738acaa865f0c2685d7c377cc87213..c32f0ab3d2c2d9b434aacd1d2c23d07125f2d8d1 100644
--- a/dm/DMWriteTask.cpp
+++ b/dm/DMWriteTask.cpp
@@ -5,6 +5,7 @@
 #include "SkImageDecoder.h"
 #include "SkImageEncoder.h"
 #include "SkString.h"
+#include "SkUnPreMultiply.h"
 
 DEFINE_string2(writePath, w, "", "If set, write GMs here as .pngs.");
 
@@ -89,22 +90,63 @@ static SkString path_to_expected_image(const char* root, const Task& task) {
 }
 
 bool WriteTask::Expectations::check(const Task& task, SkBitmap bitmap) const {
+    // PNG is stored unpremultiplied, and going from premul to unpremul to premul is lossy.  To
+    // skirt this problem, we decode the PNG into an unpremul bitmap, convert our bitmap to unpremul
+    // if needed, and compare those.  Each image goes once from premul to unpremul, never back.
     const SkString path = path_to_expected_image(fRoot, task);
 
+    SkAutoTUnref<SkStreamRewindable> stream(SkStream::NewFromFile(path.c_str()));
+    if (NULL == stream.get()) {
+        SkDebugf("Could not read %s.\n", path.c_str());
+        return false;
+    }
+
+    SkAutoTDelete<SkImageDecoder> decoder(SkImageDecoder::Factory(stream));
+    if (NULL == decoder.get()) {
+        SkDebugf("Could not find a decoder for %s.\n", path.c_str());
+        return false;
+    }
+
+    SkImageInfo info;
+    SkAssertResult(bitmap.asImageInfo(&info));
+
     SkBitmap expected;
-    if (SkImageDecoder::DecodeFile(path.c_str(), &expected)) {
-        if (expected.config() != bitmap.config()) {
-            SkBitmap converted;
-            SkAssertResult(expected.copyTo(&converted, bitmap.config()));
-            expected.swap(converted);
+    expected.setConfig(info);
+    expected.allocPixels();
+
+    // expected will be unpremultiplied.
+    decoder->setRequireUnpremultipliedColors(true);
+    if (!decoder->decode(stream, &expected, SkImageDecoder::kDecodePixels_Mode)) {
+        SkDebugf("Could not decode %s.\n", path.c_str());
+        return false;
+    }
+
+    // We always seem to decode to 8888.  This puts 565 back in 565.
+    if (expected.config() != bitmap.config()) {
+        SkBitmap converted;
+        SkAssertResult(expected.copyTo(&converted, bitmap.config()));
+        expected.swap(converted);
+    }
+    SkASSERT(expected.config() == bitmap.config());
+
+    // Manually unpremultiply 8888 bitmaps to match expected.
+    // Their pixels are shared, concurrently even, so we must copy them.
+    if (bitmap.config() == SkBitmap::kARGB_8888_Config) {

[bsalomon, 2014/01/06 18:46:43]

Can we look at the bitmap alpha type rather than config?

+        SkBitmap unpremul;
+        unpremul.setConfig(info);
+        unpremul.allocPixels();
+
+        SkAutoLockPixels lockSrc(bitmap), lockDst(unpremul);
+        const SkPMColor* src = (SkPMColor*)bitmap.getPixels();
+        SkColor* dst = (SkColor*)unpremul.getPixels();
+
+        for (size_t i = 0; i < bitmap.getSize()/4; i++) {
+            dst[i] = SkUnPreMultiply::PMColorToColor(src[i]);
         }
-        SkASSERT(expected.config() == bitmap.config());
-        return BitmapsEqual(expected, bitmap);
+        bitmap.swap(unpremul);
     }
 
-    // Couldn't read the file, etc.
-    SkDebugf("Problem decoding %s to SkBitmap.\n", path.c_str());
-    return false;
+    return BitmapsEqual(expected, bitmap);
 }
 
 }  // namespace DM