W3C home > Mailing lists > Public > www-dom@w3.org > January to March 2014

Re: Change to Attr interface in light of XPath

From: Jonas Sicking <jonas@sicking.cc>
Date: Tue, 18 Mar 2014 13:25:26 -0700
Message-ID: <CA+c2ei-NHNNspJ8y1YHuiAQ67-9AURsTXJ903L=GCoruXJ5eWA@mail.gmail.com>
To: Anne van Kesteren <annevk@annevk.nl>
Cc: Boris Zbarsky <bzbarsky@mit.edu>, DOM public list <www-dom@w3.org>
On Tue, Mar 18, 2014 at 10:33 AM, Anne van Kesteren <annevk@annevk.nl> wrote:
> On Tue, Feb 18, 2014 at 5:20 PM, Jonas Sicking <jonas@sicking.cc> wrote:
>> attributenode.parentNode has always returned null. In all versions of
>> the spec and in all implementations. What you want is
>> attributenode.ownerElement. I'm surprised that it's been removed from
>> the new spec. I agree that it needs to be put back.
>
> Why?
>
> It returns undefined in all implementations. (I didn't test this and
> just went by your word initially, reverting that now.)

I guess that at some point Gecko and other browsers removed this
property since it was removed from the spec.

However that still leaves us without a plan to deprecate Attr objects
(https://www.w3.org/Bugs/Public/show_bug.cgi?id=25016), but with those
Attr objects being pretty broken since there's no way to get back to
their "parent".

I guess since the web didn't break when we removed Attr.ownerElement
we won't break that many people by leaving it out. But it does mean
that we have created a pretty awkward API where you can get to Attr
objects, but you can't get back.

The only real thing that I can think of that we've broken is the DOM
XPath API, which isn't much in favor any more, but I have no idea how
much usage it has (more than XSLT almost certainly).

On the other hand, it seems somewhat silly to remove ownerElement
since the implementation needs to hold a reference from the Attr
object to the Element anyway in order to implement the .value setter.

/ Jonas
Received on Tuesday, 18 March 2014 20:26:23 UTC

This archive was generated by hypermail 2.3.1 : Tuesday, 20 October 2015 10:46:22 UTC