From: robertphillips <robertphillips@google.com>
Date: Sun, 21 Dec 2014 18:52:01 +0000 (-0800)
Subject: Allow the alpha folding optimization if the single draw's paint has a shader
X-Git-Tag: accepted/tizen/5.0/unified/20181102.025319~4298
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ca32da7533ed86fbbe50e078c751b0f1c198bb5b;p=platform%2Fupstream%2FlibSkiaSharp.git

Allow the alpha folding optimization if the single draw's paint has a shader

Let's land this and then leave for 2 weeks.

BUG=skia:3119

Review URL: https://codereview.chromium.org/817033002
---

diff --git a/src/core/SkRecordOpts.cpp b/src/core/SkRecordOpts.cpp
index 7c24a08..4a51bee 100644
--- a/src/core/SkRecordOpts.cpp
+++ b/src/core/SkRecordOpts.cpp
@@ -112,7 +112,7 @@ struct SaveLayerDrawRestoreNooper {
         const uint32_t layerColor = layerPaint->getColor();
         const uint32_t  drawColor =  drawPaint->getColor();
         if (!IsOnlyAlpha(layerColor)  || !IsOpaque(drawColor) ||
-            HasAnyEffect(*layerPaint) || HasAnyEffect(*drawPaint)) {
+            HasAnyEffect(*layerPaint) || CantFoldAlpha(*drawPaint)) {
             // Too fancy for us.  Actually, as long as layerColor is just an alpha
             // we can blend it into drawColor's alpha; drawColor doesn't strictly have to be opaque.
             return false;
@@ -139,6 +139,17 @@ struct SaveLayerDrawRestoreNooper {
                paint.getImageFilter();
     }
 
+    // The alpha folding can proceed if the single draw's paint has a shader,
+    // path effect, mask filter and/or rasterizer.
+    // TODO: most likely the looper and only some xfer modes are the hard
+    // constraints
+    static bool CantFoldAlpha(const SkPaint& paint) {
+        return paint.getXfermode()    ||
+               paint.getColorFilter() ||
+               paint.getLooper()      ||
+               paint.getImageFilter();
+    }
+
     static bool IsOpaque(SkColor color) {
         return SkColorGetA(color) == SK_AlphaOPAQUE;
     }