Assigning a value outside the float range to a 'float' attribute should throw a TypeError, as per:
Blink follows the specification but WebKit just casts the double to a float.
Created attachment 431723 [details]
We might want to fix the canvas IDLs still using float that should be double before doing this to avoid accidental fallout.
It looks like almost all uses of floats in our canvas IDLs are `unrestricted float`, which will at least saturate out of range Number values to +/-Infinity.
(In reply to Cameron McCormack (:heycam) from comment #3)
> It looks like almost all uses of floats in our canvas IDLs are `unrestricted
> float`, which will at least saturate out of range Number values to
Right, the behavior change in this patch for unrestricted float is not large and a strict progression. I would be more concerned about APIs where we use (restricted) float instead of double.
Fair (just trying to bully people into fixing that because I keep forgetting to).
Committed r279047 (238967@main): <https://commits.webkit.org/238967@main>
All reviewed patches have been landed. Closing bug and clearing flags on attachment 431723 [details].