Use the right point for marker orientation when closing a subpath

Use the right point for marker orientation when closing a subpath

Path::apply doesn't pass a point along for the PathElementCloseSubpath
command. This would yield the wrong orientation on the last marker-mid
on the path (see crbug.com/633012#c1 for an example.)
Use m_subpathStart where needed instead.

BUG=633012
Committed: https://crrev.com/f5372970166aa5d5124da806c698558d27e2dad5
Cr-Commit-Position: refs/heads/master@{#434988}

[fs, 2016-11-29 13:16:13]

Description was changed from

==========
Use the right point for marker orientation when closing a subpath

Path::apply doesn't pass a point along for the PathElementCloseSubpath
command. Use m_subpathStart where needed instead.

BUG=633012
==========

to

==========
Use the right point for marker orientation when closing a subpath

Path::apply doesn't pass a point along for the PathElementCloseSubpath
command. This would yield the wrong orientation on the last marker-mid
on the path. Use m_subpathStart where needed instead.

BUG=633012
==========

[fs, 2016-11-29 13:17:07]

Description was changed from

==========
Use the right point for marker orientation when closing a subpath

Path::apply doesn't pass a point along for the PathElementCloseSubpath
command. This would yield the wrong orientation on the last marker-mid
on the path. Use m_subpathStart where needed instead.

BUG=633012
==========

to

==========
Use the right point for marker orientation when closing a subpath

Path::apply doesn't pass a point along for the PathElementCloseSubpath
command. This would yield the wrong orientation on the last marker-mid
on the path (see crbug.com/633012#c1 for an example.)
Use m_subpathStart where needed instead.

BUG=633012
==========

[fs, 2016-11-29 13:17:32]

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

[commit-bot: I haz the power, 2016-11-29 13:17:44]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[fs, 2016-11-29 14:33:57]

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

[f(malita), 2016-11-29 14:37:28]

lgtm

[commit-bot: I haz the power, 2016-11-29 14:43:40]

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

[commit-bot: I haz the power, 2016-11-29 14:43:41]

Dry run: This issue passed the CQ dry run.

[fs, 2016-11-29 15:03:22]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-11-29 15:03:45]

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, 2016-11-29 15:07:24]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480431802682000, "parent_rev":
"dbc6835f14c78b30975fc1e268dbf4a93414dde5", "commit_rev":
"17ad6e395c94c1e1ca71ada4a39416fde30d2410"}

[commit-bot: I haz the power, 2016-11-29 15:07:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-29 15:10:58]

Description was changed from

==========
Use the right point for marker orientation when closing a subpath

Path::apply doesn't pass a point along for the PathElementCloseSubpath
command. This would yield the wrong orientation on the last marker-mid
on the path (see crbug.com/633012#c1 for an example.)
Use m_subpathStart where needed instead.

BUG=633012
==========

to

==========
Use the right point for marker orientation when closing a subpath

Path::apply doesn't pass a point along for the PathElementCloseSubpath
command. This would yield the wrong orientation on the last marker-mid
on the path (see crbug.com/633012#c1 for an example.)
Use m_subpathStart where needed instead.

BUG=633012

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

[commit-bot: I haz the power, 2016-11-29 15:10:59]

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