Summary: | Node.lookupNamespaceURI doesn't convert empty strings to null | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | nanto_vi (TOYAMA Nao) <nanto> | ||||||||
Component: | DOM | Assignee: | Rob Buis <rwlbuis> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | annevk, ap, buildbot, bzbarsky, cdumez, commit-queue, esprehn+autocc, joepeck, kangil.han, mjs, rbuis, rniwa, rwlbuis, sam, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Attachments: |
|
Description
nanto_vi (TOYAMA Nao)
2009-11-01 05:59:22 PST
Yeah, <http://lists.w3.org/Archives/Public/public-webapps/2008JulSep/0126.html>. If Gecko isn't going to make this behavior strictly compatible with the letter of DOM 3 Core, we should probably bite the bullet and implement their quirk. You might be interested in https://bugzilla.mozilla.org/show_bug.cgi?id=312019 I'm also pretty sure the Gecko behavior for "" here is unintentional, for what it's worth. Created attachment 206117 [details]
Patch
Comment on attachment 206117 [details] Patch Attachment 206117 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/991244 New failing tests: svg/batik/text/smallFonts.svg Created attachment 206120 [details]
Archive of layout-test-results from webkit-ews-07 for mac-mountainlion
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-07 Port: mac-mountainlion Platform: Mac OS X 10.8.3
Created attachment 206152 [details]
Patch
Comment on attachment 206152 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=206152&action=review > Source/WebCore/dom/Node.cpp:1446 > // http://www.w3.org/TR/2004/REC-DOM-Level-3-Core-20040407/namespaces-algorithms.html#lookupNamespaceURIAlgo If we not longer plan on following DOM Level 3 here, we should update the comment. What's with the two failing tests? Is DOM4 in conflict with DOM3? Is that an intentional diversion? I don't recall the specifics. I guess I wrote down what Gecko implemented assuming the rule about null and the empty string in DOM Level 3 was also meant for prefixes and not just namespaces. (In reply to comment #9) > I don't recall the specifics. I guess I wrote down what Gecko implemented assuming the rule about null and the empty string in DOM Level 3 was also meant for prefixes and not just namespaces. The test suite differs I guess. Not sure how I feel here. Comment on attachment 206152 [details]
Patch
Sorry, this patch has grown stale.
I'm going to r- because, this should add a new test for the new expected behavior instead of just depending on the xhtml/level3 test failures. Perhaps this is now covered by `LayoutTests/imported/w3c/web-platform-tests/dom/nodes/Node-lookupNamespaceURI.html` and all we need to do is rebase the patch and re-run all tests.
Closing, this was fixed by: http://trac.webkit.org/changeset/204536 Mass moving XML DOM bugs to the "DOM" Component. |