Do not convert non gray PNG to A8.

src/images/SkImageDecoder_libpng.cpp

Index: src/images/SkImageDecoder_libpng.cpp
diff --git a/src/images/SkImageDecoder_libpng.cpp b/src/images/SkImageDecoder_libpng.cpp
index b6aa329104288aead5ae32f788e0f80b9aba153d..fabd9bd231b17699f55744c06995853dcae19416 100644
--- a/src/images/SkImageDecoder_libpng.cpp
+++ b/src/images/SkImageDecoder_libpng.cpp
@@ -535,9 +535,13 @@ bool SkPNGImageDecoder::getBitmapConfig(png_structp png_ptr, png_infop info_ptr,
                 *configp = SkBitmap::kARGB_8888_Config;
             }
         } else {
-            if (*configp != SkBitmap::kRGB_565_Config &&
-                *configp != SkBitmap::kARGB_4444_Config &&
-                *configp != SkBitmap::kA8_Config) {
+            if (SkBitmap::kA8_Config == *configp) {
+                if (PNG_COLOR_TYPE_GRAY != colorType) {

[reed1, 2013/09/27 14:20:54]

Is this the same as comparing against srcDepth? If so, I think that might read
better, since earlier (#525) we seem to be converting from png-specific
colorType to skia-specific srcDepth.

[scroggo, 2013/09/27 21:21:35]

Yes, that would be equivalent.

[scroggo, 2013/09/30 20:26:56]

Done.

+                    // Converting a non grayscale image to A8 is not currently supported.
+                    *configp = SkBitmap::kARGB_8888_Config;
+                }
+            } else if (*configp != SkBitmap::kRGB_565_Config &&
+                       *configp != SkBitmap::kARGB_4444_Config) {
                 *configp = SkBitmap::kARGB_8888_Config;
             }
         }