Correct normal computation for inverted polygons.

Correct normal computation for inverted polygons.

Rather than rely solely the vertices, and their winding
order for sign, we rely on the transform of the previous
normal for the sign, and the vertices (cross product of
edge vectors, similar to Newell's algorithm) for the
numerics of the direction.

BUG=595820
TEST=unittest
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/59d3768dd104c22554d86f410d22ad8f714c82eb
Cr-Commit-Position: refs/heads/master@{#386984}

[Peter Mayo, 2016-04-12 17:18:04]

Description was changed from

==========
Correct normal computation for inverted polygons.

Rather than rely solely the vertices, and their winding
order for sign, we rely on the transform of the previous
normal for the sign, and the vertices (cross product of
edge vectors, similar to Newell's algorithm) for the
numerics of the direction.

BUG=595820
TEST=unittest
==========

to

==========
Correct normal computation for inverted polygons.

Rather than rely solely the vertices, and their winding
order for sign, we rely on the transform of the previous
normal for the sign, and the vertices (cross product of
edge vectors, similar to Newell's algorithm) for the
numerics of the direction.

BUG=595820
TEST=unittest
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[Peter Mayo, 2016-04-12 17:22:02]

petermayo@chromium.org changed reviewers:
+ vollick@chromium.org

[Ian Vollick, 2016-04-12 17:30:58]

On 2016/04/12 17:22:03, Peter Mayo wrote:

Nice! lgtm if the bots are happy.

[Peter Mayo, 2016-04-12 18:29:44]

The CQ bit was checked by petermayo@chromium.org

[commit-bot: I haz the power, 2016-04-12 18:29:59]

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

[commit-bot: I haz the power, 2016-04-12 18:37:49]

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

[commit-bot: I haz the power, 2016-04-12 18:37:50]

Try jobs failed on following builders:
  cast_shell_android on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)

[Peter Mayo, 2016-04-13 14:20:09]

The CQ bit was checked by petermayo@chromium.org

[commit-bot: I haz the power, 2016-04-13 14:20:19]

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

[commit-bot: I haz the power, 2016-04-13 14:37:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-13 14:39:12]

Description was changed from

==========
Correct normal computation for inverted polygons.

Rather than rely solely the vertices, and their winding
order for sign, we rely on the transform of the previous
normal for the sign, and the vertices (cross product of
edge vectors, similar to Newell's algorithm) for the
numerics of the direction.

BUG=595820
TEST=unittest
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Correct normal computation for inverted polygons.

Rather than rely solely the vertices, and their winding
order for sign, we rely on the transform of the previous
normal for the sign, and the vertices (cross product of
edge vectors, similar to Newell's algorithm) for the
numerics of the direction.

BUG=595820
TEST=unittest
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/59d3768dd104c22554d86f410d22ad8f714c82eb
Cr-Commit-Position: refs/heads/master@{#386984}
==========

[commit-bot: I haz the power, 2016-04-13 14:39:13]

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