commit 22f348281a202f79989447cfde45a3834649e89f
author Mike Reed <reed@google.com> Mon Nov 28 17:17:38 2016 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Tue Nov 29 12:58:26 2016 +0000
tree 76d6c3c768934d87c44b022690b6e872ff154ec7
parent 2c8104eb18d15916db8f7b648bd79f7384397af9

don't support converting from alpha to non-alpha

BUG=skia:

Change-Id: Ia06bef6c0bfc03b5ca9f569c07e993da0bbd67c3
Reviewed-on: https://skia-review.googlesource.com/5288
Reviewed-by: Matt Sarett <msarett@google.com>
Commit-Queue: Mike Reed <reed@google.com>

src/core/SkBitmap.cpp

diff --git a/src/core/SkBitmap.cpp b/src/core/SkBitmap.cpp
index 7ea49a8..e4097b4 100644
--- a/src/core/SkBitmap.cpp
+++ b/src/core/SkBitmap.cpp
@@ -707,15 +707,18 @@
 
 ///////////////////////////////////////////////////////////////////////////////
 
-bool SkBitmap::canCopyTo(SkColorType dstColorType) const {
+bool SkBitmap::canCopyTo(SkColorType dstCT) const {
     const SkColorType srcCT = this->colorType();
 
     if (srcCT == kUnknown_SkColorType) {
         return false;
     }
+    if (srcCT == kAlpha_8_SkColorType && dstCT != kAlpha_8_SkColorType) {
+        return false;   // can't convert from alpha to non-alpha
+    }
 
-    bool sameConfigs = (srcCT == dstColorType);
-    switch (dstColorType) {
+    bool sameConfigs = (srcCT == dstCT);
+    switch (dstCT) {
         case kAlpha_8_SkColorType:
         case kRGB_565_SkColorType:
         case kRGBA_8888_SkColorType:

src/core/SkConfig8888.cpp

diff --git a/src/core/SkConfig8888.cpp b/src/core/SkConfig8888.cpp
index 9af3621..3906a9a 100644
--- a/src/core/SkConfig8888.cpp
+++ b/src/core/SkConfig8888.cpp
@@ -324,6 +324,12 @@
         return false;
     }
 
+    if (srcInfo.colorType() == kAlpha_8_SkColorType &&
+        dstInfo.colorType() != kAlpha_8_SkColorType)
+    {
+        return false;   // can't convert from alpha to non-alpha
+    }
+
     const int width = srcInfo.width();
     const int height = srcInfo.height();
 

tests/BitmapCopyTest.cpp

diff --git a/tests/BitmapCopyTest.cpp b/tests/BitmapCopyTest.cpp
index 639c51a..ab0ec30 100644
--- a/tests/BitmapCopyTest.cpp
+++ b/tests/BitmapCopyTest.cpp
@@ -183,7 +183,7 @@
 
 static const Pair gPairs[] = {
     { kUnknown_SkColorType,     "000000"  },
-    { kAlpha_8_SkColorType,     "010101"  },
+    { kAlpha_8_SkColorType,     "010000"  },
     { kIndex_8_SkColorType,     "011111"  },
     { kRGB_565_SkColorType,     "010101"  },
     { kARGB_4444_SkColorType,   "010111"  },