Bug 49281 – Remove broken code for embedded CSS xml-stylesheet

• Status: RESOLVED
• Resolution: FIXED
• Component: CSS

[Alexey Proskuryakov]

We have code in Document.cpp to support something like

<?xml-stylesheet href="#mystyle" type="text/css" ?>
<foo id="mystyle">heading { color: red; }</foo>

It's not clear to me whether this is supposed to work. The spec <http://www.w3.org/TR/xml-stylesheet/> only says:

---------------------------------
It is possible to associate a style sheet that is in the same document as the xml-stylesheet processing instruction by using just a fragment identifier in the href pseudo-attribute.

For details on how this is managed in XSLT, see 3.11 Embedded Stylesheet Modules in [XSLT].
---------------------------------

One needs an actual xsl:stylesheet element to reference. For CSS, the analog would be a <style> element, but it makes no sense to reference it - it works by the virtue of being in document.

I couldn't make a reference like this work in Firefox. This code is completely untested, and I think that it's wrong, because it sets stylesheet owner node to document, but assigns it to processing instruction node.

[Alexey Proskuryakov]

https://bugzilla.mozilla.org/show_bug.cgi?id=61675

[Alexey Proskuryakov]

So, it's a bit more useful for Mozilla, because you can style arbitrary XML, not just XHTML. But there is no clarity on how this should work, and our implementation is definitely broken.

[Alexey Proskuryakov]

Created attachment 73423 [details]
proposed patch

[Alexey Proskuryakov]

Committed <http://trac.webkit.org/changeset/71684>.