Re: Intent to unship xml:base

2018-11-29 Thread Jonathan Kingston
Sorry I forgot to link to the bug itself: https://bugzilla.mozilla.org/show_bug.cgi?id=903372 which hasn't changed from last time. On Thu, Nov 29, 2018 at 8:18 PM Jonathan Kingston wrote: > > Since the deprecation of this feature the usage of this feature has > reached near 0 numbers in our

Intent to unship xml:base

2018-11-29 Thread Jonathan Kingston
Since the deprecation of this feature the usage of this feature has reached near 0 numbers in our usage counters[1]. 0.003% based on 64 beta [2] The specification has removed xml:base many years ago and this change has an advantage of code simplification and also reduces the need to compute base

Re: Intent to unship: xml:base attribute

2017-05-23 Thread Xidorn Quan
On Wed, May 24, 2017, at 11:30 AM, Jet Villegas wrote: > xml:base (bug 1349024) has been removed in Nightly 55 for 2 months > now, and we haven't sen any reports of ill effects. Let's have this > testing expand to Beta 55, and on to Release if all goes well. Bug > 1350521 tracks this change riding

Re: Intent to unship: xml:base attribute

2017-05-23 Thread Jet Villegas
xml:base (bug 1349024) has been removed in Nightly 55 for 2 months now, and we haven't sen any reports of ill effects. Let's have this testing expand to Beta 55, and on to Release if all goes well. Bug 1350521 tracks this change riding the trains. --Jet On Thu, Feb 16, 2017 at 2:51 PM, Xidorn

Re: Intent to unship: xml:base attribute

2017-02-17 Thread Ehsan Akhgari
On 2017-02-16 6:06 PM, Xidorn Quan wrote: > On Fri, Feb 17, 2017, at 01:38 AM, Ehsan Akhgari wrote: >> On 2017-02-16 1:51 AM, Xidorn Quan wrote: >>> Tracking bug: https://bugzilla.mozilla.org/show_bug.cgi?id=903372 >>> >>> Summary: >>> * It has been removed from the spec years ago. >>> * No other

Re: Intent to unship: xml:base attribute

2017-02-16 Thread Boris Zbarsky
On 2/16/17 5:52 PM, Xidorn Quan wrote: Is there any reason you think we should keep the Get/SetExplicitBaseURI API? As long as we can address its current users in some other way, no. -Boris ___ dev-platform mailing list

Re: Intent to unship: xml:base attribute

2017-02-16 Thread Xidorn Quan
On Fri, Feb 17, 2017, at 01:38 AM, Ehsan Akhgari wrote: > On 2017-02-16 1:51 AM, Xidorn Quan wrote: > > Tracking bug: https://bugzilla.mozilla.org/show_bug.cgi?id=903372 > > > > Summary: > > * It has been removed from the spec years ago. > > * No other browser supports it. > > * We pay

Re: Intent to unship: xml:base attribute

2017-02-16 Thread Xidorn Quan
On Fri, Feb 17, 2017, at 01:36 AM, Boris Zbarsky wrote: > On 2/16/17 7:12 AM, Xidorn Quan wrote: > > The perf penalty of xml:base is basically that we have to dynamically > > construct a URL from bottom to top along the tree whenever we need a > > base URL of an element. And since this URL may be

Re: Intent to unship: xml:base attribute

2017-02-16 Thread Boris Zbarsky
On 2/16/17 7:12 AM, Xidorn Quan wrote: The perf penalty of xml:base is basically that we have to dynamically construct a URL from bottom to top along the tree whenever we need a base URL of an element. And since this URL may be constructed dynamically, the caller would have to hold a strong

Re: Intent to unship: xml:base attribute

2017-02-16 Thread Ehsan Akhgari
On 2017-02-16 1:51 AM, Xidorn Quan wrote: > Tracking bug: https://bugzilla.mozilla.org/show_bug.cgi?id=903372 > > Summary: > * It has been removed from the spec years ago. > * No other browser supports it. > * We pay performance penalty for it. > * It makes things trickier for stylo to handle URL

Re: Intent to unship: xml:base attribute

2017-02-16 Thread Xidorn Quan
On Thu, Feb 16, 2017, at 10:25 PM, Gijs Kruitbosch wrote: > On 16/02/2017 06:51, Xidorn Quan wrote: > > Tracking bug: https://bugzilla.mozilla.org/show_bug.cgi?id=903372 > > > > Summary: > > * It has been removed from the spec years ago. > > * No other browser supports it. > > * We pay performance

Re: Intent to unship: xml:base attribute

2017-02-16 Thread Gijs Kruitbosch
On 16/02/2017 06:51, Xidorn Quan wrote: Tracking bug: https://bugzilla.mozilla.org/show_bug.cgi?id=903372 Summary: * It has been removed from the spec years ago. * No other browser supports it. * We pay performance penalty for it. * It makes things trickier for stylo to handle URL values. The

Intent to unship: xml:base attribute

2017-02-15 Thread Xidorn Quan
Tracking bug: https://bugzilla.mozilla.org/show_bug.cgi?id=903372 Summary: * It has been removed from the spec years ago. * No other browser supports it. * We pay performance penalty for it. * It makes things trickier for stylo to handle URL values. The tricky thing is that