Fix textPath textLength position adjustment for the spacing case

Fix textPath textLength position adjustment for the spacing case

Follow-up to https://codereview.chromium.org/2870393002, adjusting the
<textPath> 'textLength' computation in the same way.

BUG=719522
Review-Url: https://codereview.chromium.org/2868413003
Cr-Commit-Position: refs/heads/master@{#471248}
Committed: https://chromium.googlesource.com/chromium/src/+/1cc74004b24c8a79dcee523e913c1d929c1167da

[fs, 2017-05-11 16:45:51]

The CQ bit was checked by fs@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2017-05-11 16:46:26]

Dry run: 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, 2017-05-11 18:48:58]

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

[commit-bot: I haz the power, 2017-05-11 18:48:58]

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

[fs, 2017-05-11 21:44:22]

The CQ bit was checked by fs@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2017-05-11 21:45:07]

Dry run: CQ is trying da patch.

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

[fs, 2017-05-11 21:45:08]

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

[fs, 2017-05-11 21:45:08]

Part Deux

[pdr., 2017-05-11 22:46:23]

On 2017/05/11 at 21:45:08, fs wrote:
> Part Deux

LGTM! Off-by-one, the 2nd hardest computer science problem (after naming).


I spent a few minutes looking for a way to combine this logic into a common
place but I don't think there's a clean way.

[commit-bot: I haz the power, 2017-05-11 23:34:09]

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

[commit-bot: I haz the power, 2017-05-11 23:34:09]

Dry run: This issue passed the CQ dry run.

[fs, 2017-05-12 08:08:14]

On 2017/05/11 at 22:46:23, pdr wrote:
> On 2017/05/11 at 21:45:08, fs wrote:
> > Part Deux
> 
> LGTM! Off-by-one, the 2nd hardest computer science problem (after naming).
> 
> 
> I spent a few minutes looking for a way to combine this logic into a common
place but I don't think there's a clean way.

Yes, I did the same, and reached a similar conclusion. (Could probably put the
logic in a single place, but some parts of it would "leak" because of how the
two code-paths differ. Consolidating the two code-paths some more is probably a
possibility, but one that brings a lot of work...)

[fs, 2017-05-12 08:08:18]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2017-05-12 08:08:32]

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, 2017-05-12 08:16:05]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1494576498327450,
"parent_rev": "18e6b4998b080ffe6c303c85a3498dab8f55f622", "commit_rev":
"1cc74004b24c8a79dcee523e913c1d929c1167da"}

[commit-bot: I haz the power, 2017-05-12 08:16:24]

Description was changed from

==========
Fix textPath textLength position adjustment for the spacing case

Follow-up to https://codereview.chromium.org/2870393002, adjusting the
<textPath> 'textLength' computation in the same way.

BUG=719522
==========

to

==========
Fix textPath textLength position adjustment for the spacing case

Follow-up to https://codereview.chromium.org/2870393002, adjusting the
<textPath> 'textLength' computation in the same way.

BUG=719522

Review-Url: https://codereview.chromium.org/2868413003
Cr-Commit-Position: refs/heads/master@{#471248}
Committed:
https://chromium.googlesource.com/chromium/src/+/1cc74004b24c8a79dcee523e913c...
==========

[commit-bot: I haz the power, 2017-05-12 08:16:25]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/1cc74004b24c8a79dcee523e913c...