Return AffineTransform from SVGTextFragment::buildFragmentTransform

Return AffineTransform from SVGTextFragment::buildFragmentTransform

Instead of passing an AffineTransform as an out-parameter, just return
it instead. Since we always copy the resulting transform into the
out-parameter anyway, there should be no loss of efficiency.
This also enables some of the users to be written in a more compact
way.

Also add boundingBox(), boundingQuad() and overflowBoundingBox()
helpers to SVGTextFragment and use where possible.

BUG=571415
Committed: https://crrev.com/c9cbd28cd596618f61dc47b897aee61382004e28
Cr-Commit-Position: refs/heads/master@{#366713}

[fs, 2015-12-22 10:53:31]

Description was changed from

==========
Return AffineTransform from SVGTextFragment::buildFragmentTransform

Instead of passing an AFfineTransform as an out-parameter, just return
it instead. Since we always copy the resulting transform into the
out-parameter anyway, there should be no loss of efficiency.
This also enables some of the users to be written in a more compact
way.

Also add boundingBox(), boundingQuad() and overflowBoundingBox()
helpers to SVGTextFragment and use where possible.

BUG=571415
==========

to

==========
Return AffineTransform from SVGTextFragment::buildFragmentTransform

Instead of passing an AffineTransform as an out-parameter, just return
it instead. Since we always copy the resulting transform into the
out-parameter anyway, there should be no loss of efficiency.
This also enables some of the users to be written in a more compact
way.

Also add boundingBox(), boundingQuad() and overflowBoundingBox()
helpers to SVGTextFragment and use where possible.

BUG=571415
==========

[fs, 2015-12-22 11:59:17]

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

[pdr., 2015-12-23 03:29:54]

The CQ bit was checked by pdr@chromium.org

[pdr., 2015-12-23 03:29:55]

lgtm

[commit-bot: I haz the power, 2015-12-23 03:30:34]

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

[commit-bot: I haz the power, 2015-12-23 06:23:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-23 06:24:36]

Description was changed from

==========
Return AffineTransform from SVGTextFragment::buildFragmentTransform

Instead of passing an AffineTransform as an out-parameter, just return
it instead. Since we always copy the resulting transform into the
out-parameter anyway, there should be no loss of efficiency.
This also enables some of the users to be written in a more compact
way.

Also add boundingBox(), boundingQuad() and overflowBoundingBox()
helpers to SVGTextFragment and use where possible.

BUG=571415
==========

to

==========
Return AffineTransform from SVGTextFragment::buildFragmentTransform

Instead of passing an AffineTransform as an out-parameter, just return
it instead. Since we always copy the resulting transform into the
out-parameter anyway, there should be no loss of efficiency.
This also enables some of the users to be written in a more compact
way.

Also add boundingBox(), boundingQuad() and overflowBoundingBox()
helpers to SVGTextFragment and use where possible.

BUG=571415

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

[commit-bot: I haz the power, 2015-12-23 06:24:37]

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