On Tuesday, 19 April 2016 22:03:48 UTC+9, smaug wrote: > > This is a temporary situation because I don't want to have to back out the > > whole Element.animate feature due to compat issues from the 'Animation' > > interface name. After we have shipped Element.animate we can ship the > > Animation constructor behind a different pref. > Why would you need to backout animate() if the interface was > [NoInterfaceObject] ?
This was referring to the alternative of shipping the constructor under the same pref as the rest of the Element.animate implementation. If we do that and compatibility issues with 'Animation' arise late in the game, we'd have to turn off the pref and effectively unship the whole Element.animate feature. > So could we ship constructor + removing [NoInterfaceObject] as a separate > step, not when shipping this initial API? And have [NoInterfaceObject] until > that. If we're worried about web-compatibility, the API spec editors should > consider changing the spec to be more likely web-compatible, and better to > do that before anyone supports the constructor (since I assume it might mean > changing the interface name) We could, however we have quite a few tests that rely on this constructor so we'd end up being unable to test a lot of things. I wouldn't want to make it [NoInterfaceObject] on trunk, at least, but perhaps we could do that on aurora/beta, for example. If we do discover compatibility issues, I'll gladly update the spec. And just to clarify, there are no specific compatibility issues that I am aware of--just the fact the Animation is a fairly generic name. At this point it seems like it depends on how likely we are to hit compatibility issues. I'll see if the blink folks have any data on this. If it seems like the compatibility risk is low, then we should just ship the constructor from the start and take the risk that we might have to disable Element.animate. _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
