Intent to unship: SMIL accessKey support

[Brian Birtles]

Background: SMIL includes a feature for triggering animations based on
keypresses:

e.g.

<rect width="100" height="100" fill="blue">
<animate begin="accessKey(s)" attributeName="opacity" to="0" dur="1s" />
</rect>

Proposal: In bug 1423098 I intend to remove this feature.

Rationale:

* Apart from Gecko, only Presto supports it.
* accessKey has been the source of security issues in the past such as
bug 704482.
* We're trying to reduce the amount of SMIL-specific code in Gecko,
since it is not used a lot, it increases our attack surface area, and
it slows down other work on the style system like the Stylo project.

Status in other browsers:

Blink: No support. There is a "// FIXME: accesskey() support" in the code.[1]
WebKit: No support. The FIXME appears to be pre-fork.[2]
IE/Edge: No SVG SMIL support at all.
Presto: Supports accessKey but you need to press Shift + Esc first.

I'm not planning on adding a developer console warning at this point
unless someone suggests that would be useful.

Best regards,

Brian

[1] https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/svg/animation/SVGSMILElement.cpp?q=syncbase+file:%5Esrc/+package:%5Echromium$&dr=CSs&l=465
[2] https://trac.webkit.org/browser/webkit/trunk/Source/WebCore/svg/animation/SVGSMILElement.cpp#L425

[Frederik Braun]

Excellent! I won't miss it :)

> _______________________________________________
> dev-platform mailing list
> dev-pl...@lists.mozilla.org
> https://lists.mozilla.org/listinfo/dev-platform
>