Don't persist the SVGTextLayoutAttributesBuilder

Don't persist the SVGTextLayoutAttributesBuilder

No partial updates are performed on the structures contained within
the builder, so keeping them around between layouts only amounts to
memory wasted. With this change the builder is now more of a proper
builder. buildLayoutAttributesForTextRoot() is folded into
buildLayoutAttributes().

BUG=594058
Committed: https://crrev.com/50e8dd8ad929f02ecfa3f020a6971a9606fdb993
Cr-Commit-Position: refs/heads/master@{#386616}

[fs, 2016-04-11 11:04:07]

Description was changed from

==========
Don't persist the SVGTextLayoutAttributesBuilder

No partial updates are performed on the structures contained within
the builder, so keeping them around between layouts only amounts to
memory wasted. With this change the builder is now more of a proper
builder.

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

to

==========
Don't persist the SVGTextLayoutAttributesBuilder

No partial updates are performed on the structures contained within
the builder, so keeping them around between layouts only amounts to
memory wasted. With this change the builder is now more of a proper
builder. buildLayoutAttributesForTextRoot() is folded into
buildLayoutAttributes().

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

[fs, 2016-04-11 11:56:03]

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

[pdr., 2016-04-12 03:44:27]

On 2016/04/11 at 11:56:04, fs wrote:
> 

LGTM

[fs, 2016-04-12 07:56:21]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-04-12 07:56:30]

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

[commit-bot: I haz the power, 2016-04-12 08:01:58]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-12 08:03:29]

Description was changed from

==========
Don't persist the SVGTextLayoutAttributesBuilder

No partial updates are performed on the structures contained within
the builder, so keeping them around between layouts only amounts to
memory wasted. With this change the builder is now more of a proper
builder. buildLayoutAttributesForTextRoot() is folded into
buildLayoutAttributes().

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

to

==========
Don't persist the SVGTextLayoutAttributesBuilder

No partial updates are performed on the structures contained within
the builder, so keeping them around between layouts only amounts to
memory wasted. With this change the builder is now more of a proper
builder. buildLayoutAttributesForTextRoot() is folded into
buildLayoutAttributes().

BUG=594058

Committed: https://crrev.com/50e8dd8ad929f02ecfa3f020a6971a9606fdb993
Cr-Commit-Position: refs/heads/master@{#386616}
==========

[commit-bot: I haz the power, 2016-04-12 08:03:30]

Patchset 2 (id:??) landed as
https://crrev.com/50e8dd8ad929f02ecfa3f020a6971a9606fdb993
Cr-Commit-Position: refs/heads/master@{#386616}