[Webkit-unassigned] [Bug 5492] XSLTProcessor ignores namespaces

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=5492





--- Comment #2 from Scott Trenda <korisu at gmail.com>  2010-04-12 08:21:09 PST ---
Created an attachment (id=53164)
 --> (https://bugs.webkit.org/attachment.cgi?id=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

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.