Re: [heycam/webidl] Fix internal methods that only target platform objects w/ indexed/named props (#202)

> https://github.com/heycam/webidl/issues/202#issuecomment-256049412

That's fair. 

> I guess you could go for n = 2 (platform objects and named-or-indexed platform objects), or even n = 4 (those two multiplied by legacy-caller-or-not)

legacycaller is used in three places in Gecko: HTMLObjectElement, HTMLEmbedElement, HTMLAllCollection.  Those are also the only places it's used in the HTML spec.  I'm about to remove the former two (see <https://bugzilla.mozilla.org/show_bug.cgi?id=909656>), because we have near-zero telemetry (2 hits out of 460+ million telemetry pings) _and_ we just dropped support for non-Flash plugins and I'm fairly certain Flash doesn't use this stuff.  I filed https://github.com/whatwg/html/issues/1971 to remove from the spec too.  Of course HTMLAllCollection already has indexed/named props.

So I think we should do n == 2, change IDL to require that legacycaller is only specified if indexed or named props are present, and then we only need interesting internal methods for the indexed/named prop case.  We can continue hating HTMLAllCollection, of course.  ;)

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/heycam/webidl/issues/202#issuecomment-256055117

Received on Tuesday, 25 October 2016 14:43:23 UTC