cc: Change the activation/ready for draw check from layer to queue.

cc: Change the activation/ready for draw check from layer to queue.

This patch changes where we do the ready to draw or ready to activate
check. Previously, this was done in the layers. Tile manager got all
of the picture layers, and asked them if they were ready. How, it
just builds a queue of an appropriate type to get this information.

R=reveman, danakj

Committed: https://crrev.com/831e0aa53be31260b65320176f56a698f2295feb
Cr-Commit-Position: refs/heads/master@{#313577}

[vmpstr, 2015-01-22 20:04:11]

Please take a look. This depends on

https://codereview.chromium.org/863013004/
https://codereview.chromium.org/862263004/
https://codereview.chromium.org/866883003/

[vmpstr, 2015-01-28 00:25:13]

On 2015/01/22 20:04:11, vmpstr wrote:
> Please take a look. This depends on
> 
> https://codereview.chromium.org/863013004/
> https://codereview.chromium.org/862263004/
> https://codereview.chromium.org/866883003/

Friendly ping.

[danakj, 2015-01-28 00:48:19]

LGTM

https://codereview.chromium.org/868803002/diff/60001/cc/resources/tile_manage...
File cc/resources/tile_manager.cc (right):

https://codereview.chromium.org/868803002/diff/60001/cc/resources/tile_manage...
cc/resources/tile_manager.cc:871: // check we really want. Hence, we need to
iterate the tiles and only return
ITYM this to explain why we're checking IsReadyToDraw() and not checking for a
resource? Can you say that if it's true?

https://codereview.chromium.org/868803002/diff/60001/cc/test/fake_tile_manage...
File cc/test/fake_tile_manager_client.h (right):

https://codereview.chromium.org/868803002/diff/60001/cc/test/fake_tile_manage...
cc/test/fake_tile_manager_client.h:30: std::vector<PictureLayerImpl*>
picture_layers_;
remove this?

https://codereview.chromium.org/868803002/diff/60001/cc/trees/layer_tree_host...
File cc/trees/layer_tree_host_impl.h (right):

https://codereview.chromium.org/868803002/diff/60001/cc/trees/layer_tree_host...
cc/trees/layer_tree_host_impl.h:253: // Returns existing picture layers.
is this TODO:remove, or do you mean for this to stick around?

[vmpstr, 2015-01-28 19:51:17]

https://codereview.chromium.org/868803002/diff/60001/cc/resources/tile_manage...
File cc/resources/tile_manager.cc (right):

https://codereview.chromium.org/868803002/diff/60001/cc/resources/tile_manage...
cc/resources/tile_manager.cc:871: // check we really want. Hence, we need to
iterate the tiles and only return
On 2015/01/28 00:48:18, danakj wrote:
> ITYM this to explain why we're checking IsReadyToDraw() and not checking for a
> resource? Can you say that if it's true?

I'm just trying to convey the fact that we still need to iterate the queue
instead of checking that it's empty. I rephrased the comment.

https://codereview.chromium.org/868803002/diff/60001/cc/test/fake_tile_manage...
File cc/test/fake_tile_manager_client.h (right):

https://codereview.chromium.org/868803002/diff/60001/cc/test/fake_tile_manage...
cc/test/fake_tile_manager_client.h:30: std::vector<PictureLayerImpl*>
picture_layers_;
On 2015/01/28 00:48:19, danakj wrote:
> remove this?

Done.

https://codereview.chromium.org/868803002/diff/60001/cc/trees/layer_tree_host...
File cc/trees/layer_tree_host_impl.h (right):

https://codereview.chromium.org/868803002/diff/60001/cc/trees/layer_tree_host...
cc/trees/layer_tree_host_impl.h:253: // Returns existing picture layers.
On 2015/01/28 00:48:19, danakj wrote:
> is this TODO:remove, or do you mean for this to stick around?

Added a TODO.

[vmpstr, 2015-01-28 19:51:21]

The CQ bit was checked by vmpstr@chromium.org

[commit-bot: I haz the power, 2015-01-28 19:52:06]

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

[commit-bot: I haz the power, 2015-01-28 20:45:00]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-01-28 20:46:11]

Patchset 5 (id:??) landed as
https://crrev.com/831e0aa53be31260b65320176f56a698f2295feb
Cr-Commit-Position: refs/heads/master@{#313577}