Make ClipStack honor save flags

src/core/SkCanvas.cpp

 
 /*
  * Copyright 2008 The Android Open Source Project
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 
 #include "SkCanvas.h"
@@ skipping 196 matching lines @@
 /*  This is the record we keep for each save/restore level in the stack.
     Since a level optionally copies the matrix and/or stack, we have pointers
     for these fields. If the value is copied for this level, the copy is
     stored in the ...Storage field, and the pointer points to that. If the
     value is not copied for this level, we ignore ...Storage, and just point
     at the corresponding value in the previous level in the stack.
 */
 class SkCanvas::MCRec {
 public:
     MCRec*          fNext;
+    int             fFlags;
     SkMatrix*       fMatrix;        // points to either fMatrixStorage or prev MCRec
     SkRasterClip*   fRasterClip;    // points to either fRegionStorage or prev MCRec
     SkDrawFilter*   fFilter;        // the current filter (or null)
 
     DeviceCM*   fLayer;
     /*  If there are any layers in the stack, this points to the top-most
         one that is at or below this level in the stack (so we know what
         bitmap/device to draw into from this level. This value is NOT
         reference counted, since the real owner is either our fLayer field,
         or a previous one in a lower level.)
     */
     DeviceCM*   fTopLayer;
 
-    MCRec(const MCRec* prev, int flags) {
+    MCRec(const MCRec* prev, int flags) : fFlags(flags) {
         if (NULL != prev) {
             if (flags & SkCanvas::kMatrix_SaveFlag) {
                 fMatrixStorage = *prev->fMatrix;
                 fMatrix = &fMatrixStorage;
             } else {
                 fMatrix = prev->fMatrix;
             }
 
             if (flags & SkCanvas::kClip_SaveFlag) {
                 fRasterClipStorage = *prev->fRasterClip;
@@ skipping 472 matching lines @@
 
 int SkCanvas::internalSave(SaveFlags flags) {
     int saveCount = this->getSaveCount(); // record this before the actual save
 
     MCRec* newTop = (MCRec*)fMCStack.push_back();
     new (newTop) MCRec(fMCRec, flags);    // balanced in restore()
 
     newTop->fNext = fMCRec;
     fMCRec = newTop;
 
-    fClipStack.save();
-    SkASSERT(fClipStack.getSaveCount() == this->getSaveCount() - 1);
+    if (SkCanvas::kClip_SaveFlag & flags) {
+        fClipStack.save();
+    }
 
     return saveCount;
 }
 
 int SkCanvas::save(SaveFlags flags) {
     // call shared impl
     return this->internalSave(flags);
 }
 
 #define C32MASK (1 << SkBitmap::kARGB_8888_Config)
@@ skipping 154 matching lines @@
         this->internalRestore();
     }
 }
 
 void SkCanvas::internalRestore() {
     SkASSERT(fMCStack.count() != 0);
 
     fDeviceCMDirty = true;
     fLocalBoundsCompareTypeDirty = true;
 
-    fClipStack.restore();
+    if (SkCanvas::kClip_SaveFlag & fMCRec->fFlags) {
+        fClipStack.restore();
+    }
+
     // reserve our layer (if any)
     DeviceCM* layer = fMCRec->fLayer;   // may be null
     // now detach it from fMCRec so we can pop(). Gets freed after its drawn
     fMCRec->fLayer = NULL;
 
     // now do the normal restore()
     fMCRec->~MCRec();       // balanced in save()
     fMCStack.pop_back();
     fMCRec = (MCRec*)fMCStack.back();
 
@@ skipping 1303 matching lines @@
     return *paint;
 }
 
 const SkRegion& SkCanvas::LayerIter::clip() const { return fImpl->getClip(); }
 int SkCanvas::LayerIter::x() const { return fImpl->getX(); }
 int SkCanvas::LayerIter::y() const { return fImpl->getY(); }
 
 ///////////////////////////////////////////////////////////////////////////////
 
 SkCanvas::ClipVisitor::~ClipVisitor() { }