cc: Add TileManager interface that allows LTHI to determine when to activate pending tree.

cc: Add TileManager interface that allows LTHI to determine when to activate pending tree.

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

[reveman, 2012-12-14 03:43:09]

How about something like this to determine when to activate pending tree?

[enne (OOO), 2012-12-14 17:35:57]

I kind of tink there needs to be a way to differentiate between what the pending
tree and the active tree needs.  Am I missing some way that the bin gives you
this?

[reveman, 2012-12-14 19:03:41]

On 2012/12/14 17:35:57, enne wrote:
> I kind of tink there needs to be a way to differentiate between what the
pending
> tree and the active tree needs.  Am I missing some way that the bin gives you
> this?

Heh, I didn't even think of that. We obviously need to differentiate between the
trees and the bin count doesn't provide that right now.

How about we compute a bin for each tree in ManageTiles() and add a tree
argument to GetTilesInBinCount()? I don't think that will have a significant
impact on the cost of calling ManageTiles() as sorting is not affected.

[enne (OOO), 2012-12-14 19:06:04]

That sounds great to me.

[reveman, 2012-12-15 21:05:40]

This is now determining the bin for each tree priority and LTHI can efficiently
ask the tile manager how many drawable tiles exist in a specific bin for a
specific tree. PTAL.

[enne (OOO), 2012-12-16 01:26:44]

lgtm

https://codereview.chromium.org/11568027/diff/6001/cc/tile_manager.cc
File cc/tile_manager.cc (right):

https://codereview.chromium.org/11568027/diff/6001/cc/tile_manager.cc#newcode36
cc/tile_manager.cc:36: cc::TileManagerBin BinFromTilePriority(const
cc::TilePriority& prio) {
This is a nice refactoring.

[commit-bot: I haz the power, 2012-12-16 07:30:04]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/reveman@chromium.org/11568027/8003

[commit-bot: I haz the power, 2012-12-16 20:13:45]

Change committed as 173385