Added grid data to logging output

Added grid data to logging output

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/06fdc6937df52942226a0a5e7bba169c27006ef1

[kelvinly, 2014-06-03 18:27:23]

Added the grid width and height to the logging output. It builds, but I haven't
rerun bench_pictures to test it; is it okay to assume that I probably didn't
break it?

[benchen, 2014-06-03 18:46:33]

https://codereview.chromium.org/318433003/diff/60001/tools/PictureRenderer.h
File tools/PictureRenderer.h (right):

https://codereview.chromium.org/318433003/diff/60001/tools/PictureRenderer.h#...
tools/PictureRenderer.h:264: config.appendS32(fGridInfo.fTileInterval.fWidth);
I didn't see fWidth used with fTileInterval.
https://github.com/google/skia/blob/770963f23f4fc313db0fa3bac18b1b8aafb55f17/...
I think they're accessed by .width() and .height()

[kelvinly, 2014-06-03 18:53:00]

Changed from fWidth/Height to width/height()

https://codereview.chromium.org/318433003/diff/60001/tools/PictureRenderer.h
File tools/PictureRenderer.h (right):

https://codereview.chromium.org/318433003/diff/60001/tools/PictureRenderer.h#...
tools/PictureRenderer.h:264: config.appendS32(fGridInfo.fTileInterval.fWidth);
On 2014/06/03 18:46:33, benchen wrote:
> I didn't see fWidth used with fTileInterval.
>
https://github.com/google/skia/blob/770963f23f4fc313db0fa3bac18b1b8aafb55f17/...
> I think they're accessed by .width() and .height()

Yep, you're right. Sorry, guess I skimmed over those

[jcgregorio, 2014-06-03 20:46:18]

On 2014/06/03 18:53:00, kelvinly wrote:
> Changed from fWidth/Height to width/height()
> 
> https://codereview.chromium.org/318433003/diff/60001/tools/PictureRenderer.h
> File tools/PictureRenderer.h (right):
> 
>
https://codereview.chromium.org/318433003/diff/60001/tools/PictureRenderer.h#...
> tools/PictureRenderer.h:264: config.appendS32(fGridInfo.fTileInterval.fWidth);
> On 2014/06/03 18:46:33, benchen wrote:
> > I didn't see fWidth used with fTileInterval.
> >
>
https://github.com/google/skia/blob/770963f23f4fc313db0fa3bac18b1b8aafb55f17/...
> > I think they're accessed by .width() and .height()
> 
> Yep, you're right. Sorry, guess I skimmed over those

LGTM

[kelvinly, 2014-06-03 20:48:18]

The CQ bit was checked by kelvinly@google.com

[commit-bot: I haz the power, 2014-06-03 20:48:31]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/kelvinly@google.com/318433003/70001

[benchen, 2014-06-03 20:49:00]

lgtm assuming local bench_pictures run succeeded with the change.

[kelvinly, 2014-06-03 22:20:44]

The CQ bit was unchecked by kelvinly@google.com

[kelvinly, 2014-06-03 22:20:49]

The CQ bit was checked by kelvinly@google.com

[commit-bot: I haz the power, 2014-06-03 22:20:52]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/kelvinly@google.com/318433003/70001

[commit-bot: I haz the power, 2014-06-03 22:43:38]

Change committed as 06fdc6937df52942226a0a5e7bba169c27006ef1