move intensive computing/rendering from bench constructor to onPreDraw.

bench/BitmapRectBench.cpp

 
 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 #include "SkBenchmark.h"
 #include "SkBitmap.h"
 #include "SkPaint.h"
@@ skipping 26 matching lines @@
     dst : same size as src, diff size
     paint : filter-p
  */
 
 class BitmapRectBench : public SkBenchmark {
     SkBitmap    fBitmap;
     bool        fDoFilter;
     bool        fSlightMatrix;
     uint8_t     fAlpha;
     SkString    fName;
     SkRect      fSrcR, fDstR;

[robertphillips, 2013/07/29 11:57:10]

kWidth, kHeight

+    static const int w = 128;
+    static const int h = 128;
     enum { N = SkBENCHLOOP(300) };
 public:
     BitmapRectBench(void* param, U8CPU alpha, bool doFilter, bool slightMatrix) : INHERITED(param) {
         fAlpha = SkToU8(alpha);
         fDoFilter = doFilter;
         fSlightMatrix = slightMatrix;
 
-        const int w = 128;
-        const int h = 128;
+        fBitmap.setConfig(SkBitmap::kARGB_8888_Config, w, h);
+    }
 
-        fBitmap.setConfig(SkBitmap::kARGB_8888_Config, w, h);
+protected:

[robertphillips, 2013/07/29 11:57:10]

SK_OVERRIDE

+    virtual const char* onGetName() {
+        fName.printf("bitmaprect_%02X_%sfilter_%s",
+                     fAlpha, fDoFilter ? "" : "no",
+                     fSlightMatrix ? "trans" : "identity");
+        return fName.c_str();
+    }
+

[robertphillips, 2013/07/29 11:57:10]

SK_OVERRIDE

+    virtual void onPreDraw() {
         fBitmap.allocPixels();
         fBitmap.setIsOpaque(true);
         fBitmap.eraseColor(SK_ColorBLACK);

[robertphillips, 2013/07/29 11:57:10]

rename to draw_into_bitmap

         drawIntoBitmap(fBitmap);
 
         fSrcR.iset(0, 0, w, h);
         fDstR.iset(0, 0, w, h);
 
-        if (slightMatrix) {
+        if (fSlightMatrix) {
             // want fractional translate
             fDstR.offset(SK_Scalar1 / 3, SK_Scalar1 * 5 / 7);
             // want enough to create a scale matrix, but not enough to scare
             // off our sniffer which tries to see if the matrix is "effectively"
             // translate-only.
             fDstR.fRight += SK_Scalar1 / (w * 60);
         }
     }
 
-protected:
-    virtual const char* onGetName() {
-        fName.printf("bitmaprect_%02X_%sfilter_%s",
-                     fAlpha, fDoFilter ? "" : "no",
-                     fSlightMatrix ? "trans" : "identity");
-        return fName.c_str();
-    }
 

[robertphillips, 2013/07/29 11:57:10]

SK_OVERRIDE

     virtual void onDraw(SkCanvas* canvas) {
         SkRandom rand;
 
         SkPaint paint;
         this->setupPaint(&paint);
         paint.setFilterBitmap(fDoFilter);
         paint.setAlpha(fAlpha);
 
         for (int i = 0; i < N; i++) {
             canvas->drawBitmapRectToRect(fBitmap, &fSrcR, fDstR, &paint);
         }
     }
 
 private:
     typedef SkBenchmark INHERITED;
 };
 
 DEF_BENCH(return new BitmapRectBench(p, 0xFF, false, false))
 DEF_BENCH(return new BitmapRectBench(p, 0x80, false, false))
 DEF_BENCH(return new BitmapRectBench(p, 0xFF, true, false))
 DEF_BENCH(return new BitmapRectBench(p, 0x80, true, false))
 
 DEF_BENCH(return new BitmapRectBench(p, 0xFF, false, true))
 DEF_BENCH(return new BitmapRectBench(p, 0xFF, true, true))