Fix aabb.fromPoints API

[karimnaaji]

Issue discovered while working on #12138. Fixes a regression introduced by 7398fb3 for a function used as part of the tile cover process with globe.

const aabbMin = [Number.MAX_VALUE, Number.MAX_VALUE, Number.MAX_VALUE];
const aabbMax = [Number.MIN_VALUE, Number.MIN_VALUE, Number.MIN_VALUE];

uses MIN_VALUE (0) instead of INFINITY, this gives the wrong values when the AABB bounds are negative. e.g., with an AABB with points like that:

[-10, -5, 10, 5]

the bounds gets clamped like this:

[0, 10]

instead of:

[-10, 10]

Launch Checklist

• briefly describe the changes in this PR
• write tests for all new functionality
• post benchmark scores
• manually test the debug page
• apply changelog label ('bug', 'feature', 'docs', etc) or use the label 'skip changelog'
• add an entry inside this element for inclusion in the mapbox-gl-js changelog: <changelog>Fix invalid AABB calculation as part of the globe tile cover</changelog>

[SnailBones]

Nice work Karim!

One small nit: perhaps it would be better to update the images instead of increasing tolerance, since the subtle change in appearance was introduced by this bug, no?

The lines also look a bit cleaner in the generated image.

[karimnaaji]

I was worried this change would necessitate a patch release for v2.10.0 as it was already released but this regression did not introduce any measurable performance regression, as benchmarks did not show any difference in top-level rendering metrics with the globe:

v2.10.0 vs v2.9.0:

main vs karim/fix-aabb-from-points: