Adding flag to switch bench_record into tilegrid mode

Adding flag to switch bench_record into tilegrid mode

BUG=skia:

Committed: http://code.google.com/p/skia/source/detail?r=13628

[iancottrell, 2014-02-28 16:35:13]

running bench_record with --tile_grid_size 512 after this patch is much more
realistic of what chrome does, and doubles the measured time.

[Sami, 2014-02-28 17:02:35]

https://codereview.chromium.org/184543003/diff/1/tools/bench_record.cpp
File tools/bench_record.cpp (right):

https://codereview.chromium.org/184543003/diff/1/tools/bench_record.cpp#newco...
tools/bench_record.cpp:29: DEFINE_int32(tile_grid_size, 0, "Set the tile grid
size, if non zero switches to tile grid picture and enables
kOptimizeForClippedPlayback_RecordingFlag.");
Can we make this 512/256/whatever by default to match what Chrome is doing?

[mtklein, 2014-02-28 17:27:38]

Other than the name nits, looks fine to me.  I have no opinion on whether this
is one flag or two; whatever you guys find most helpful is good by me.

https://codereview.chromium.org/184543003/diff/1/tools/bench_record.cpp
File tools/bench_record.cpp (right):

https://codereview.chromium.org/184543003/diff/1/tools/bench_record.cpp#newco...
tools/bench_record.cpp:29: DEFINE_int32(tile_grid_size, 0, "Set the tile grid
size, if non zero switches to tile grid picture and enables
kOptimizeForClippedPlayback_RecordingFlag.");
tileGridSize in Skia.

https://codereview.chromium.org/184543003/diff/1/tools/bench_record.cpp#newco...
tools/bench_record.cpp:38: int recording_flags = FLAGS_flags;
recordingFlags

[iancottrell, 2014-02-28 18:50:46]

Incidentally, running bench record with all the bbox types...

Nothing : 21.84
R Tree : 34.28
Tile grid : 36.66
Quad tree : 47.14

https://codereview.chromium.org/184543003/diff/1/tools/bench_record.cpp
File tools/bench_record.cpp (right):

https://codereview.chromium.org/184543003/diff/1/tools/bench_record.cpp#newco...
tools/bench_record.cpp:29: DEFINE_int32(tile_grid_size, 0, "Set the tile grid
size, if non zero switches to tile grid picture and enables
kOptimizeForClippedPlayback_RecordingFlag.");
On 2014/02/28 17:27:38, mtklein wrote:
> tileGridSize in Skia.

Done.

https://codereview.chromium.org/184543003/diff/1/tools/bench_record.cpp#newco...
tools/bench_record.cpp:38: int recording_flags = FLAGS_flags;
On 2014/02/28 17:27:38, mtklein wrote:
> recordingFlags

Done.

[mtklein, 2014-02-28 18:52:44]

On 2014/02/28 18:50:46, iancottrell wrote:
> Incidentally, running bench record with all the bbox types...
> 
> Nothing : 21.84
> R Tree : 34.28
> Tile grid : 36.66
> Quad tree : 47.14
Good to know.

lgtm

[mtklein, 2014-02-28 18:52:57]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2014-02-28 18:53:01]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/iancottrell@google.com/184543003/20001

[commit-bot: I haz the power, 2014-02-28 19:25:13]

Change committed as 13628