Add rect clip to convex_poly_clip GM

gm/convexpolyclip.cpp

 
 /*
  * Copyright 2014 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"
 
@@ skipping 65 matching lines @@
     ConvexPolyClip() {
         this->setBGColor(0xFFFFFFFF);
     }
 
 protected:
     virtual SkString onShortName() SK_OVERRIDE {
         return SkString("convex_poly_clip");
     }
 
     virtual SkISize onISize() SK_OVERRIDE {
-        return make_isize(435, 440);
+        return make_isize(435, 540);
     }
 
     virtual void onOnceBeforeDraw() SK_OVERRIDE {
         SkPath tri;
         tri.moveTo(5.f, 5.f);
         tri.lineTo(100.f, 20.f);
         tri.lineTo(15.f, 100.f);
 
-        fPaths.addToTail(tri);
+        fClips.addToTail()->setPath(tri);
 
         SkPath hexagon;
         static const SkScalar kRadius = 45.f;
         const SkPoint center = { kRadius, kRadius };
         for (int i = 0; i < 6; ++i) {
             SkScalar angle = 2 * SK_ScalarPI * i / 6;
             SkPoint point;
             point.fY = SkScalarSinCos(angle, &point.fX);
             point.scale(kRadius);
             point = center + point;
             if (0 == i) {
                 hexagon.moveTo(point);
             } else {
                 hexagon.lineTo(point);
             }
         }
-        fPaths.addToTail(hexagon);
+        fClips.addToTail()->setPath(hexagon);
 
         SkMatrix scaleM;
         scaleM.setScale(1.1f, 0.4f, kRadius, kRadius);
         hexagon.transform(scaleM);
-        fPaths.addToTail(hexagon);
+        fClips.addToTail()->setPath(hexagon);
+
+        fClips.addToTail()->setRect(SkRect::MakeXYWH(8.3f, 11.6f, 78.2f, 72.6f));
 
         SkPath rotRect;
         SkRect rect = SkRect::MakeLTRB(10.f, 12.f, 80.f, 86.f);
         rotRect.addRect(rect);
         SkMatrix rotM;
         rotM.setRotate(23.f, rect.centerX(), rect.centerY());
         rotRect.transform(rotM);
-        fPaths.addToTail(rotRect);
-
+        fClips.addToTail()->setPath(rotRect);
+        
         fBmp = make_bmp(100, 100);
     }
 
     virtual void onDraw(SkCanvas* canvas) SK_OVERRIDE {
-        const SkPath* path;
         SkScalar y = 0;
         static const SkScalar kMargin = 10.f;
 
         SkPaint bgPaint;
         bgPaint.setAlpha(0x15);
         SkISize size = canvas->getDeviceSize();
         SkRect dstRect = SkRect::MakeWH(SkIntToScalar(size.fWidth),
                                         SkIntToScalar(size.fHeight));
         canvas->drawBitmapRectToRect(fBmp, NULL, dstRect, &bgPaint);
 
-        for (SkTLList<SkPath>::Iter iter(fPaths, SkTLList<SkPath>::Iter::kHead_IterStart);
-             NULL != (path = iter.get());
+        for (SkTLList<Clip>::Iter iter(fClips, SkTLList<Clip>::Iter::kHead_IterStart);
+             NULL != iter.get();
              iter.next()) {
+            const Clip* clip = iter.get();
             SkScalar x = 0;
             for (int aa = 0; aa < 2; ++aa) {
                 canvas->save();
                 canvas->translate(x, y);
-                canvas->clipPath(*path, SkRegion::kIntersect_Op, SkToBool(aa));
+                clip->setOnCanvas(canvas, SkRegion::kIntersect_Op, SkToBool(aa));
                 canvas->drawBitmap(fBmp, 0, 0);
                 canvas->restore();
                 x += fBmp.width() + kMargin;
             }
             for (int aa = 0; aa < 2; ++aa) {
                 static const char kTxt[] = "Clip Me!";
                 SkPaint txtPaint;
                 txtPaint.setTextSize(23.f);
                 txtPaint.setAntiAlias(true);
                 txtPaint.setColor(SK_ColorDKGRAY);
 
                 SkPaint clipOutlinePaint;
                 clipOutlinePaint.setAntiAlias(true);
                 clipOutlinePaint.setColor(0x50505050);
                 clipOutlinePaint.setStyle(SkPaint::kStroke_Style);
                 clipOutlinePaint.setStrokeWidth(0);
 
                 canvas->save();
                 canvas->translate(x, y);
-                SkPath closedClipPath = *path;
-                closedClipPath.close();
+                SkPath closedClipPath;
+                clip->asClosedPath(&closedClipPath);
                 canvas->drawPath(closedClipPath, clipOutlinePaint);
-                canvas->clipPath(*path, SkRegion::kIntersect_Op, SkToBool(aa));
+                clip->setOnCanvas(canvas, SkRegion::kIntersect_Op, SkToBool(aa));
                 canvas->scale(1.f, 1.8f);
                 canvas->drawText(kTxt, SK_ARRAY_COUNT(kTxt)-1,
                                  0, 1.5f * txtPaint.getTextSize(),
                                  txtPaint);
                 canvas->restore();
                 x += fBmp.width() + kMargin;
             }
 
             y += fBmp.height() + kMargin;
         }
     }
 
     virtual uint32_t onGetFlags() const {
         return kAsBench_Flag;
     }
 
 private:
-    SkTLList<SkPath> fPaths;
+    class Clip {
+    public:
+        enum ClipType {
+            kNone_ClipType,
+            kPath_ClipType,
+            kRect_ClipType
+        };
+
+        Clip () : fClipType(kNone_ClipType) {}
+        
+        void setOnCanvas(SkCanvas* canvas, SkRegion::Op op, bool aa) const {
+            switch (fClipType) {
+                case kPath_ClipType:
+                    canvas->clipPath(fPath, op, aa);
+                    break;
+                case kRect_ClipType:
+                    canvas->clipRect(fRect, op, aa);
+                    break;
+                case kNone_ClipType:
+                    SkDEBUGFAIL("Uninitialized Clip.");
+                    break;
+            }
+        }
+        
+        void asClosedPath(SkPath* path) const {
+            switch (fClipType) {
+                case kPath_ClipType:
+                    *path = fPath;
+                    path->close();
+                    break;
+                case kRect_ClipType:
+                    path->reset();
+                    path->addRect(fRect);
+                    break;
+                case kNone_ClipType:
+                    SkDEBUGFAIL("Uninitialized Clip.");
+                    break;
+            }
+        }
+        
+        void setPath(const SkPath& path) {
+            fClipType = kPath_ClipType;
+            fPath = path;
+        }
+        
+        void setRect(const SkRect& rect) {
+            fClipType = kRect_ClipType;
+            fRect = rect;

[robertphillips, 2014/02/03 22:32:45]

if (kPath_ClipType == fClipType) {
    fPath.reset();
}

moved up to start of method?

[bsalomon, 2014/02/04 14:24:34]

The path object is always valid (no union).

+            fPath.reset();
+        }
+        
+        ClipType getType() const { return fClipType; }
+        
+    private:
+        ClipType fClipType;

[robertphillips, 2014/02/03 22:32:45]

union?

[bsalomon, 2014/02/04 14:24:34]

SkPath can't be in a union, it'd have to be SkPath*. I think it'd just add
complexity.

+        SkPath fPath;
+        SkRect fRect;
+    };
+    
+    SkTLList<Clip>   fClips;
     SkBitmap         fBmp;
 
     typedef GM INHERITED;
 };
 
 DEF_GM( return SkNEW(ConvexPolyClip); )
 
 }