Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(270)

Unified Diff: tests/PictureTest.cpp

Issue 22987003: Fix clip expansion in SkPictureRecord::recordRestoreOffsetPlaceholder() (Closed) Base URL: https://skia.googlecode.com/svn/trunk
Patch Set: Updated per comments. Created 7 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/core/SkPictureRecord.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/PictureTest.cpp
diff --git a/tests/PictureTest.cpp b/tests/PictureTest.cpp
index 1827fdcb5a28ceb0a33541825c58a0fe57914790..f823806d0dfbcf1a442069fd4b834cf9b8adbbbf 100644
--- a/tests/PictureTest.cpp
+++ b/tests/PictureTest.cpp
@@ -8,6 +8,7 @@
#include "SkCanvas.h"
#include "SkColorPriv.h"
#include "SkData.h"
+#include "SkDevice.h"
#include "SkError.h"
#include "SkPaint.h"
#include "SkPicture.h"
@@ -550,6 +551,62 @@ static void test_clip_bound_opt(skiatest::Reporter* reporter) {
}
}
+/**
+ * A canvas that records the number of clip commands.
+ */
+class ClipCountingCanvas : public SkCanvas {
+public:
+ explicit ClipCountingCanvas(SkDevice* device)
+ : SkCanvas(device)
+ , fClipCount(0){
+ }
+
+ virtual bool clipRect(const SkRect& r, SkRegion::Op op, bool doAA)
+ SK_OVERRIDE {
+ fClipCount += 1;
+ return this->INHERITED::clipRect(r, op, doAA);
+ }
+
+ virtual bool clipRRect(const SkRRect& rrect, SkRegion::Op op, bool doAA)
+ SK_OVERRIDE {
+ fClipCount += 1;
+ return this->INHERITED::clipRRect(rrect, op, doAA);
+ }
+
+ virtual bool clipPath(const SkPath& path, SkRegion::Op op, bool doAA)
+ SK_OVERRIDE {
+ fClipCount += 1;
+ return this->INHERITED::clipPath(path, op, doAA);
+ }
+
+ unsigned getClipCount() const { return fClipCount; }
+
+private:
+ unsigned fClipCount;
+
+ typedef SkCanvas INHERITED;
+};
+
+static void test_clip_expansion(skiatest::Reporter* reporter) {
+ SkPicture picture;
+ SkCanvas* canvas = picture.beginRecording(10, 10, 0);
+
+ canvas->clipRect(SkRect::MakeEmpty(), SkRegion::kReplace_Op);
+ // The following expanding clip should not be skipped.
+ canvas->clipRect(SkRect::MakeXYWH(4, 4, 3, 3), SkRegion::kUnion_Op);
+ // Draw something so the optimizer doesn't just fold the world.
+ SkPaint p;
+ p.setColor(SK_ColorBLUE);
+ canvas->drawPaint(p);
+
+ SkDevice testDevice(SkBitmap::kNo_Config, 10, 10);
+ ClipCountingCanvas testCanvas(&testDevice);
+ picture.draw(&testCanvas);
+
+ // Both clips should be present on playback.
+ REPORTER_ASSERT(reporter, testCanvas.getClipCount() == 2);
+}
+
static void TestPicture(skiatest::Reporter* reporter) {
#ifdef SK_DEBUG
test_deleting_empty_playback();
@@ -562,6 +619,7 @@ static void TestPicture(skiatest::Reporter* reporter) {
test_bitmap_with_encoded_data(reporter);
test_clone_empty(reporter);
test_clip_bound_opt(reporter);
+ test_clip_expansion(reporter);
}
#include "TestClassDef.h"
« no previous file with comments | « src/core/SkPictureRecord.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698