Summary: | Browser is failing 16 baseline test from Selector API Test Suite | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Abhijit <abhijitgupta312> | ||||||
Component: | WebCore JavaScript | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | abarth, ademar, commit-queue, eric, hao.1.lu, jwieczorek, nokiabugz, sam, suresh.voruganti, webkit.review.bot, yael | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | PC | ||||||||
OS: | Windows XP | ||||||||
Attachments: |
|
Description
Abhijit
2010-07-26 05:09:29 PDT
*** Bug 42975 has been marked as a duplicate of this bug. *** A fix for this would be simple (add the RequiresAllArguments flag to the IDLs) but I don't think these failing test cases are valid. At least neither the Selectors API nor the WebIDL specifications specify what should happen when an argument is left out. Venkat Penukonda added a comment - 12/Aug/10 11:45 AM These are not test case issues. Earlier version of FireFox browser (upto 3.6.8) used to fail all these 16 tests. But they all pass with latest FireFox (4.0 Beta) version. This means, mozilla did some fixes in their latest version to pass these tests. Opera 10.61 also passes all these tests. Hence, in my opinion, webkit too need to treat this as a valid bug and fix the issue. (In reply to comment #2) > A fix for this would be simple (add the RequiresAllArguments flag to the IDLs) but I don't think these failing test cases are valid. At least neither the Selectors API nor the WebIDL specifications specify what should happen when an argument is left out. FireFox 4, IE 9 and Opera pass these tests. I think webkit based browsers should pass them too. Created attachment 77113 [details]
Patch.
Throw an exception if there is no argument to querySelector or to querySelectorAll. This follows the same behavior as Firefox 4.07b, IE 9 beta and Opera 10.63.
Actually, 17 tests are failing, so after this patch, one test still fails, and it should be fixed separately.
I am not sure we should take this until it is worked out in the working group what the default stringification behavior is. (In reply to comment #6) > I am not sure we should take this until it is worked out in the working group what the default stringification behavior is. WebIDL is still under discussion, however this test suite was adopted by W3C. It is linked from http://www.w3.org/TR/selectors-api/#sotd. And available at http://dev.w3.org/2006/webapi/selectors-api-testsuite/003.xhtml#target The :target selector test is failing because when the test was brought in, setting window.location.href="#target" was commented out. When I un-commented those 2 lines, the test passed both with Qt port and mac port. Sam, do you remember why are those lines commented out? Comment on attachment 77113 [details] Patch. View in context: https://bugs.webkit.org/attachment.cgi?id=77113&action=review Otherwise looks fine. > WebCore/ChangeLog:8 > + Throw an exception if there is no argument to querySelecotr or querySelecotr Created attachment 77493 [details]
Patch, fixing the typo in querySelector.
Comment on attachment 77493 [details] Patch, fixing the typo in querySelector. Clearing flags on attachment: 77493 Committed r74681: <http://trac.webkit.org/changeset/74681> http://trac.webkit.org/changeset/74681 might have broken Leopard Intel Debug (Tests) (In reply to comment #12) > http://trac.webkit.org/changeset/74681 might have broken Leopard Intel Debug (Tests) I think this is a false alarm. fast/blockflow/broken-ideograph-small-caps.html is failing, but it was also failing yesterday, see <http://build.webkit.org/results/Leopard%20Intel%20Debug%20%28Tests%29/r74679%20%2825145%29/results.html> . And it is not related to the change in this patch. Please cherry pick the fix for Qtwebkit 2.2 Revision r74681 cherry-picked into qtwebkit-2.2 with commit 8c22a83 <http://gitorious.org/webkit/qtwebkit/commit/8c22a83> |