Remove extra per-character walk in SVGTextLayoutEngine

Remove extra per-character walk in SVGTextLayoutEngine

This patch removes a per-character walk in SVGTextLayoutEngine which
was unnecessary because the glyph computation has the data available
already.

BUG=597076
Committed: https://crrev.com/f75cb329068c3768b2d4e6b0c0243795c81d1ae6
Cr-Commit-Position: refs/heads/master@{#382908}

[pdr., 2016-03-22 22:28:21]

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

[pdr., 2016-03-22 22:28:21]

We need to wait for the rebaselines from crbug.com/596721 to land first, but
this is ready to review otherwise.

[fs, 2016-03-22 22:58:37]

On 2016/03/22 at 22:28:21, pdr wrote:
> We need to wait for the rebaselines from crbug.com/596721 to land first, but
this is ready to review otherwise.

LGTM!

[pdr., 2016-03-23 04:53:48]

The CQ bit was checked by pdr@chromium.org

[pdr., 2016-03-23 04:53:48]

The patchset sent to the CQ was uploaded after l-g-t-m from fs@opera.com
Link to the patchset: https://codereview.chromium.org/1824193002/#ps40001
(title: "Improve user interebase")

[commit-bot: I haz the power, 2016-03-23 04:53:56]

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

[commit-bot: I haz the power, 2016-03-23 05:55:50]

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

[commit-bot: I haz the power, 2016-03-23 05:55:51]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[pdr., 2016-03-23 06:04:09]

The CQ bit was checked by pdr@chromium.org

[pdr., 2016-03-23 06:04:09]

The patchset sent to the CQ was uploaded after l-g-t-m from fs@opera.com
Link to the patchset: https://codereview.chromium.org/1824193002/#ps60001
(title: "Great Expectations")

[commit-bot: I haz the power, 2016-03-23 06:04:17]

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

[commit-bot: I haz the power, 2016-03-23 07:21:42]

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

[commit-bot: I haz the power, 2016-03-23 07:21:43]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[pdr., 2016-03-23 18:45:58]

The CQ bit was checked by pdr@chromium.org

[pdr., 2016-03-23 18:45:58]

The patchset sent to the CQ was uploaded after l-g-t-m from fs@opera.com
Link to the patchset: https://codereview.chromium.org/1824193002/#ps80001
(title: "Greatest Expectations")

[commit-bot: I haz the power, 2016-03-23 18:46:15]

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

[commit-bot: I haz the power, 2016-03-23 20:07:07]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-03-23 20:09:49]

Description was changed from

==========
Remove extra per-character walk in SVGTextLayoutEngine

This patch removes a per-character walk in SVGTextLayoutEngine which
was unnecessary because the glyph computation has the data available
already.

BUG=597076
==========

to

==========
Remove extra per-character walk in SVGTextLayoutEngine

This patch removes a per-character walk in SVGTextLayoutEngine which
was unnecessary because the glyph computation has the data available
already.

BUG=597076

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

[commit-bot: I haz the power, 2016-03-23 20:09:50]

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