Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

domxpath/interfaces.html has no spec to back it up #7562

Closed
bzbarsky opened this issue Oct 3, 2017 · 3 comments
Closed

domxpath/interfaces.html has no spec to back it up #7562

bzbarsky opened this issue Oct 3, 2017 · 3 comments
Labels

Comments

@bzbarsky
Copy link
Contributor

bzbarsky commented Oct 3, 2017

This file has IDL that is not backed by any actual spec; it was just checked in on someone's say-so as far as I can tell.

In addition to that, the IDL it uses is about to become invalid (assuming it's valid already, which is not clear to me) when whatwg/webidl#433 lands.

@Ms2ger
Copy link
Contributor

Ms2ger commented Oct 4, 2017

That would be my say-so :)

The closest we have to a spec for DOM XPath is https://wiki.whatwg.org/wiki/DOM_XPath; having something better would be great, but priorities... I think it's useful to have this test as a regression test, though. If I'm not mistaken, Gecko currently uses the exact same syntax; perhaps whoever fixes Gecko could just update the test as well? I can update the wiki if you cc me on the bug.

@bzbarsky
Copy link
Contributor Author

bzbarsky commented Oct 4, 2017

We could use our "tentative" or whatever it is naming convention...

I don't have a timeframe for Gecko updating to the new mixin syntax, and more importantly that timeframe may not match idlharness's.

@Ms2ger
Copy link
Contributor

Ms2ger commented Oct 4, 2017

CC @tobie @darobin

@foolip foolip added the domxpath label Oct 4, 2017
@gsnedders gsnedders changed the title domxpath/interfaces.html has not spec to back it up domxpath/interfaces.html has no spec to back it up Oct 5, 2017
foolip added a commit that referenced this issue Oct 9, 2017
gsnedders pushed a commit that referenced this issue Oct 9, 2017
jakearchibald pushed a commit to jakearchibald/web-platform-tests that referenced this issue Nov 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants