5492 2005-10-24 21:53:49 -0700 XSLTProcessor ignores namespaces 2021-01-15 08:27:34 -0800 1 1 1 Unclassified WebKit XML 420+ All All NEW P4 Normal --- 3275 1 eric webkit-unassigned ap ian kings14x korisu lamargoddard oldest_to_newest 22872 0 eric 2005-10-24 21:53:49 -0700 At least getParameter, setParameter and removeParameter ignore namespaces. There are currently test cases to show this. The reason why I left these failing as they are, is that I'm not sure libxml2 supports namespaces... infact, I'm not even sure what "namespace" support would mean for parameters. "prefix" suport makes some sense. But does that mean that we are expected to translate from namespace URIs to prefixes when used? It's not clear at all what mozilla does in this case, or what they indended when they wrote their documentation. 52848 1 10100 lamargoddard 2006-08-17 12:23:06 -0700 Created attachment 10100 Zip archive of 1 xsl and 1 xhtml test files It looks like it's not just the parameters that ignore namespaces. Elements don't have their namespace set properly for the transform. The attached identity_new_class.xsl file is the identity transform but it also adds a class to each node if a class isn't already present. If you load the attached xsltprocessor_namespace.xhtml file in WebKit it does the transform, but because the namespace isn't properly passed to the transform, the resulting nodes have no namespace and are treated like plain xml. Also because the parameters don't pass in their namespace, the xsl:param doesn't match and it gets set to the empty string. It behaves as expected when loading the xsltprocessor_namespace.xhtml file in FireFox showing an unordered list with two blue list items "Does" and "Work". 211609 2 53164 korisu 2010-04-12 08:21:09 -0700 Created attachment 53164 Another file showing the incorrect behavior of WebKit's setParameter method. +1 for this, although it looks like no one's seen this bug in at least two years. It's appearing in Chrome as well. It looks like these lines in the source note this bug and are ultimately to blame: void XSLTProcessorImpl::setParameter(DOMStringImpl *namespaceURI, DOMStringImpl *localName, DOMStringImpl *value) { // FIXME: namespace support? m_parameters.replace(DOMString(localName).qstring(), new DOMString(value)); } SharedPtr<DOMStringImpl> XSLTProcessorImpl::getParameter(DOMStringImpl *namespaceURI, DOMStringImpl *localName) const { // FIXME: namespace support? if (DOMString *value = m_parameters.find(DOMString(localName).qstring())) return value->impl(); return 0; } void XSLTProcessorImpl::removeParameter(DOMStringImpl *namespaceURI, DOMStringImpl *localName) { // FIXME: namespace support? m_parameters.remove(DOMString(localName).qstring()); } Please get this taken care of with all appropriate speed. For the record, Firefox and Opera return the correct result from the transformation. These are the outputs from the most current versions of the following browsers as of 4/12/2010: Firefox/Opera: XSLTProcessor param namespace test $param = non-namespaced param $ns1:param = namespace-1 param $ns2:param = namespace-2 param XSLTProcessor param namespace test $param = non-namespaced param $ns1:param = not set $ns2:param = not set Chrome/Safari: XSLTProcessor param namespace test $param = namespace-2 param $ns1:param = not set $ns2:param = not set XSLTProcessor param namespace test $param = non-namespaced param $ns1:param = namespace-1 param $ns2:param = namespace-2 param 211613 3 korisu 2010-04-12 08:25:15 -0700 Also worth noting that this happens on Chrome and Safari in Windows as well, not just OSX. 1719881 4 kings14x 2021-01-15 08:27:34 -0800 hey im new at filling bugs but i have quite a few issues i need to discuss 10100 2006-08-17 12:23:06 -0700 2006-08-17 12:23:06 -0700 Zip archive of 1 xsl and 1 xhtml test files xslt_ns.tgz application/octet-stream 1043 lamargoddard H4sIAF+75EQAA+1W62/bNhA3BgxF8gfsM6sNsFMkounn4sru0NhYCyRpkGio9ylgRdpWR4mqSMUx uv7vO0pW7LyaPZIt7fgDBFG8J493unPxuRL6NKYRVwkNOK7cP+qAbrsNb0JIu1l8dzr5e4kKqbe6 hDTbrS7wkXYDyKj9AL5cQ6Y0TRGqTCVjNGW38t1FLw9Svr8QuFfv3z0NGY91qBenMZ+fBoIq5QLP P7Bh4tHKQ7O6/7VQ1UmjUd5/q9OE+2+QNqQL+leC+Gfvfz7j/HNB+ELvv/Ltd08q31QqBzRAb07Q GC1h9iob8DTgQfCY72Mj4I/G/vGzYPbfuWxxf7hW//de/UVpdFqtW///nVbnav13Sf2R1f9X+v/3 XpxHAp3xVIUy7jvErTuIx4FkYTztO5me7PzovBhsepADPaUXgiv4D+rLApsIgZJY9YCp78y0TnoY z+dzd950ZTrFZHd3F49P9rGf0lhNZBqticx0dLtQTnUGm4X9hKY0QiZV+05O6eX56SBccmgeJYJq jiKqg1nf+enZ77FkvLYFOjZyjkAmC1gXH1TrNHyXAX+htFA3yGlnVGR8R06Q4oIHuu/8cMWkhy+p WClNErHYKT1RF/IrX7BxBq+8KdalhDkLvhzuwcPd/w3932wkqQy4UjJdUdz8/H/Hxl/q/y0C9d8k 7e4jq3/b/23//wpxrf7vvfrv7P+kC3tX6r9JGo+s/v/n/f/p8M2e/+vRCJk0QEe/vNx/vYecHYzf NvcwHvpDNH7lH+wj4hKMR4cOuqGl+8dFQyfECJRrl2kG7dnLFedDwR3jgGHqCWr843EuySkzLRXG VsEHMGj4R2UC17bQRQoj6MbawwUXsKsgDRON9CKB1q/5ucbv6Rktds28sDEPYybnroyFpAz10SSL Aw1hAqUfgbxxBomT8g9AgVkZjQ/2X4Hbx/xDBnZqW88NC5BdmfC4Vv155Fe3UfXm8RooEyoUXwkp HrNanAlRbBlTpjJLW5fPmLMYshtGiUz1ycXkUDO6Uji/jBUHD9dYFddHZqDimqe16ucibvzOXTWL aFG4XV05lglwi8kgi+Bs7pTrkeBm+XLxmtWqmVhjBS7gze3rchj05XApW8vWjjtJ6dRsrusujndo xijYc8v9pb1tpNNsGcNMuDAuwqZhhgjAaBXwvVkoWK1UvI2W5j6Zcay49zwvTPTW0yIwQyHwubOQ wfWhj4iFChQueiiWMX+OPhliGRggB1LItIe+r9cnORG0G51m0ity1XsnmRlDPQhdyKDOIKtz4b4z kTI35olwMJRceRgWxXfJUrjhDA6lXqMO3sr0t+W3hzNhrBVmwCrc4gMOkRYWFhYWFhYWFhYWFhYW FhYWFhaPEn8Aeg3xZwAoAAA= 53164 2010-04-12 08:21:09 -0700 2010-04-12 08:21:09 -0700 Another file showing the incorrect behavior of WebKit's setParameter method. XSLTProcessor-namespace-param-test.html text/html 1875 korisu PGh0bWw+DQoJPGhlYWQ+DQoJCTx0aXRsZT5YU0xUUHJvY2Vzc29yIHBhcmFtIG5hbWVzcGFjZSB0 ZXh0PC90aXRsZT4NCgkJPHNjcmlwdCB0eXBlPSJ0ZXh0L2phdmFzY3JpcHQiPg0KCQkJd2luZG93 Lm9ubG9hZCA9IGZ1bmN0aW9uICgpIHsNCgkJCQl2YXIgeG1sRG9jID0gKG5ldyBET01QYXJzZXIp LnBhcnNlRnJvbVN0cmluZygiPHgvPiIsICJ0ZXh0L3htbCIpOw0KCQkJCXZhciB4c2xEb2MgPSAo bmV3IERPTVBhcnNlcikucGFyc2VGcm9tU3RyaW5nKCI8eHNsOnN0eWxlc2hlZXQgdmVyc2lvbj0n MS4wJyB4bWxuczp4c2w9J2h0dHA6Ly93d3cudzMub3JnLzE5OTkvWFNML1RyYW5zZm9ybScgeG1s bnM6bnMxPSduYW1lc3BhY2UtMScgeG1sbnM6bnMyPSduYW1lc3BhY2UtMic+IiArDQoJCQkJCQki PHhzbDpvdXRwdXQgbWV0aG9kPSdodG1sJyBlbmNvZGluZz0ndXRmLTgnIC8+IiArDQoJCQkJCQki PHhzbDpwYXJhbSBuYW1lPSdwYXJhbSc+bm90IHNldDwveHNsOnBhcmFtPiIgKw0KCQkJCQkJIjx4 c2w6cGFyYW0gbmFtZT0nbnMxOnBhcmFtJz5ub3Qgc2V0PC94c2w6cGFyYW0+IiArDQoJCQkJCQki PHhzbDpwYXJhbSBuYW1lPSduczI6cGFyYW0nPm5vdCBzZXQ8L3hzbDpwYXJhbT4iICsNCgkJCQkJ CSI8eHNsOnRlbXBsYXRlIG1hdGNoPScvJz4iICsNCgkJCQkJCQkiPGgxPlhTTFRQcm9jZXNzb3Ig cGFyYW0gbmFtZXNwYWNlIHRlc3Q8L2gxPiIgKw0KCQkJCQkJCSI8ZGl2PiRwYXJhbSA9ICIgKw0K CQkJCQkJCQkiPHhzbDp2YWx1ZS1vZiBzZWxlY3Q9JyRwYXJhbScgLz4iICsNCgkJCQkJCQkiPC9k aXY+IiArDQoJCQkJCQkJIjxkaXY+JG5zMTpwYXJhbSA9ICIgKw0KCQkJCQkJCQkiPHhzbDp2YWx1 ZS1vZiBzZWxlY3Q9JyRuczE6cGFyYW0nIC8+IiArDQoJCQkJCQkJIjwvZGl2PiIgKw0KCQkJCQkJ CSI8ZGl2PiRuczI6cGFyYW0gPSAiICsNCgkJCQkJCQkJIjx4c2w6dmFsdWUtb2Ygc2VsZWN0PSck bnMyOnBhcmFtJyAvPiIgKw0KCQkJCQkJCSI8L2Rpdj4iICsNCgkJCQkJCSI8L3hzbDp0ZW1wbGF0 ZT4iICsNCgkJCQkJIjwveHNsOnN0eWxlc2hlZXQ+IiwgInRleHQveG1sIik7DQoNCgkJCQl2YXIg eHNsID0gbmV3IFhTTFRQcm9jZXNzb3I7DQoJCQkJeHNsLmltcG9ydFN0eWxlc2hlZXQoeHNsRG9j KTsNCgkJCQl4c2wuc2V0UGFyYW1ldGVyKG51bGwsICJwYXJhbSIsICJub24tbmFtZXNwYWNlZCBw YXJhbSIpOw0KCQkJCXhzbC5zZXRQYXJhbWV0ZXIoIm5hbWVzcGFjZS0xIiwgInBhcmFtIiwgIm5h bWVzcGFjZS0xIHBhcmFtIik7DQoJCQkJeHNsLnNldFBhcmFtZXRlcigibmFtZXNwYWNlLTIiLCAi cGFyYW0iLCAibmFtZXNwYWNlLTIgcGFyYW0iKTsNCgkJCQlkb2N1bWVudC5ib2R5LmFwcGVuZENo aWxkKHhzbC50cmFuc2Zvcm1Ub0ZyYWdtZW50KHhtbERvYywgZG9jdW1lbnQpKTsNCg0KCQkJCXhz bC5yZW1vdmVQYXJhbWV0ZXIobnVsbCwgInBhcmFtIik7DQoJCQkJeHNsLnJlbW92ZVBhcmFtZXRl cigibmFtZXNwYWNlLTEiLCAicGFyYW0iKTsNCgkJCQl4c2wucmVtb3ZlUGFyYW1ldGVyKCJuYW1l c3BhY2UtMiIsICJwYXJhbSIpOw0KCQkJCXhzbC5zZXRQYXJhbWV0ZXIobnVsbCwgInBhcmFtIiwg Im5vbi1uYW1lc3BhY2VkIHBhcmFtIik7DQoJCQkJeHNsLnNldFBhcmFtZXRlcihudWxsLCAibnMx OnBhcmFtIiwgIm5hbWVzcGFjZS0xIHBhcmFtIik7DQoJCQkJeHNsLnNldFBhcmFtZXRlcihudWxs LCAibnMyOnBhcmFtIiwgIm5hbWVzcGFjZS0yIHBhcmFtIik7DQoJCQkJZG9jdW1lbnQuYm9keS5h cHBlbmRDaGlsZCh4c2wudHJhbnNmb3JtVG9GcmFnbWVudCh4bWxEb2MsIGRvY3VtZW50KSk7DQoJ CQl9DQoJCTwvc2NyaXB0Pg0KCTwvaGVhZD4NCgk8Ym9keT48L2JvZHk+DQo8L2h0bWw+