tree 8c02cb6b815f0d52340960bec4400404237c879b
parent 7c6bc4616cee740b197e4e8f2b704b3f55fd8e6d
author Mike Klein <mtklein@chromium.org> 1483644237 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> 1483646426 +0000

monobench: don't sort the Bench vector while iterating through it...

We sort to display Bench results in an ascending order, but we're doing this while iterating through the bench vector.  This is probably undefined, and really screws up the sample distribution in practice.  Slow benches run more often.

Instead, copy the results of the benches to a new Result vector to sort and display.  Each bench now gets its fair share of samples.

Change-Id: I4ead0d9d69af271c9971eedb35604a4b3bca0784
Reviewed-on: https://skia-review.googlesource.com/6623
Reviewed-by: Mike Klein <mtklein@chromium.org>
Reviewed-by: Herb Derby <herb@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>