[tracing] Remove counter logic and just use address on glyph cache dumps

[tracing] Remove counter logic and just use address on glyph cache dumps

This CL removes the logic of counting the index of glyphs and uses the
address of the glyph object for dump name. This makes the dump names
consistent across multiple dumps.

BUG=543479
Committed: https://skia.googlesource.com/skia/+/93775bd279e7db34f074ce9820531663fa708696

[ssid, 2015-10-23 10:57:42]

Description was changed from

==========
[tracing] Remove counter logic and just use address

This CL removes the logic of counting the index of glyphs and uses the
address of the glyph object for dump name. This makes the dump names to
be consistent accross multiple dumps.

BUG=543479
==========

to

==========
[tracing] Remove counter logic and just use address on glyph cache dumps

This CL removes the logic of counting the index of glyphs and uses the
address of the glyph object for dump name. This makes the dump names to
be consistent accross multiple dumps.

BUG=543479
==========

[ssid, 2015-10-23 10:58:15]

ssid@chromium.org changed reviewers:
+ petrcermak@chromium.org

[ssid, 2015-10-23 10:58:15]

PTAL, thanks

[petrcermak, 2015-10-23 11:11:11]

Thanks for taking care of this :-)

LGTM with one description nit:

s/makes the dump names to be consistent/makes the dump names consistent/

Petr

[ssid, 2015-10-23 12:56:59]

Description was changed from

==========
[tracing] Remove counter logic and just use address on glyph cache dumps

This CL removes the logic of counting the index of glyphs and uses the
address of the glyph object for dump name. This makes the dump names to
be consistent accross multiple dumps.

BUG=543479
==========

to

==========
[tracing] Remove counter logic and just use address on glyph cache dumps

This CL removes the logic of counting the index of glyphs and uses the
address of the glyph object for dump name. This makes the dump names
consistent accross multiple dumps.

BUG=543479
==========

[ssid, 2015-10-23 12:57:06]

Description was changed from

==========
[tracing] Remove counter logic and just use address on glyph cache dumps

This CL removes the logic of counting the index of glyphs and uses the
address of the glyph object for dump name. This makes the dump names
consistent accross multiple dumps.

BUG=543479
==========

to

==========
[tracing] Remove counter logic and just use address on glyph cache dumps

This CL removes the logic of counting the index of glyphs and uses the
address of the glyph object for dump name. This makes the dump names
consistent across multiple dumps.

BUG=543479
==========

[ssid, 2015-10-23 12:57:39]

ssid@chromium.org changed reviewers:
+ reed@google.com

[ssid, 2015-10-23 12:57:39]

Small fix, PTAL, thanks

[reed1, 2015-10-23 17:35:49]

lgtm

[ssid, 2015-10-23 17:38:45]

The CQ bit was checked by ssid@chromium.org

[commit-bot: I haz the power, 2015-10-23 17:38:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1425443002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1425443002/1

[commit-bot: I haz the power, 2015-10-23 17:49:37]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/93775bd279e7db34f074ce9820531663fa708696