aboutsummaryrefslogtreecommitdiffhomepage
path: root/bench/ImageBench.cpp
diff options
context:
space:
mode:
authorGravatar reed <reed@google.com>2015-12-03 12:56:59 -0800
committerGravatar Commit bot <commit-bot@chromium.org>2015-12-03 12:56:59 -0800
commit6f2753deee7b8cd03224e3df91dfea21bbeb9f7f (patch)
tree8cf4844a31ce15a3952f4bf9d9d911713ffc53ee /bench/ImageBench.cpp
parentcb0d5c940a881ee38f0f95e240b31a9052a4e440 (diff)
add bench for sprite-like drawing w/ imagefilters
visualbench run on Mac Pro curr/maxrss loops min median mean max stddev samples config bench 96/96 MB 16 412µs 413µs 413µs 414µs 0% ▄▁█▅▅▅▅▆▅▃▅ gpu warmupbench 98/98 MB 4 1.86ms 1.86ms 1.86ms 1.86ms 0% ▁▂▇█▆▅▇▄▃▃▃ gpu image-filter-sprite-draw-image 100/100 MB 4 1.86ms 1.86ms 1.86ms 1.86ms 0% ▆▇▅▄▁▆▅█▅▄▅ gpu image-filter-sprite-draw-bitmap 100/100 MB 32 547µs 548µs 590µs 1.01ms 24% █▁▁▁▁▁▁▁▁▁▁ gpu image-filter-sprite-draw-sprite BUG=skia: Review URL: https://codereview.chromium.org/1497843003
Diffstat (limited to 'bench/ImageBench.cpp')
-rw-r--r--bench/ImageBench.cpp114
1 files changed, 113 insertions, 1 deletions
diff --git a/bench/ImageBench.cpp b/bench/ImageBench.cpp
index b81c57c6dc..0263758da5 100644
--- a/bench/ImageBench.cpp
+++ b/bench/ImageBench.cpp
@@ -61,6 +61,118 @@ private:
typedef Benchmark INHERITED;
};
+DEF_BENCH( return new Image2RasterBench; )
+///////////////////////////////////////////////////////////////////////////////////////////////////
+
+#include "SkOffsetImageFilter.h"
+
+#if SK_SUPPORT_GPU
+#include "SkGrPixelRef.h"
+#endif
+
+enum MyDrawType {
+ kSprite_Type,
+ kBitmap_Type,
+ kImage_Type,
+};
+
+/*
+ * Want to time drawing images/bitmaps via drawSprite, and via drawBitmap/drawImage but with
+ * a non-scaling matrix and a clip that is tight to the image bounds. In this scenario, we
+ * should be able to match the speed of drawSprite.
+ *
+ * An optimal result should be that all three types: sprite/bitmap/image draw at the same speed.
+ */
+class ImageFilterSpriteBench : public Benchmark {
+ SkAutoTUnref<SkImage> fImage;
+ SkBitmap fBitmap;
+ SkString fName;
+ MyDrawType fType;
+
+public:
+ ImageFilterSpriteBench(MyDrawType dt, const char suffix[]) : fType(dt) {
+ fName.printf("image-filter-sprite-draw-%s", suffix);
+ }
+
+ bool isSuitableFor(Backend backend) override {
+ return kGPU_Backend == backend || kRaster_Backend == backend;
+ }
+
+protected:
+ bool isVisual() override {
+ return true;
+ }
+
+ const char* onGetName() override {
+ return fName.c_str();
+ }
+
+ void onPerCanvasPreDraw(SkCanvas* canvas) override {
+ const SkImageInfo info = SkImageInfo::MakeN32Premul(500, 500);
+ SkAutoTUnref<SkSurface> surface(canvas->newSurface(info));
+
+ surface->getCanvas()->drawColor(SK_ColorRED);
+ fImage.reset(surface->newImageSnapshot());
+
+ fBitmap.setInfo(info);
+ if (fImage->getTexture()) {
+#if SK_SUPPORT_GPU
+ fBitmap.setPixelRef(new SkGrPixelRef(info, fImage->getTexture()))->unref();
+#endif
+ } else {
+ SkPixmap pmap;
+ if (!fImage->peekPixels(&pmap)) {
+ sk_throw();
+ }
+ fBitmap.installPixels(pmap.info(), pmap.writable_addr(), pmap.rowBytes());
+ }
+ }
+
+ void onPerCanvasPostDraw(SkCanvas*) override {
+ // Release the image and raster surface here to prevent out of order destruction
+ // between these and the gpu interface.
+ fImage.reset(nullptr);
+ fBitmap.reset();
+ }
+
+ void onDraw(int loops, SkCanvas* canvas) override {
+ // This clip is important; it allows the drawImage/drawBitmap code to fall into the
+ // fast (sprite) case, since the imagefilter's output should match.
+ //
+ // When we address skbug.com/4526 we should be able to remove the need for this clip.
+ //
+ canvas->clipRect(SkRect::MakeIWH(fImage->width(), fImage->height()));
+
+ const SkScalar kDelta = 10;
+ SkPaint paint;
+ for (int i = 0; i < loops; i++) {
+ for (int inner = 0; inner < 10; ++inner) {
+ // build the filter everytime, so we don't accidentally draw a cached version,
+ // since the point of this bench is to time the actual imagefilter
+ // handling/overhead.
+ SkAutoTUnref<SkImageFilter> filter(SkOffsetImageFilter::Create(kDelta, kDelta));
+ paint.setImageFilter(filter);
+
+ switch (fType) {
+ case kSprite_Type:
+ canvas->drawSprite(fBitmap, 0, 0, &paint);
+ break;
+ case kBitmap_Type:
+ canvas->drawBitmap(fBitmap, 0, 0, &paint);
+ break;
+ case kImage_Type:
+ canvas->drawImage(fImage, 0, 0, &paint);
+ break;
+ }
+ }
+ }
+ }
+
+private:
+ typedef Benchmark INHERITED;
+};
+DEF_BENCH( return new ImageFilterSpriteBench(kSprite_Type, "sprite"); )
+DEF_BENCH( return new ImageFilterSpriteBench(kBitmap_Type, "bitmap"); )
+DEF_BENCH( return new ImageFilterSpriteBench(kImage_Type, "image"); )
-DEF_BENCH( return new Image2RasterBench; )