base polygon geometry rectangle IDL handling on Rectangle.fromCartesianArray

[likangning93]

@shehzan10 noticed that geodetic polygons had weird rectangles sometimes when crossing the PM.

Here in master

This PR updates the IDL handling for geodetic polygons to more closely match what Rectangle.fromCartesianArray does, which seems to be correct.

[cesium-concierge]

Thanks for the pull request @likangning93!

• ✔️ Signed CLA found.
• ❔ CHANGES.md was not updated.
  • If this change updates the public API in any way, please add a bullet point to CHANGES.md.
• ❔ Unit tests were not updated.
  • Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

• Cesium Viewer works.
• Works in 2D/CV.
• Works (or fails gracefully) in IE11.

[likangning93]

Fixes a bug introduced in this release, so no CHANGES.md update.

[hpinkos]

@likangning93 should there be a unit test for this?

[likangning93]

There's an IDL test case already, the corner case @shehzan10 found was a false-positive for IDL cross. I can add the cases from RectangleSpec to make sure coverage is similar.

[ggetz]

Thanks @likangning93 this fixes the issue. We can get it in if you fix up those tests soon.

[likangning93]

There's an IDL test case already

Looks like the previous case was also only for rhumb, whoops.

[likangning93]

Okay looks like eslint 5.13.0 is breaking things everywhere: https://github.com/eslint/eslint/issues things broke due to eslint 5.13.0, holding off on this until next release...

[likangning93]

@ggetz @hpinkos updated!

[hpinkos]

Looks great, thanks @likangning93!