extend SkBenchmark to allow a bench to return a durationScale, which allows it to perform fewer act…

extend SkBenchmark to allow a bench to return a durationScale, which allows it to perform fewer actual interations, but report a scale factor to account for that. Thus a very slow bench can be compared head-to-head with a faster one, w/o actually forcing the tool to run for the full duration of the slower test.

Extend BitmapBench to time bicubic filtering, and use this durationScale for it.

Extend SkBenchmark to have setBitmapFlags(or, clear), allowing it to request a set of paint flags to be cleared, and set to be set.

BUG=
R=robertphillips@google.com

Committed: https://code.google.com/p/skia/source/detail?r=9315

[robertphillips, 2013-05-29 13:14:17]

lgtm + some questions & suggestions

https://codereview.chromium.org/16069010/diff/2001/bench/BitmapBench.cpp
File bench/BitmapBench.cpp (right):

https://codereview.chromium.org/16069010/diff/2001/bench/BitmapBench.cpp#newc...
bench/BitmapBench.cpp:149: int count = N;
make sure count remains 1 in debug?

https://codereview.chromium.org/16069010/diff/2001/bench/BitmapBench.cpp#newc...
bench/BitmapBench.cpp:167: 
return 1 in debug?

https://codereview.chromium.org/16069010/diff/2001/bench/SkBenchmark.h
File bench/SkBenchmark.h (right):

https://codereview.chromium.org/16069010/diff/2001/bench/SkBenchmark.h#newcod...
bench/SkBenchmark.h:109: /**
// Sets the bit manipulation masks so in ::setupPaint the following operation
will be performed on the paint's flags.

Also: setPaintFlags -> setPaintBitMasks, setPaintMasks?

[reed1, 2013-05-29 15:02:30]

https://codereview.chromium.org/16069010/diff/2001/bench/BitmapBench.cpp
File bench/BitmapBench.cpp (right):

https://codereview.chromium.org/16069010/diff/2001/bench/BitmapBench.cpp#newc...
bench/BitmapBench.cpp:149: int count = N;
On 2013/05/29 13:14:17, robertphillips wrote:
> make sure count remains 1 in debug?

Done.

https://codereview.chromium.org/16069010/diff/2001/bench/BitmapBench.cpp#newc...
bench/BitmapBench.cpp:167: 
On 2013/05/29 13:14:17, robertphillips wrote:
> return 1 in debug?

Done.

https://codereview.chromium.org/16069010/diff/2001/bench/SkBenchmark.h
File bench/SkBenchmark.h (right):

https://codereview.chromium.org/16069010/diff/2001/bench/SkBenchmark.h#newcod...
bench/SkBenchmark.h:109: /**
On 2013/05/29 13:14:17, robertphillips wrote:
> // Sets the bit manipulation masks so in ::setupPaint the following operation
> will be performed on the paint's flags.
> 
> Also: setPaintFlags -> setPaintBitMasks, setPaintMasks?

Done.

[reed1, 2013-05-29 15:40:01]

Committed patchset #3 manually as r9315 (presubmit successful).