WebIDL impl: Replace use of NonNull<JSObject> about servo HOT 4 OPEN

This seems reasonable to me! I'm not sure why we didn't integrate the concrete types originally.

from servo.

@jdm Does this make sense? Do you know why this wasn't done to begin with?

from servo.

Had a quick look everywhere, it seems there are really four cases:

1. Should return some variant of js::typedarray (the majority). Dealt with in this issue
2. Should return a ReadableStream, see #30891
3. Should return an enum, see #30890
4. Should return a WebGL extension, which is actually an object? in WebIDL. See #30892

from servo.

See also https://github.com/servo/mozjs/blob/de842d84a07eabac4e1bfe72a954b74ee0244b82/mozjs/src/conversions.rs#L722

It may be that what is required is simply implementing ToJSValConvertible for the various types in js::rust::typedarray

In some ways it would in that case still be nicer to wrap the use of js:: inside dom/bindings...

from servo.