Use Analytic AA in SkAAClip

src/core/SkAAClip.cpp

Index: src/core/SkAAClip.cpp
diff --git a/src/core/SkAAClip.cpp b/src/core/SkAAClip.cpp
index 088ee55844473aab14fecb4fb4286f457f5ed331..ba018e1c9cf4d29d3e806814fd5a9fbbe14c67fb 100644
--- a/src/core/SkAAClip.cpp
+++ b/src/core/SkAAClip.cpp
@@ -1037,8 +1037,11 @@ public:
 
     void addAntiRectRun(int x, int y, int width, int height,
                         SkAlpha leftAlpha, SkAlpha rightAlpha) {
+        // According to SkBlitter.cpp, no matter whether leftAlpha is 0 or positive,
+        // we should always consider [x, x+1] as the left-most column and [x+1, x+1+width]
+        // as the rect with full alpha.
         SkASSERT(fBounds.contains(x + width - 1 +
-                 (leftAlpha > 0 ? 1 : 0) + (rightAlpha > 0 ? 1 : 0),
+                 1 + (rightAlpha > 0 ? 1 : 0),

[caryclark, 2016/10/20 16:58:14]

could this be

SkASSERT(fBounds.contains(x + width + (rightAlpha > 0), y + height - 1));

[liyuqian, 2016/10/20 17:19:50]

Sure, done. I was just preserving the old format for comparison.

                  y + height - 1));
         SkASSERT(width >= 0);
 
@@ -1048,6 +1051,9 @@ public:
             width++;
         } else if (leftAlpha > 0) {
           this->addRun(x++, y, leftAlpha, 1);
+        } else {
+          // leftAlpha is 0, ignore the left column
+          x++;
         }
         if (rightAlpha == 0xFF) {
             width++;
@@ -1274,9 +1280,17 @@ public:
        any failure cases that misses may have minor artifacts.
     */
     void blitV(int x, int y, int height, SkAlpha alpha) override {
-        this->recordMinY(y);
-        fBuilder->addColumn(x, y, alpha, height);
-        fLastY = y + height - 1;
+        if (height == 1) {
+            // We're still in scan-line order if height is 1
+            // This is useful for Analytic AA
+            const SkAlpha alphas[2] = {alpha, 0};
+            const int16_t runs[2] = {1, 0};
+            this->blitAntiH(x, y, alphas, runs);
+        } else {
+            this->recordMinY(y);
+            fBuilder->addColumn(x, y, alpha, height);
+            fLastY = y + height - 1;
+        }
     }
 
     void blitRect(int x, int y, int width, int height) override {
@@ -1398,7 +1412,11 @@ bool SkAAClip::setPath(const SkPath& path, const SkRegion* clip, bool doAA) {
     BuilderBlitter blitter(&builder);
 
     if (doAA) {
-        SkScan::AntiFillPath(path, *clip, &blitter, true);
+        if (gSkUseAnalyticAA.load()) {
+            SkScan::AAAFillPath(path, *clip, &blitter, true);
+        } else {
+            SkScan::AntiFillPath(path, *clip, &blitter, true);
+        }
     } else {
         SkScan::FillPath(path, *clip, &blitter);
     }