setDrawFilter needs to trigger any deferred saves

BUG=skia:
TBR=

Review URL: https://codereview.chromium.org/1077353002

1 files changed, 44 insertions, 0 deletions

diff --git a/tests/DrawFilterTest.cpp b/tests/DrawFilterTest.cpp
new file mode 100644
index 0000000000..d01fa07338
--- /dev/null
+++ b/tests/DrawFilterTest.cpp
@@ -0,0 +1,44 @@
+/*
+ * Copyright 2015 Google Inc.
+ *
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ */
+
+#include "SkCanvas.h"
+#include "SkDrawFilter.h"
+#include "SkSurface.h"
+#include "Test.h"
+
+class TestFilter : public SkDrawFilter {
+public:
+    bool filter(SkPaint* p, Type) override {
+        return true;
+    }
+};
+
+/**
+ *  canvas.setDrawFilter is defined to be local to the save/restore block, such that if you
+ *  do the following: save / modify-drawfilter / restore, the current drawfilter should be what
+ *  it was before the save.
+ */
+static void test_saverestore(skiatest::Reporter* reporter) {
+    SkAutoTUnref<SkSurface> surface(SkSurface::NewRasterN32Premul(10, 10));
+    SkCanvas* canvas = surface->getCanvas();
+
+
+    SkAutoTUnref<TestFilter> df(SkNEW(TestFilter));
+
+    REPORTER_ASSERT(reporter, NULL == canvas->getDrawFilter());
+
+    canvas->save();
+    canvas->setDrawFilter(df);
+    REPORTER_ASSERT(reporter, NULL != canvas->getDrawFilter());
+    canvas->restore();
+
+    REPORTER_ASSERT(reporter, NULL == canvas->getDrawFilter());
+}
+
+DEF_TEST(DrawFilter, reporter) {
+    test_saverestore(reporter);
+}