[css-houdini-drafts] partial interface AnimationEffect exposure set doesn't match original defintion

majido has just created a new issue for https://github.com/w3c/css-houdini-drafts:

== partial interface AnimationEffect exposure set doesn't match original defintion ==
_From @foolip on August 13, 2018 14:6_

https://wicg.github.io/animation-worklet/#worklet-group-effect has:
```webidl
[Exposed=AnimationWorklet]
partial interface AnimationEffect {
    // Intended for use inside Animation Worklet scope to drive the effect.
    attribute double localTime;
};
```

And https://drafts.csswg.org/web-animations/#the-animationeffect-interface has:
```webidl
[Exposed=Window]
interface AnimationEffect {
    EffectTiming         getTiming();
    ComputedEffectTiming getComputedTiming();
    void                 updateTiming(optional OptionalEffectTiming timing);
};
```

This combination is not valid because https://heycam.github.io/webidl/#Exposed say: "If [Exposed] appears on a partial interface or partial namespace, then the partial’s own exposure set must be a subset of the exposure set of the partial’s original interface or namespace."

In other words, `[Exposed=AnimationWorklet]` needs to be added to https://drafts.csswg.org/web-animations/, or some change made in this spec.

@lukebjerring, I discovered this thanks to a failing "Partial AnimationEffect interface is exposed to 'AnimationWorklet', the original interface is not." test when reviewing https://github.com/web-platform-tests/wpt/pull/12439.

_Copied from original issue: WICG/animation-worklet#108_

Please view or discuss this issue at https://github.com/w3c/css-houdini-drafts/issues/816 using your GitHub account

Received on Wednesday, 29 August 2018 18:20:21 UTC