Sanitize tangents when creating mesh surfaces to avoid triggering the compressed mesh path in the shader

[clayjohn]

Fixes: #83175

This was a fun one to track down.

In the shader we check if (tangent.x > 0 || tangent.y < 1.0), to see if we should run the uncompressed path. I wrongly assumed that we would never send (0, 1) to the shader because (0, 1) maps to a tangent of (0, 0, -1, 1), but a tangent of (0, 0, -1, 1) maps to (1, 1). However, (-1e8, 0, -1, 1) maps to (0, 1) and sometimes we create tangents that are (-1e8, 0, -1, 1). We can't represent (-1e8, 0, -1, 1) in octahedral tangents anyway, so its fine to just force all cases of (0, 1) to (1, 1)

[akien-mga]

That looks pretty hacky, but I trust that it's the correct solution :)

[akien-mga]

Thanks!