Add a stat to the smoothness benchmark for avg number of missing tiles.

Add a stat to the smoothness benchmark for avg number of missing tiles.

BUG=None


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

[Ian Vollick, 2012-12-07 01:08:26]

nduca, can you check if I'm measuring checkerboarding correctly?

hartmanng, can you please take a look at tools/perf/?

[nduca, 2012-12-07 01:14:37]

lgtm but good to get quick eye from dana or enne

https://codereview.chromium.org/11478016/diff/1/cc/layer_tree_host_impl.h
File cc/layer_tree_host_impl.h (right):

https://codereview.chromium.org/11478016/diff/1/cc/layer_tree_host_impl.h#new...
cc/layer_tree_host_impl.h:385: int64 m_cumulativeNumMissingTiles;
any reason using the word missingTiles vs numCheckerboards?

[Ian Vollick, 2012-12-07 01:33:24]

On 2012/12/07 01:14:37, nduca wrote:
> lgtm but good to get quick eye from dana or enne
sg. +enne, +danakj
> 
> https://codereview.chromium.org/11478016/diff/1/cc/layer_tree_host_impl.h
> File cc/layer_tree_host_impl.h (right):
> 
>
https://codereview.chromium.org/11478016/diff/1/cc/layer_tree_host_impl.h#new...
> cc/layer_tree_host_impl.h:385: int64 m_cumulativeNumMissingTiles;
> any reason using the word missingTiles vs numCheckerboards?

Only that the old name for the member on AppendQuadsData was hadMissingTiles.
I'm not tied to the name and would be happy to change it.

[enne (OOO), 2012-12-07 17:40:24]

lgtm

[commit-bot: I haz the power, 2012-12-07 17:44:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vollick@chromium.org/11478016/1

[commit-bot: I haz the power, 2012-12-07 21:40:54]

Change committed as 171845