Report decoded webfont blob size to memory-infra

Report decoded webfont blob size to memory-infra

Decoded webfonts are stored as SkData, and its backing store is malloc.
(See WebFontDecoder::decode() for the implementation)
Report them as a part of web_cache because FontResource is the owner
of FontCustomPlatformData, which retains a reference to SkTypeface and
SkData.

BUG=668578
Committed: https://crrev.com/c33cef3224fde184a75ca9f79adc1a2f56ac3c14
Cr-Commit-Position: refs/heads/master@{#434486}

[bashi, 2016-11-25 02:21:37]

The CQ bit was checked by bashi@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-25 02:21:52]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[bashi, 2016-11-25 02:31:30]

bashi@chromium.org changed reviewers:
+ drott@chromium.org, haraken@chromium.org

[bashi, 2016-11-25 02:31:31]

PTAL

[haraken, 2016-11-25 02:40:10]

LGTM on my side

[commit-bot: I haz the power, 2016-11-25 03:37:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-25 03:37:53]

Dry run: This issue passed the CQ dry run.

[drott, 2016-11-25 11:54:26]

LGTM - it would be nice if you could add something like ", see
WebFontDecoder.cpp" for backing up the "malloc" backing store information in the
CL description.

So it's pretty tricky, right? The FontCustomPlatformData is held by
FontResource, but how is the lifecycle of FontResource controlled? Through
Oilpan and the FontCache?

[bashi, 2016-11-25 12:22:27]

Description was changed from

==========
Report decoded webfont blob size to memory-infra

Decoded webfonts are stored as SkData, and its backing store is malloc.
Report them as a part of web_cache because FontResource is the owner
of FontCustomPlatformData, which retains a reference to SkTypeface and
SkData.

BUG=668578
==========

to

==========
Report decoded webfont blob size to memory-infra

Decoded webfonts are stored as SkData, and its backing store is malloc.
(See WebFontDecoder::decode() for the implementation)
Report them as a part of web_cache because FontResource is the owner
of FontCustomPlatformData, which retains a reference to SkTypeface and
SkData.

BUG=668578
==========

[bashi, 2016-11-25 12:26:20]

bashi@chromium.org changed reviewers:
+ hiroshige@chromium.org

[bashi, 2016-11-25 12:26:21]

The lifetime of Resource has changed recently and it's basically destroyed by
Oilpan's GC if it's inactive.

hiroshige@: Correct me if I'm wrong.

[bashi, 2016-11-25 12:26:28]

The CQ bit was checked by bashi@chromium.org

[commit-bot: I haz the power, 2016-11-25 12:26:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-25 12:30:20]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480076788117560, "parent_rev":
"44434972e3790f0b4c2bc4f4f2d747bdc8fef64b", "commit_rev":
"40608b4bde22eec5d32b505c05f3ee7da0e34c88"}

[commit-bot: I haz the power, 2016-11-25 12:30:43]

Description was changed from

==========
Report decoded webfont blob size to memory-infra

Decoded webfonts are stored as SkData, and its backing store is malloc.
(See WebFontDecoder::decode() for the implementation)
Report them as a part of web_cache because FontResource is the owner
of FontCustomPlatformData, which retains a reference to SkTypeface and
SkData.

BUG=668578
==========

to

==========
Report decoded webfont blob size to memory-infra

Decoded webfonts are stored as SkData, and its backing store is malloc.
(See WebFontDecoder::decode() for the implementation)
Report them as a part of web_cache because FontResource is the owner
of FontCustomPlatformData, which retains a reference to SkTypeface and
SkData.

BUG=668578
==========

[commit-bot: I haz the power, 2016-11-25 12:30:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-25 12:32:29]

Description was changed from

==========
Report decoded webfont blob size to memory-infra

Decoded webfonts are stored as SkData, and its backing store is malloc.
(See WebFontDecoder::decode() for the implementation)
Report them as a part of web_cache because FontResource is the owner
of FontCustomPlatformData, which retains a reference to SkTypeface and
SkData.

BUG=668578
==========

to

==========
Report decoded webfont blob size to memory-infra

Decoded webfonts are stored as SkData, and its backing store is malloc.
(See WebFontDecoder::decode() for the implementation)
Report them as a part of web_cache because FontResource is the owner
of FontCustomPlatformData, which retains a reference to SkTypeface and
SkData.

BUG=668578

Committed: https://crrev.com/c33cef3224fde184a75ca9f79adc1a2f56ac3c14
Cr-Commit-Position: refs/heads/master@{#434486}
==========

[commit-bot: I haz the power, 2016-11-25 12:32:31]

Patchset 1 (id:??) landed as
https://crrev.com/c33cef3224fde184a75ca9f79adc1a2f56ac3c14
Cr-Commit-Position: refs/heads/master@{#434486}