Split radial_gradients2 GM into two to reproduce its bug.

gm/gradients.cpp

 
 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 #include "gm.h"
 #include "SkGradientShader.h"
 
@@ skipping 345 matching lines @@
         };
         canvas->drawRect(r, paint);
     }
 private:
     typedef GM INHERITED;
 };
 
 
 class RadialGradient2GM : public GM {
 public:
-    RadialGradient2GM() {}
+    explicit RadialGradient2GM(bool premulFirst) : fPremulFirst(premulFirst) {}
 
 protected:
-    SkString onShortName() { return SkString("radial_gradient2"); }
+    SkString onShortName() {
+        return SkString(fPremulFirst ? "radial_gradient2_premul_then_interp"
+                                     : "radial_gradient2_interp_then_premul");
+    }
     virtual SkISize onISize() { return make_isize(400, 400); }
     void drawBG(SkCanvas* canvas) {
         canvas->drawColor(0xFF000000);
     }
 
     // Reproduces the example given in bug 7671058.
     virtual void onDraw(SkCanvas* canvas) {
         SkPaint paint1, paint2, paint3;
         paint1.setStyle(SkPaint::kFill_Style);
         paint2.setStyle(SkPaint::kFill_Style);
         paint3.setStyle(SkPaint::kFill_Style);
 
         const SkColor sweep_colors[] =
             { 0xFFFF0000, 0xFFFFFF00, 0xFF00FF00, 0xFF00FFFF, 0xFF0000FF, 0xFFFF00FF, 0xFFFF0000 };
         const SkColor colors1[] = { 0xFFFFFFFF, 0x00000000 };
         const SkColor colors2[] = { 0xFF000000, 0x00000000 };
 
         const SkScalar cx = 200, cy = 200, radius = 150;
         SkPoint center;
         center.set(cx, cy);
 
+        // We can either interpolate endpoints and premultiply each point (default, more precision),
+        // or premultiply the endpoints first, avoiding the need to premultiply each point (cheap).
+        const uint32_t flags = fPremulFirst ? SkGradientShader::kInterpolateColorsInPremul_Flag : 0;
+
         SkAutoTUnref<SkShader> sweep(
                 SkGradientShader::CreateSweep(cx, cy, sweep_colors,
-                                              NULL, SK_ARRAY_COUNT(sweep_colors)));
+                                              NULL, SK_ARRAY_COUNT(sweep_colors),
+                                              NULL, flags));
         SkAutoTUnref<SkShader> radial1(
                 SkGradientShader::CreateRadial(center, radius, colors1,
                                                NULL, SK_ARRAY_COUNT(colors1),
-                                               SkShader::kClamp_TileMode));
+                                               SkShader::kClamp_TileMode,
+                                               NULL, flags));
         SkAutoTUnref<SkShader> radial2(
                 SkGradientShader::CreateRadial(center, radius, colors2,
                                                NULL, SK_ARRAY_COUNT(colors2),
-                                               SkShader::kClamp_TileMode));
+                                               SkShader::kClamp_TileMode,
+                                               NULL, flags));
         paint1.setShader(sweep);
         paint2.setShader(radial1);
         paint3.setShader(radial2);
 
         canvas->drawCircle(cx, cy, radius, paint1);
         canvas->drawCircle(cx, cy, radius, paint3);
         canvas->drawCircle(cx, cy, radius, paint2);
     }
 
 private:
+    bool fPremulFirst;
     typedef GM INHERITED;
 };
 
 ///////////////////////////////////////////////////////////////////////////////
 
 static GM* MyFactory(void*) { return new GradientsGM; }
 static GMRegistry reg(MyFactory);
 
 static GM* MyFactory2(void*) { return new GradientsDegenrate2PointGM; }
 static GMRegistry reg2(MyFactory2);
 
 static GM* MyFactory3(void*) { return new ClampedGradientsGM; }
 static GMRegistry reg3(MyFactory3);
 
 static GM* MyFactory4(void*) { return new RadialGradientGM; }
 static GMRegistry reg4(MyFactory4);
 
 static GM* MyFactory5(void*) { return new GradientsLocalPerspectiveGM; }
 static GMRegistry reg5(MyFactory5);
 
 static GM* MyFactory6(void*) { return new GradientsViewPerspectiveGM; }
 static GMRegistry reg6(MyFactory6);
 
-static GM* MyFactory7(void*) { return new RadialGradient2GM; }
+static GM* MyFactory7(void*) { return new RadialGradient2GM(true); }

[reed1, 2013/09/11 14:33:27]

nitties:

1. If we're going to have 2 gms, might be slightly clearer if we just pass the
"flags" for the gradient as a param, rather than a bool which we happen to know
translates into diff flags.

2. Can we instead change the curr GM to just draw twice on screen, so we can see
at a glance the two different effects side-by-side?

questions, not demands

 static GMRegistry reg7(MyFactory7);
+
+static GM* MyFactory8(void*) { return new RadialGradient2GM(false); }
+static GMRegistry reg8(MyFactory8);
 }