Replace explicit checks for DeveloperErrors with calls to the new Check type

[pjcozzi]

#4726 introduced a new private class, Check, with static utility functions to do common parameter validation and throw a DeveloperError. Using Check instead of explicit hand-coded conditionals is more concise. For example:

if (!defined(cartesian)) {
    throw new DeveloperError('cartesian is required');
}

becomes

Check.typeOf.object('cartesian', cartesian);

#4726 also updated Cartesian3 to use Check.

We still want to update the rest of the Cesium codebase to use Check when reasonable (very specific checks for DeveloperErrors can still be hand-coded). This is a great beginner issue.

• Start with the fundamental numeric types in the Core directory: cartesian, matrices, quaternions, etc.
• Open a pull request for one file at a time to start.
• Verify the reference documentation for the @exceptions is still correct and that there are unit tests for the exceptions in the corresponding Spec file (try the WebStorm plugin to flip between the source and spec file).

[pjcozzi]

@shehzan10

• Matrix2
• Matrix4
• Cartesian4
• Spherical

@austinEng

• Matrix3
• Cartesian2
• Quaternion
• Rectangle

All of these files are in the Core directory.

[lilleyse]

@WilliamKHo @moneimne @omh1280 @AnimatedRNG Let's work on these to start. Open a separate pull request for each file to start.

@rahwang would you like to do the first round of reviews for these?

All these files are in the Core directory. For any checks that involve arrays, just use Check.defined as there is not a Check.typeOf.array. Some checks may also require using Check.typeOf.number.lessThan and related functions.

@WilliamKHo

• arrayRemoveDuplicates
• AttributeCompression (fix any of the "____ is required" checks, skip others)
• AxisAlignedBoundingBox
• barycentricCoordinates

@moneimne

• binarySearch
• BingMapsGeocoderService
• BoxGeometry
• BoxOutlineGeometry

@omh1280

• Cartesian2 (fix just the "left and right are required" checks)
• Cartesian3 (fix just the "____ is required" checks)
• Cartographic
• CartographicGeocoderService

@AnimatedRNG

• Event
• Color
• Ellipsoid - skip "Ellipsoid must be an ellipsoid of revolution (radii.x == radii.y)"
• Geometry - skip "All attribute lists must have the same number of attributes"

[rahwang]

I'd be happy to @lilleyse !

@WilliamKHo @moneimne @omh1280

When you open a pull request for any of these, please tag. Ping if you need help!

[ottaviohartman]

@rms13

• PolygonPipeline
• GeographicTilingScheme
• sampleTerrain
• EncodedCartesian3
• TimeInterval
• Fix home button to work with 2D view (Fixed the home button to work with 2D view #5561)

@Rudraksha20

• TileAvailability
• CatmullRomSpline
• loadKTX
• EllipsoidalOccluder
• Add Check.typeOf.number.equals function (Updated check with Check.typeOf.number.equals #5555)

[ggetz]

This is unlikely to become a priority anytime soon. We'd still happily accept a PR addressing this issue. But tracking this isn't particularly useful because of its low priority, so I will close it. Thanks!