Turn off per-iteration tile time reporting

Turn off per-iteration tile time reporting

It's overloading the bench graph generator.
Committed: https://code.google.com/p/skia/source/detail?r=7691

[robertphillips, 2013-02-11 16:25:53]

LGTM + request for expanded comment

https://codereview.appspot.com/7300088/diff/2001/tools/PictureBenchmark.cpp
File tools/PictureBenchmark.cpp (right):

https://codereview.appspot.com/7300088/diff/2001/tools/PictureBenchmark.cpp#n...
tools/PictureBenchmark.cpp:135: 
LGTM - but maybe expand on the comment like:
Printing out every run for each tile expands the data file by 20x causing the XX
step to YYY. We can reenable this output once the graph dashboard retrieves its
data directly from ZZZ and we have moved away from generating the SVG graphs.

[Leon, 2013-02-11 16:28:06]

Is longRunningTimer working as desired on 10.6?

Also, if we use longRunningTimer, it should be created with setupTimer(true)
(not shown in your version, but it is currently created with setupTimer(false)).

Should we go ahead and delete the timer we're not using? Or might we reenable
it?

[robertphillips, 2013-02-11 16:31:42]

Leon is right we need to flip which timer gets the gpu timer.

[EricB, 2013-02-11 17:59:02]

Patch set 3: set longRunningTimer to use GL timer.

https://codereview.appspot.com/7300088/diff/2001/tools/PictureBenchmark.cpp
File tools/PictureBenchmark.cpp (right):

https://codereview.appspot.com/7300088/diff/2001/tools/PictureBenchmark.cpp#n...
tools/PictureBenchmark.cpp:135: 
On 2013/02/11 16:25:53, robertphillips wrote:
> LGTM - but maybe expand on the comment like:
> Printing out every run for each tile expands the data file by 20x causing the
XX
> step to YYY. We can reenable this output once the graph dashboard retrieves
its
> data directly from ZZZ and we have moved away from generating the SVG graphs.

Done.

[Leon, 2013-02-11 18:14:26]

lgtm