cc: Add layer eviction iterator perftests.

cc: Add layer eviction iterator perftests.

This patch adds a construct and a construct and iterate tests for
layer level eviction iterators.

Initial runs:
[ RUN      ] PictureLayerImplPerfTest.LayerEvictionTileIteratorConstructAndIterate
*RESULT layer_eviction_tile_iterator_construct_and_iterate: 32_100x100= 47900.80859375 runs/s
*RESULT layer_eviction_tile_iterator_construct_and_iterate: 32_500x500= 50885.39453125 runs/s
*RESULT layer_eviction_tile_iterator_construct_and_iterate: 64_100x100= 49707.58984375 runs/s
*RESULT layer_eviction_tile_iterator_construct_and_iterate: 64_500x500= 49089.5078125 runs/s
[       OK ] PictureLayerImplPerfTest.LayerEvictionTileIteratorConstructAndIterate (8069 ms)
[ RUN      ] PictureLayerImplPerfTest.LayerEvictionTileIteratorConstruct
*RESULT layer_eviction_tile_iterator_construct: 0_0_100x100= 52248.0390625 runs/s
*RESULT layer_eviction_tile_iterator_construct: 5000_0_100x100= 28544.1015625 runs/s
*RESULT layer_eviction_tile_iterator_construct: 9999_0_100x100= 29921.828125 runs/s
[       OK ] PictureLayerImplPerfTest.LayerEvictionTileIteratorConstruct (6042 ms)


R=reveman

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=283979

[vmpstr, 2014-07-17 19:09:25]

PTAL

[reveman, 2014-07-17 19:30:37]

https://codereview.chromium.org/404583002/diff/1/cc/layers/picture_layer_impl...
File cc/layers/picture_layer_impl_perftest.cc (right):

https://codereview.chromium.org/404583002/diff/1/cc/layers/picture_layer_impl...
cc/layers/picture_layer_impl_perftest.cc:32: const std::vector<Tile*>&
tiling_tiles = tiling->AllTilesForTesting();
why "const std::vector<Tile*>&"? doesn't AllTilesForTesting() return a copy.

[vmpstr, 2014-07-17 19:39:13]

https://codereview.chromium.org/404583002/diff/1/cc/layers/picture_layer_impl...
File cc/layers/picture_layer_impl_perftest.cc (right):

https://codereview.chromium.org/404583002/diff/1/cc/layers/picture_layer_impl...
cc/layers/picture_layer_impl_perftest.cc:32: const std::vector<Tile*>&
tiling_tiles = tiling->AllTilesForTesting();
On 2014/07/17 19:30:36, reveman wrote:
> why "const std::vector<Tile*>&"? doesn't AllTilesForTesting() return a copy.

const ref extends the lifetime of temporaries. I can change to a copy if you
prefer?

[reveman, 2014-07-17 20:19:28]

lgtm

https://codereview.chromium.org/404583002/diff/1/cc/layers/picture_layer_impl...
File cc/layers/picture_layer_impl_perftest.cc (right):

https://codereview.chromium.org/404583002/diff/1/cc/layers/picture_layer_impl...
cc/layers/picture_layer_impl_perftest.cc:32: const std::vector<Tile*>&
tiling_tiles = tiling->AllTilesForTesting();
On 2014/07/17 19:39:13, vmpstr wrote:
> On 2014/07/17 19:30:36, reveman wrote:
> > why "const std::vector<Tile*>&"? doesn't AllTilesForTesting() return a copy.
> 
> const ref extends the lifetime of temporaries. I can change to a copy if you
> prefer?

I think that would be better but I don't feel that strongly about it.
performance is not really of concern here and I'm pretty sure the compiler would
end up doing the same anyhow.

[vmpstr, 2014-07-17 20:21:47]

https://codereview.chromium.org/404583002/diff/1/cc/layers/picture_layer_impl...
File cc/layers/picture_layer_impl_perftest.cc (right):

https://codereview.chromium.org/404583002/diff/1/cc/layers/picture_layer_impl...
cc/layers/picture_layer_impl_perftest.cc:32: const std::vector<Tile*>&
tiling_tiles = tiling->AllTilesForTesting();
On 2014/07/17 20:19:28, reveman wrote:
> On 2014/07/17 19:39:13, vmpstr wrote:
> > On 2014/07/17 19:30:36, reveman wrote:
> > > why "const std::vector<Tile*>&"? doesn't AllTilesForTesting() return a
copy.
> > 
> > const ref extends the lifetime of temporaries. I can change to a copy if you
> > prefer?
> 
> I think that would be better but I don't feel that strongly about it.
> performance is not really of concern here and I'm pretty sure the compiler
would
> end up doing the same anyhow.

Changed.

[vmpstr, 2014-07-17 20:21:50]

The CQ bit was checked by vmpstr@chromium.org

[commit-bot: I haz the power, 2014-07-17 20:24:44]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/vmpstr@chromium.org/404583002/20001

[commit-bot: I haz the power, 2014-07-18 02:43:40]

Change committed as 283979