Bug 30715 – Fails on DOM-Level 3 test attrisid01

• Component: DOM
• Status: RESOLVED DUPLICATE

[Chang Shu]

Steps to Reproduce:
1. Launch WebKit browser.

2. Open the url
"http://www.w3.org/2004/04/ecmascript/level3/core/alltests.html"

3. Select the test "attrisid01" from the test drop-down list.

4. Click on "LoadJSUNIT" button to run the test

Expected Outcome:
-----------------
The test should pass(i,e Progress bar should displayed in Green color)

Actual Outcome:
---------------
Test fails and progress bar displayed in Red color

[Chang Shu]

Created attachment 41727 [details]
fix patch

[Eric Seidel (no email)]

Comment on attachment 41727 [details]
fix patch

LayoutTests/dom/xhtml/level3/core/attrisid01.xhtml

already exists.  Did you run the layout tests?  I don't think this needs an additional test.  Certainly the results for that test should be updated.

[Chang Shu]

(In reply to comment #2)
> (From update of attachment 41727 [details])
> LayoutTests/dom/xhtml/level3/core/attrisid01.xhtml
> 
> already exists.  Did you run the layout tests?  I don't think this needs an
> additional test.  Certainly the results for that test should be updated.

I didn't notice the test case exists already. I will remove my test case from the patch. However, running on Qt-linux, the existing attrisid test cases must have something wrong. They all passed successfully even without my implementation. To make sure I was doing the right thing, I changed the "assertFalse" to "assertTrue" in attrisid01.js, but the test results are both success. If I ran dumprendertree instead of run-webkit-tests, I could see the isid return value was "undefined", which is right. I guess I need additional effort to make the test case work.

[Chang Shu]

Created attachment 41743 [details]
using existing test cases

[Chang Shu]

(In reply to comment #3)
> (In reply to comment #2)
> > (From update of attachment 41727 [details] [details])
> > LayoutTests/dom/xhtml/level3/core/attrisid01.xhtml
> > 
> > already exists.  Did you run the layout tests?  I don't think this needs an
> > additional test.  Certainly the results for that test should be updated.

I enabled attrisid04 and attrisid05 so all attrisid cases will pass with my patch. However, I still think there is flaw in the test cases based on the fact that attrisid01 etc. cannot tell 'undefined' from 'false'. But these test cases seem to be written by w3c. When I ran these test cases manually on the website, I saw similar problem.

[Alexey Proskuryakov]

(In reply to comment #3)
> However, running on Qt-linux, the existing attrisid test cases must
> have something wrong. They all passed successfully even without my
> implementation.

That may be caused by a different version of framework used for converting generic XML-based DOM tests into a browser-friendly form (looks like assertFalse() is now more strict, and no longer accepts undefined values).

I do not think that we should add a naive isId implementation that just checks for attribute name. Which attribute name is to be an ID is actually defined by DTD, see bug 12971. We also have bug 5578 about adding support for isId() and setIdAttribute().

I think that bug 5578 is the closest thing, so I'll mark this as a duplicate of that bug.

*** This bug has been marked as a duplicate of bug 5578 ***

[Lucas Forschler]

Mass moving XML DOM bugs to the "DOM" Component.