Add asRRect method to SkClipStack::Element

src/core/SkClipStack.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 "SkAtomics.h"
 #include "SkCanvas.h"
 #include "SkClipStack.h"
 #include "SkPath.h"
 #include "SkPathOps.h"
 
 #include <new>
 
 
 // 0-2 are reserved for invalid, empty & wide-open
 static const int32_t kFirstUnreservedGenID = 3;
 int32_t SkClipStack::gGenID = kFirstUnreservedGenID;
 
 SkClipStack::Element::Element(const Element& that) {
     switch (that.getType()) {
         case kEmpty_Type:
+            fRRect.setEmpty();

[bsalomon, 2016/03/17 13:07:12]

Do we really care about doing this? The reason the path is reset is because it
can hold an arbitrarily large amount of memory. I don't think it's a big deal
either way. Could also just do this in debug.

Otherwise, lgtm.

             fPath.reset();
             break;
         case kRect_Type: // Rect uses rrect
         case kRRect_Type:
             fPath.reset();
             fRRect = that.fRRect;
             break;
         case kPath_Type:
             fPath.set(that.getPath());
             break;
@@ skipping 130 matching lines @@
     fPath.reset();
     fGenID = kEmptyGenID;
     SkDEBUGCODE(this->checkEmpty();)
 }
 
 void SkClipStack::Element::checkEmpty() const {
     SkASSERT(fFiniteBound.isEmpty());
     SkASSERT(kNormal_BoundsType == fFiniteBoundType);
     SkASSERT(!fIsIntersectionOfRects);
     SkASSERT(kEmptyGenID == fGenID);
+    SkASSERT(fRRect.isEmpty());
     SkASSERT(!fPath.isValid());
 }
 
 bool SkClipStack::Element::canBeIntersectedInPlace(int saveCount, SkRegion::Op op) const {
     if (kEmpty_Type == fType &&
         (SkRegion::kDifference_Op == op || SkRegion::kIntersect_Op == op)) {
         return true;
     }
     // Only clips within the same save/restore frame (as captured by
     // the save count) can be merged
@@ skipping 725 matching lines @@
 
 void SkClipStack::dump() const {
     B2TIter iter(*this);
     const Element* e;
     while ((e = iter.next())) {
         e->dump();
         SkDebugf("\n");
     }
 }
 #endif