Move metrics list storage to LayoutSVGInlineText

Move metrics list storage to LayoutSVGInlineText

It was previously stored in SVGTextLayoutAttributes (which is stored in
LayoutSVGInlineText). The connection between these two is very loose, so
letting the metrics be a part of the attributes structure doesn't feel
entirely logical. There's still a back-pointer in the attributes
structure, which means it's still reachable in the same way (albeit with
one additional indirection.)
Rename the various accessors to metricsList().

BUG=594058
Committed: https://crrev.com/91d29a30053008ec263fdeeda7e51d0c7cadc1a8
Cr-Commit-Position: refs/heads/master@{#384450}

[fs, 2016-03-30 18:23:15]

fs@opera.com changed reviewers:
+ pdr@chromium.org

[pdr., 2016-03-31 23:10:56]

On 2016/03/30 at 18:23:15, fs wrote:
> 

LGTM!

I was unsure at first, but https://crbug.com/594058#c9 cleared this up for me.

[pdr., 2016-03-31 23:10:58]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2016-03-31 23:11:29]

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

[commit-bot: I haz the power, 2016-04-01 00:44:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-01 00:45:30]

Description was changed from

==========
Move metrics list storage to LayoutSVGInlineText

It was previously stored in SVGTextLayoutAttributes (which is stored in
LayoutSVGInlineText). The connection between these two is very loose, so
letting the metrics be a part of the attributes structure doesn't feel
entirely logical. There's still a back-pointer in the attributes
structure, which means it's still reachable in the same way (albeit with
one additional indirection.)
Rename the various accessors to metricsList().

BUG=594058
==========

to

==========
Move metrics list storage to LayoutSVGInlineText

It was previously stored in SVGTextLayoutAttributes (which is stored in
LayoutSVGInlineText). The connection between these two is very loose, so
letting the metrics be a part of the attributes structure doesn't feel
entirely logical. There's still a back-pointer in the attributes
structure, which means it's still reachable in the same way (albeit with
one additional indirection.)
Rename the various accessors to metricsList().

BUG=594058

Committed: https://crrev.com/91d29a30053008ec263fdeeda7e51d0c7cadc1a8
Cr-Commit-Position: refs/heads/master@{#384450}
==========

[commit-bot: I haz the power, 2016-04-01 00:45:31]

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