WedIDL: partial interface definition cannot specify inheritance

[mwatson2]

We have:

partial interface HTMLMediaElement : EventTarget { ... };

however according to https://heycam.github.io/webidl/#dfn-partial-interface a partial interface definition cannot specify inheritance. HTMLMediaElement inherits Element which inherits Node which inherits EventTarget, so we do not need to specify this.

[jdsmith3000]

@mwatson2: I believe the current spec here may be correct. It defines HTMLMediaElement as a partial interface so that we can make interface additions in a separate document. I don't believe an interface can do this. The limitation on inheritance may apply only to inheritance rules we define ourselves, and we don't have any of these.

[mwatson2]

@jdsmith3000 What I meant is that we do not need to include the : EventTarget. WebIDL doesn't allow us to specify this on a partial interface and the original definition of HTMLMediaElement already has EventTarget in the inheritance chain.

[jdsmith3000]

@mwatson2: Makes sense. I'll remove the EventTarget.

[ddorwin]

@jdsmith3000's commit above needs to be converted to a PR against mainline. It looks like this is an easy fix and merge.

[jdsmith3000]

I was preparing to do this, but found that the latest editors draft already fixes this problem. It is possible it was corrected as part of Issue #170 "Switch to Contiguous IDL". It is at least present in that version of the spec.

I'm closing this issue as fixed.

[mwatson2]

Apologies - I forgot I needed to fix this to get the IDL to compile in Contiguous IDL mode.