[webkit-dev] Remove SVGTests.hasExtension?

[Ryosuke Niwa]

The concern here isn't with the future versions of Edge or Firefox but
rather with the content that may be relying on this feature even
unintentionally.
Removing a JS method or object is very high risk because that could result
in an immediate JS exception and break a key functionality of a website.

This is why HTML and DOM standards has a bunch of methods that don't do
anything like Range's detach: https://dom.spec.whatwg.org/#dom-range-detach

- R. Niwa

On Fri, Jun 7, 2019 at 11:54 AM Frédéric Wang <fwang at igalia.com> wrote:

> Patch for Firefox was uploaded 4 years ago and they announced plan for
> removal at that time. We pinged Mozilla some weeks ago about it, they
> landed the patch and announced it:
>
> https://bugzilla.mozilla.org/show_bug.cgi?id=1133175
>
> https://www.fxsitecompat.com/en-CA/docs/2019/hasextension-has-been-removed-from-some-svg-interfaces/
>
> https://www.fxsitecompat.com/en-CA/docs/2015/several-properties-will-be-removed-from-svgsvgelement/
>
> I didn't check Edge, but Microsoft is migrating to Chromium so that's not
> going to be there in future versions.
>
> On 07/06/2019 20:40, Ryosuke Niwa wrote:
>
> Does Edge support it? When did Firefox remove this feature?
>
> On Fri, Jun 7, 2019 at 1:55 AM Frédéric Wang <fwang at igalia.com> wrote:
>
>> Hi,
>>
>> 4 years ago, the SVG WG resolved to remove SVGTests.hasExtension due to lack of use and being a poor API.
>> It was removed from Chromium at that time and has been removed from Firefox recently.
>>
>> I'm not sure whether the WebKit community is willing to keep that feature. Anyway I opened a bug entry to track possible removal:
>> https://bugs.webkit.org/show_bug.cgi?id=198652
>>
>> --
>> Frédéric Wang
>>
>> _______________________________________________
>> webkit-dev mailing list
>> webkit-dev at lists.webkit.org
>> https://lists.webkit.org/mailman/listinfo/webkit-dev
>>
> --
> - R. Niwa
>
>
> --
> Frédéric Wang
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20190607/2502150d/attachment.html>