cc: Add PictureLayerImpl::AsValue

cc: Add PictureLayerImpl::AsValue

This allows for information about contents scales and tilings to be recorded
for the frame viewer.

R=nduca@chromium.org
BUG=176413
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182747

[nduca, 2013-02-15 01:10:24]

lgtm minor nits

https://codereview.chromium.org/12258044/diff/1/cc/layer_impl.cc
File cc/layer_impl.cc (right):

https://codereview.chromium.org/12258044/diff/1/cc/layer_impl.cc#newcode977
cc/layer_impl.cc:977: dict->SetInteger("layer_height", bounds().width());
bounds_size? MathUtils should have a size->value

https://codereview.chromium.org/12258044/diff/1/cc/picture_layer_tiling.cc
File cc/picture_layer_tiling.cc (right):

https://codereview.chromium.org/12258044/diff/1/cc/picture_layer_tiling.cc#ne...
cc/picture_layer_tiling.cc:538: state->SetInteger("content_height",
ContentRect().height());
maybe also use a Size->Value?

[enne (OOO), 2013-02-15 01:14:04]

https://codereview.chromium.org/12258044/diff/1/cc/layer_impl.cc
File cc/layer_impl.cc (right):

https://codereview.chromium.org/12258044/diff/1/cc/layer_impl.cc#newcode977
cc/layer_impl.cc:977: dict->SetInteger("layer_height", bounds().width());
On 2013/02/15 01:10:24, nduca wrote:
> bounds_size? MathUtils should have a size->value

Done.

https://codereview.chromium.org/12258044/diff/1/cc/picture_layer_tiling.cc
File cc/picture_layer_tiling.cc (right):

https://codereview.chromium.org/12258044/diff/1/cc/picture_layer_tiling.cc#ne...
cc/picture_layer_tiling.cc:538: state->SetInteger("content_height",
ContentRect().height());
On 2013/02/15 01:10:24, nduca wrote:
> maybe also use a Size->Value?

Done.

[commit-bot: I haz the power, 2013-02-15 01:14:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/enne@chromium.org/12258044/5001

[commit-bot: I haz the power, 2013-02-15 14:18:32]

Retried try job too often on win7_aura for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...

[commit-bot: I haz the power, 2013-02-15 17:16:26]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/enne@chromium.org/12258044/5001

[commit-bot: I haz the power, 2013-02-15 17:36:45]

Change committed as 182747