Replace defineProperties with native Object.defineProperties

[fredj]

Related to #8515

Object.defineProperties is supported since IE 9 (ES5)

I've neither updated CHANGES.md nor deprecated the defineProperties polyfill because it was private.

eslint is now correctly checking if the getters are returning a value so I had to add few eslint-disable-line (211148e)

[cesium-concierge]

Thanks for the pull request @fredj!

• ✔️ 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.

[mramato]

@shunter any arguments against here? I know the original reason for this polyfill was so CesiumJS could fail gracefully on IE < 11, but at this point those browsers should no longer be a concern.

[hpinkos]

@mramato @shunter threw his two cents in in this PR #8526 (comment)

I think we can probably merge both if you agree

[mramato]

Thanks @hpinkos

And thanks @fredj!

[nmschulte]

I've neither updated CHANGES.md nor deprecated the defineProperties polyfill because it was private.

Ouch.