265204 - transformToDocument() method don't process processing instructions

Status: RESOLVED INVALID

(Core :: XSLT, defect)

Description

[alexander :surkov (:asurkov)]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7) Gecko/20040616
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7) Gecko/20040616

I add processing instructions of stylesheet to my xslt document, in instance: 
<xsl:processing-instruction name="xml-stylesheet">
  href="chrome://global/skin.css" type="text/css"
</xsl:processing-instruction>
When I use trasformToDocument() method of XSLTProcessor, then stylesheet isn't
added to trasformed document i.e. styleSheets.length of document equals 0.

Reproducible: Always
Steps to Reproduce:

Comment 1

[Axel Hecht]

If you do this in html output method, then it is WORKSASDESIGNED.
HTML in Mozilla does not support processing instructions in general, so XSLT in
particular doesn't as well.

Comment 2

[alexander :surkov (:asurkov)]

No I don't specify output method, by default i guess it will be used xml mehtod.
What output method I should use?

Comment 3

[alexander :surkov (:asurkov)]

Attachment: xml file

Comment 4

[alexander :surkov (:asurkov)]

Attachment: xsl file

Comment 5

[alexander :surkov (:asurkov)]

Attachment: run it

Comment 6

[Axel Hecht]

Attachment: fix stylesheet to have just one document element

This is a modified stylesheet that adds a top level document element.
The problem is, that when adding the second top level element, transformiix
wraps
the result in a special element, reparenting the PI as well. Which removes it
from the stylesheet list.

Not sure if we should exclude PIs from the reparenting or if this is just 
WORKSFORME.

Note, XUL output may or may not work in any degree, and is completely 
unsupported.

Comment 7

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

hmm.. I actually think we're doing the right thing. What we're doing is
DoWhatYouSay rather then DoWhatYouMean which I think is what we should stick to
as much as possible. I don't see any usecases for having multiple roots so
making that work 'smootly' by leaving stylesheet PIs in the root seems like
unneccesary work. Note that leaving the PI in the root might not be the right
thing to do for other PIs...

Comment 8

[Gervase Markham [:gerv]]

This is an automated message, with ID "auto-resolve01".

This bug has had no comments for a long time. Statistically, we have found that
bug reports that have not been confirmed by a second user after three months are
highly unlikely to be the source of a fix to the code.

While your input is very important to us, our resources are limited and so we
are asking for your help in focussing our efforts. If you can still reproduce
this problem in the latest version of the product (see below for how to obtain a
copy) or, for feature requests, if it's not present in the latest version and
you still believe we should implement it, please visit the URL of this bug
(given at the top of this mail) and add a comment to that effect, giving more
reproduction information if you have it.

If it is not a problem any longer, you need take no action. If this bug is not
changed in any way in the next two weeks, it will be automatically resolved.
Thank you for your help in this matter.

The latest beta releases can be obtained from:
Firefox:     http://www.mozilla.org/projects/firefox/
Thunderbird: http://www.mozilla.org/products/thunderbird/releases/1.5beta1.html
Seamonkey:   http://www.mozilla.org/projects/seamonkey/

Comment 9

[alexander :surkov (:asurkov)]

confirm Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20050909
SeaMonkey/1.1a

Comment 10

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

marking as invalid per comment 7