cc: Move tiling raster tile iterator to be a tiling set queue class.

cc: Move tiling raster tile iterator to be a tiling set queue class.

This patch moves tiling raster tile iterator and makes it a private
class of the tiling set raster queue. This reduces the public interface
for accessing tiles.

R=danakj, reveman

Committed: https://crrev.com/dfcd77d6aa00197035c78693d9aca183e00e6884
Cr-Commit-Position: refs/heads/master@{#310446}

[vmpstr, 2015-01-06 19:53:50]

Please take a look.

https://codereview.chromium.org/800853007/diff/1/cc/resources/picture_layer_t...
File cc/resources/picture_layer_tiling_perftest.cc (left):

https://codereview.chromium.org/800853007/diff/1/cc/resources/picture_layer_t...
cc/resources/picture_layer_tiling_perftest.cc:242:
TEST_F(PictureLayerTilingPerfTest, TilingRasterTileIteratorConstruct) {
These perftests don't make sense anymore, since they are covered by the tiling
set perftests.

https://codereview.chromium.org/800853007/diff/1/cc/resources/picture_layer_t...
File cc/resources/picture_layer_tiling_unittest.cc (left):

https://codereview.chromium.org/800853007/diff/1/cc/resources/picture_layer_t...
cc/resources/picture_layer_tiling_unittest.cc:1090: TEST(PictureLayerTilingTest,
TilingRasterTileIteratorStaticViewport) {
This is moved to the tile manager unittests with the changes being that we use a
TilingSetRasterQueue instead of PictureLayerTiling::TilingRasterTileIterator.

[danakj, 2015-01-07 20:05:31]

LGTM

https://codereview.chromium.org/800853007/diff/1/cc/resources/picture_layer_t...
File cc/resources/picture_layer_tiling.cc (right):

https://codereview.chromium.org/800853007/diff/1/cc/resources/picture_layer_t...
cc/resources/picture_layer_tiling.cc:792: void
PictureLayerTiling::UpdateTilePriority(Tile* tile) const {
I liked the old function name, how come it changed?

[vmpstr, 2015-01-08 00:24:00]

https://codereview.chromium.org/800853007/diff/1/cc/resources/picture_layer_t...
File cc/resources/picture_layer_tiling.cc (right):

https://codereview.chromium.org/800853007/diff/1/cc/resources/picture_layer_t...
cc/resources/picture_layer_tiling.cc:792: void
PictureLayerTiling::UpdateTilePriority(Tile* tile) const {
On 2015/01/07 20:05:31, danakj wrote:
> I liked the old function name, how come it changed?

I thought it would make more sense, but I think you're right: the other way was
better. Changed it back

[vmpstr, 2015-01-08 00:32:08]

The CQ bit was checked by vmpstr@chromium.org

[commit-bot: I haz the power, 2015-01-08 00:33:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/800853007/40001

[commit-bot: I haz the power, 2015-01-08 02:52:15]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-01-08 02:53:11]

Patchset 3 (id:??) landed as
https://crrev.com/dfcd77d6aa00197035c78693d9aca183e00e6884
Cr-Commit-Position: refs/heads/master@{#310446}