Re: Intent to unship: SMIL accessKey support

2017-12-05 Thread Frederik Braun
Excellent! I won't miss it :)

On 05.12.2017 08:25, Brian Birtles wrote:
> Background: SMIL includes a feature for triggering animations based on
> keypresses:
> 
> e.g.
> 
> 
>   
> 
> 
> 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$=CSs=465
> [2] 
> https://trac.webkit.org/browser/webkit/trunk/Source/WebCore/svg/animation/SVGSMILElement.cpp#L425
> ___
> dev-platform mailing list
> dev-platform@lists.mozilla.org
> https://lists.mozilla.org/listinfo/dev-platform
> 
___
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform


Intent to unship: SMIL accessKey support

2017-12-04 Thread Brian Birtles
Background: SMIL includes a feature for triggering animations based on
keypresses:

e.g.


  


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$=CSs=465
[2] 
https://trac.webkit.org/browser/webkit/trunk/Source/WebCore/svg/animation/SVGSMILElement.cpp#L425
___
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform