No mixin [NoInterfaceObject] usage

[annevk]

Per #423 (comment) this exists. I'd like to study such usage and see if we can get rid of it.

[tobie]

Here's a manually filtered list I build for #363 based on running awk on Gecko's src files:

• Geolocation
• Coordinates
• Position
• PositionError
• DeviceAcceleration
• DeviceRotationRate
• ConstrainablePattern
• WEBGL_compressed_texture_astc
• WEBGL_compressed_texture_s3tc_srgb
• WEBGL_draw_buffers.
• WEBGL_lose_context
• ANGLE_instanced_arrays
• EXT_blend_minmax
• EXT_color_buffer_float
• EXT_disjoint_timer_query
• OES_standard_derivatives
• OES_vertex_array_object.

[annevk]

Thanks, seems like we could just drop the keyword from those and be done with it. I guess part of the problem is some of those specifications not being maintained, but that can be fixed one way or another.

[tobie]

Well, we'd want to make sure suddenly exposing these interfaces to the Web doesn't break stuff, no?

[annevk]

Yeah, I suppose the ones with overly generic names should be prefixed, but seems safe otherwise.

[tobie]

Prefixing is an observable change. No idea if it would have real impact, though.

[annevk]

How is it observable?

[tobie]

navigator.geolocation.getCurrentPosition(p => console.log(Object.prototype.toString.call(p)));
// [object Position]

[annevk]

Sigh. Part of the reason we had [NoInterfaceObject] was the ability to ship objects and change the name later. I guess that's another reason we should really get rid of it (and decide on names we can live with upfront).

[annevk]

WebGL seems fixed. I filed issues against two other specifications I identified from the list above. Anything else?

[domenic]

Searching through Blink, External in the HTML spec needs fixing. There's also w3c/mediacapture-main#501 which is weird.

[domenic]

Also https://w3c.github.io/webappsec-feature-policy/#featurepolicy ... a little hard to tell since Blink doesn't have interface mixin syntax yet, so lots of stuff is mixed together.

[bzbarsky]

Fwiw, that's why I haven't looked at this in Gecko either. It'll be a lot simpler once we implement mixin syntax...