reverting r3257 because of gm failure.
git-svn-id: http://skia.googlecode.com/svn/trunk@3258 2bbb7eff-a529-9590-31e7-b0007b416f81
diff --git a/src/core/SkCanvas.cpp b/src/core/SkCanvas.cpp
index 78751de..6e061c2 100644
--- a/src/core/SkCanvas.cpp
+++ b/src/core/SkCanvas.cpp
@@ -707,12 +707,19 @@
return (flags & SkCanvas::kClipToLayer_SaveFlag) != 0;
}
-bool SkCanvas::clipRectBounds(const SkRect* bounds, SaveFlags flags,
- SkIRect* intersection) {
+int SkCanvas::saveLayer(const SkRect* bounds, const SkPaint* paint,
+ SaveFlags flags) {
+ // do this before we create the layer. We don't call the public save() since
+ // that would invoke a possibly overridden virtual
+ int count = this->internalSave(flags);
+
+ fDeviceCMDirty = true;
+
SkIRect clipBounds;
if (!this->getClipDeviceBounds(&clipBounds)) {
- return false;
+ return count;
}
+
SkIRect ir;
if (NULL != bounds) {
SkRect r;
@@ -724,36 +731,16 @@
if (bounds_affects_clip(flags)) {
fMCRec->fRasterClip->setEmpty();
}
- return false;
+ return count;
}
} else { // no user bounds, so just use the clip
ir = clipBounds;
}
fClipStack.clipDevRect(ir, SkRegion::kIntersect_Op);
-
// early exit if the clip is now empty
if (bounds_affects_clip(flags) &&
!fMCRec->fRasterClip->op(ir, SkRegion::kIntersect_Op)) {
- return false;
- }
-
- if (intersection) {
- *intersection = ir;
- }
- return true;
-}
-
-int SkCanvas::saveLayer(const SkRect* bounds, const SkPaint* paint,
- SaveFlags flags) {
- // do this before we create the layer. We don't call the public save() since
- // that would invoke a possibly overridden virtual
- int count = this->internalSave(flags);
-
- fDeviceCMDirty = true;
-
- SkIRect ir;
- if (!this->clipRectBounds(bounds, flags, &ir)) {
return count;
}
diff --git a/src/core/SkPictureRecord.cpp b/src/core/SkPictureRecord.cpp
index ba49b72..c56c104 100644
--- a/src/core/SkPictureRecord.cpp
+++ b/src/core/SkPictureRecord.cpp
@@ -62,9 +62,7 @@
clip starts out the size of the picture, which is often much larger
than the size of the actual device we'll use during playback).
*/
- int count = this->INHERITED::save(flags);
- this->clipRectBounds(bounds, flags, NULL);
- return count;
+ return this->INHERITED::save(flags);
}
bool SkPictureRecord::isDrawingToLayer() const {
diff --git a/src/utils/SkDeferredCanvas.cpp b/src/utils/SkDeferredCanvas.cpp
index 1254392..00a26b9 100644
--- a/src/utils/SkDeferredCanvas.cpp
+++ b/src/utils/SkDeferredCanvas.cpp
@@ -101,6 +101,7 @@
void SkDeferredCanvas::validate() const
{
SkASSERT(getDevice());
+ SkASSERT(INHERITED::getTotalMatrix().isIdentity());
}
SkCanvas* SkDeferredCanvas::drawingCanvas() const
@@ -215,87 +216,78 @@
int SkDeferredCanvas::save(SaveFlags flags)
{
- drawingCanvas()->save(flags);
- return this->INHERITED::save(flags);
+ return drawingCanvas()->save(flags);
}
int SkDeferredCanvas::saveLayer(const SkRect* bounds, const SkPaint* paint,
SaveFlags flags)
{
- drawingCanvas()->saveLayer(bounds, paint, flags);
- int count = this->INHERITED::save(flags);
- this->clipRectBounds(bounds, flags, NULL);
- return count;
+ return drawingCanvas()->saveLayer(bounds, paint, flags);
}
void SkDeferredCanvas::restore()
{
drawingCanvas()->restore();
- this->INHERITED::restore();
}
-bool SkDeferredCanvas::isDrawingToLayer() const
+int SkDeferredCanvas::getSaveCount() const
{
- return drawingCanvas()->isDrawingToLayer();
+ return drawingCanvas()->getSaveCount();
}
bool SkDeferredCanvas::translate(SkScalar dx, SkScalar dy)
{
- drawingCanvas()->translate(dx, dy);
- return this->INHERITED::translate(dx, dy);
+ return drawingCanvas()->translate(dx, dy);
}
bool SkDeferredCanvas::scale(SkScalar sx, SkScalar sy)
{
- drawingCanvas()->scale(sx, sy);
- return this->INHERITED::scale(sx, sy);
+ return drawingCanvas()->scale(sx, sy);
}
bool SkDeferredCanvas::rotate(SkScalar degrees)
{
- drawingCanvas()->rotate(degrees);
- return this->INHERITED::rotate(degrees);
+ return drawingCanvas()->rotate(degrees);
}
bool SkDeferredCanvas::skew(SkScalar sx, SkScalar sy)
{
- drawingCanvas()->skew(sx, sy);
- return this->INHERITED::skew(sx, sy);
+ return drawingCanvas()->skew(sx, sy);
}
bool SkDeferredCanvas::concat(const SkMatrix& matrix)
{
- drawingCanvas()->concat(matrix);
- return this->INHERITED::concat(matrix);
+ return drawingCanvas()->concat(matrix);
}
void SkDeferredCanvas::setMatrix(const SkMatrix& matrix)
{
drawingCanvas()->setMatrix(matrix);
- this->INHERITED::setMatrix(matrix);
+}
+
+const SkMatrix& SkDeferredCanvas::getTotalMatrix() const
+{
+ return drawingCanvas()->getTotalMatrix();
}
bool SkDeferredCanvas::clipRect(const SkRect& rect,
SkRegion::Op op,
bool doAntiAlias)
{
- drawingCanvas()->clipRect(rect, op, doAntiAlias);
- return this->INHERITED::clipRect(rect, op, doAntiAlias);
+ return drawingCanvas()->clipRect(rect, op, doAntiAlias);
}
bool SkDeferredCanvas::clipPath(const SkPath& path,
SkRegion::Op op,
bool doAntiAlias)
{
- drawingCanvas()->clipPath(path, op, doAntiAlias);
- return this->INHERITED::clipPath(path, op, doAntiAlias);
+ return drawingCanvas()->clipPath(path, op, doAntiAlias);
}
bool SkDeferredCanvas::clipRegion(const SkRegion& deviceRgn,
SkRegion::Op op)
{
- drawingCanvas()->clipRegion(deviceRgn, op);
- return this->INHERITED::clipRegion(deviceRgn, op);
+ return drawingCanvas()->clipRegion(deviceRgn, op);
}
void SkDeferredCanvas::clear(SkColor color)
@@ -462,14 +454,14 @@
SkBounder* SkDeferredCanvas::setBounder(SkBounder* bounder)
{
- drawingCanvas()->setBounder(bounder);
- return INHERITED::setBounder(bounder);
+ INHERITED::setBounder(bounder); // So non-virtual getBounder works
+ return drawingCanvas()->setBounder(bounder);
}
SkDrawFilter* SkDeferredCanvas::setDrawFilter(SkDrawFilter* filter)
{
- drawingCanvas()->setDrawFilter(filter);
- return INHERITED::setDrawFilter(filter); // So non-virtual getDrawFilter works
+ INHERITED::setDrawFilter(filter); // So non-virtual getDrawFilter works
+ return drawingCanvas()->setDrawFilter(filter);
}
SkCanvas* SkDeferredCanvas::canvasForDrawIter() {
@@ -490,7 +482,8 @@
fImmediateDevice = immediateDevice; // ref counted via fImmediateCanvas
fImmediateCanvas = SkNEW_ARGS(SkCanvas, (fImmediateDevice));
fRecordingCanvas = fPicture.beginRecording(fImmediateDevice->width(),
- fImmediateDevice->height(), 0);
+ fImmediateDevice->height(),
+ SkPicture::kUsePathBoundsForClip_RecordingFlag);
}
SkDeferredCanvas::DeferredDevice::~DeferredDevice()
@@ -525,7 +518,8 @@
// old one, hence purging deferred draw ops.
fRecordingCanvas = fPicture.beginRecording(
fImmediateDevice->width(),
- fImmediateDevice->height(), 0);
+ fImmediateDevice->height(),
+ SkPicture::kUsePathBoundsForClip_RecordingFlag);
// Restore pre-purge state
if (!clipRegion.isEmpty()) {
@@ -554,7 +548,8 @@
}
fPicture.draw(fImmediateCanvas);
fRecordingCanvas = fPicture.beginRecording(fImmediateDevice->width(),
- fImmediateDevice->height(), 0);
+ fImmediateDevice->height(),
+ SkPicture::kUsePathBoundsForClip_RecordingFlag);
}
void SkDeferredCanvas::DeferredDevice::flush()