849637 - Follow up 841493 comment 26

Status: NEW

(Core :: DOM: Core & HTML, defect, P5)

Description

[Andrea Marchesini [:baku]]

@@ +213,5 @@
>    if (JS_IsFloat32Array(darray)) {
>      tsrc = darray;
>    } else if (JS_IsArrayObject(aCx, darray)) {
>      JSObject* nobj = JS_NewFloat32ArrayFromArray(aCx, darray);
>      if (!nobj) {

Ugh. Definitely want a followup for this crap, if you don't want to do it in this bug.

Comment 1

[Boris Zbarsky [:bzbarsky]]

That's bug 841493 comment 26.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Yeah, this should use an overload with sequence<unrestricted float> and Float32Array or something....

Of course the XPCOM version will still need the goop.  Unless we just remove it.

Comment 3

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1472046

Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.