Revert of Perform BSP polygon splitting and orientation selection in a single step.

Revert of Perform BSP polygon splitting and orientation selection in a single step. (patchset #4 id:60001 of https://codereview.chromium.org/2043283002/ )

Reason for revert:
Broke Windows 7 cc_unittests: DrawPolygonSplitTest.AngledSplit
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%281%29/builds/53692

Original issue's description:
> Perform BSP polygon splitting and orientation selection in a single step.
>
> This eliminates redundant testing of vertices for orientation with
> respect to the splitting polygon. Previously, up to 3 sets of tests
> were made (once to determine whether the polygon was split, then once
> during the split, and then finally to determine the orientation of the
> split polygons.
>
> Merging these steps in order to reuse calculated values also eliminates
> the possibility that different calculations in testing and splitting
> could be inconsistent.
>
> BUG=606984
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/89c62c8f69d76471e866f21ee4f1ae5e0c5bca48
> Cr-Commit-Position: refs/heads/master@{#399459}

TBR=enne@chromium.org,tobiasjs@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=606984
Committed: https://crrev.com/2d02c4132568ac72b26c7198ad112b19ba75cd95
Cr-Commit-Position: refs/heads/master@{#399472}

[Fady Samuel, 2016-06-13 17:21:56]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-06-13 17:21:56]

Created Revert of Perform BSP polygon splitting and orientation selection in a
single step.

[commit-bot: I haz the power, 2016-06-13 17:22:40]

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

[commit-bot: I haz the power, 2016-06-13 17:23:27]

Description was changed from

==========
Revert of Perform BSP polygon splitting and orientation selection in a single
step. (patchset #4 id:60001 of https://codereview.chromium.org/2043283002/ )

Reason for revert:
Broke Windows 7 cc_unittests: DrawPolygonSplitTest.AngledSplit
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%281%29/bui...

Original issue's description:
> Perform BSP polygon splitting and orientation selection in a single step.
> 
> This eliminates redundant testing of vertices for orientation with
> respect to the splitting polygon. Previously, up to 3 sets of tests
> were made (once to determine whether the polygon was split, then once
> during the split, and then finally to determine the orientation of the
> split polygons.
> 
> Merging these steps in order to reuse calculated values also eliminates
> the possibility that different calculations in testing and splitting
> could be inconsistent.
> 
> BUG=606984
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/89c62c8f69d76471e866f21ee4f1ae5e0c5bca48
> Cr-Commit-Position: refs/heads/master@{#399459}

TBR=enne@chromium.org,tobiasjs@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=606984
==========

to

==========
Revert of Perform BSP polygon splitting and orientation selection in a single
step. (patchset #4 id:60001 of https://codereview.chromium.org/2043283002/ )

Reason for revert:
Broke Windows 7 cc_unittests: DrawPolygonSplitTest.AngledSplit
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%281%29/bui...

Original issue's description:
> Perform BSP polygon splitting and orientation selection in a single step.
> 
> This eliminates redundant testing of vertices for orientation with
> respect to the splitting polygon. Previously, up to 3 sets of tests
> were made (once to determine whether the polygon was split, then once
> during the split, and then finally to determine the orientation of the
> split polygons.
> 
> Merging these steps in order to reuse calculated values also eliminates
> the possibility that different calculations in testing and splitting
> could be inconsistent.
> 
> BUG=606984
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/89c62c8f69d76471e866f21ee4f1ae5e0c5bca48
> Cr-Commit-Position: refs/heads/master@{#399459}

TBR=enne@chromium.org,tobiasjs@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=606984
==========

[commit-bot: I haz the power, 2016-06-13 17:23:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-13 17:23:30]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-13 17:24:42]

Description was changed from

==========
Revert of Perform BSP polygon splitting and orientation selection in a single
step. (patchset #4 id:60001 of https://codereview.chromium.org/2043283002/ )

Reason for revert:
Broke Windows 7 cc_unittests: DrawPolygonSplitTest.AngledSplit
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%281%29/bui...

Original issue's description:
> Perform BSP polygon splitting and orientation selection in a single step.
> 
> This eliminates redundant testing of vertices for orientation with
> respect to the splitting polygon. Previously, up to 3 sets of tests
> were made (once to determine whether the polygon was split, then once
> during the split, and then finally to determine the orientation of the
> split polygons.
> 
> Merging these steps in order to reuse calculated values also eliminates
> the possibility that different calculations in testing and splitting
> could be inconsistent.
> 
> BUG=606984
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/89c62c8f69d76471e866f21ee4f1ae5e0c5bca48
> Cr-Commit-Position: refs/heads/master@{#399459}

TBR=enne@chromium.org,tobiasjs@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=606984
==========

to

==========
Revert of Perform BSP polygon splitting and orientation selection in a single
step. (patchset #4 id:60001 of https://codereview.chromium.org/2043283002/ )

Reason for revert:
Broke Windows 7 cc_unittests: DrawPolygonSplitTest.AngledSplit
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%281%29/bui...

Original issue's description:
> Perform BSP polygon splitting and orientation selection in a single step.
>
> This eliminates redundant testing of vertices for orientation with
> respect to the splitting polygon. Previously, up to 3 sets of tests
> were made (once to determine whether the polygon was split, then once
> during the split, and then finally to determine the orientation of the
> split polygons.
>
> Merging these steps in order to reuse calculated values also eliminates
> the possibility that different calculations in testing and splitting
> could be inconsistent.
>
> BUG=606984
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/89c62c8f69d76471e866f21ee4f1ae5e0c5bca48
> Cr-Commit-Position: refs/heads/master@{#399459}

TBR=enne@chromium.org,tobiasjs@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=606984

Committed: https://crrev.com/2d02c4132568ac72b26c7198ad112b19ba75cd95
Cr-Commit-Position: refs/heads/master@{#399472}
==========

[commit-bot: I haz the power, 2016-06-13 17:24:43]

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