update bitmapshader to avoid potential rendering error for kA8_Config

update bitmapshader to avoid potential rendering error for kA8_Config

A local matrix is set to the shader, which will show the potential error for SkCanvas::drawBitmap when the bitmap is kA8_Config, as well as the potential error for drawing a geometry with a bitmapshader who's bitmap is kA8_Config.

A simple case is also added for SkCanvas::drawBitmap when the bitmap is kA8_Config, but there is not shader. Then the drawing area will be colorized by the solid color set in SkPaint.

BUG=skia:2278
BUG=skia:2293
Committed: http://code.google.com/p/skia/source/detail?r=13904

[yunchao, 2014-03-19 06:45:23]

PTAL

[yunchao, 2014-03-21 14:50:28]

On 2014/03/19 06:45:23, Richard Ho wrote:
> PTAL

Mike and Brian, any suggestion about this CL?

[reed1, 2014-03-21 16:01:30]

lgtm

[yunchao, 2014-03-22 13:59:03]

The CQ bit was checked by yunchao.he@intel.com

[commit-bot: I haz the power, 2014-03-22 13:59:08]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/yunchao.he@intel.com/204143004/40001

[commit-bot: I haz the power, 2014-03-22 13:59:10]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-22 13:59:11]

Failed to apply patch for expectations/gm/ignored-tests.txt:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file expectations/gm/ignored-tests.txt
  Hunk #1 FAILED at 51.
  1 out of 1 hunk FAILED -- saving rejects to file
expectations/gm/ignored-tests.txt.rej

Patch:       expectations/gm/ignored-tests.txt
Index: expectations/gm/ignored-tests.txt
diff --git a/expectations/gm/ignored-tests.txt
b/expectations/gm/ignored-tests.txt
index
d9db6bce52b2042a432bd695171df2063e3bc64d..dfa033071e77adc619ada80aff3f503c499e97ef
100644
--- a/expectations/gm/ignored-tests.txt
+++ b/expectations/gm/ignored-tests.txt
@@ -51,3 +51,7 @@ rrect_clip_aa
 # This change removes an API that this GM was testing. If/when it lands and
sticks,
 # I will likely just delete the GM.
 canvas-layer-state
+
+# yunchao: Additional test cases were added to bitmapshader to avoid potential
+# drawing error for kA8_Config: https://codereview.chromium.org/204143004
+bitmapshaders

[yunchao, 2014-03-24 02:33:11]

The CQ bit was checked by yunchao.he@intel.com

[commit-bot: I haz the power, 2014-03-24 02:33:17]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/yunchao.he@intel.com/204143004/80001

[commit-bot: I haz the power, 2014-03-24 02:49:50]

Change committed as 13904