Bug 47166 – querySelectorAll('body>:last-child') does not work unless HTML file ends with 'new-line'.

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Hayato Ito]

Created attachment 69768 [details]
test1.html

Please see attachment files, test1.html and test2.html. The only difference between them is whether there is a 'new-line' at the end of file or not.

- test1.html ends with 'new-line', like '... </html>\n<EOF>'
- test2.html does not end with 'new-line', like '.... </html><EOF>'.

Open each files, then try the following JavaScript code in JavaScript console.

 > document.querySelectorAll('body>:last-child')

The results differs in test1.html and test2.html:
  - In test1.html, querySelectorAll returns a correct result, ['<div id="last">last</div>].
  - In test2.html, querySelectorAll returns an empty list, [], which is wrong result.

This might be a regression because Safari Version 5.0 (6533.16) returns the same correct results for both test1.html and test2.html.

[Hayato Ito]

Created attachment 69769 [details]
test2.html

[Adam Barth]

We had trouble with the last-child selector early on when developing the new tree builder.  There's some API we need to call on an element when it's received its last child (as I recall).  I'll possible we're screwing that up in this case.

The newline at the end of the file gets moved to just inside the body element, which might be confusing something.

Thanks for the report.  I'll take a look.

[Adam Barth]

Looking now.

[Adam Barth]

Created attachment 72148 [details]
Patch

[Eric Seidel (no email)]

Comment on attachment 72148 [details]
Patch

OK.  Your test case is really subtle.  It should document that it's important to not have a newline at the end.

Would probably be clearer if you document.wrote the contnet into an ifram.

[Eric Seidel (no email)]

You mentioned there was a spec bug?  What's the bug?

[Adam Barth]

(In reply to comment #6)
> You mentioned there was a spec bug?  What's the bug?

It's for one of the FIXME cases.  It's just a typo in the spec.

[Adam Barth]

Committed r71147: <http://trac.webkit.org/changeset/71147>