179231 - importing a stylesheet that imports another stylesheet doesn't work (templates are ignored)

Status: RESOLVED FIXED

(Core :: XML, defect)

Description

[Silverbanana]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.2b) Gecko/20021016
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.2b) Gecko/20021016

Assume a XHTML-page uses Stylesheet A which imports (via <xsl:import>) another
stylesheet B which again imports a stylesheet C, then any templates (and other
stuff) inside of stylesheet C will be ignored.

While Mozilla 1.1 processed cases like this correctly Mozilla 1.2b does not.

You may get a small test suite that demonstrates the problem:
http://home.tu-clausthal.de/~ifbf/mozilla-test/mozilla-test.zip

All test pages (test1.xml to test3.xml) should render similar, but they don't
for this bug.

Reproducible: Always

Steps to Reproduce:
1. Just take a look at http://home.tu-clausthal.de/~ifbf/mozilla-test/test3.xml
and compare it with http://home.tu-clausthal.de/~ifbf/mozilla-test/test2.xml and
test1.xml

Actual Results:  
Page 3 doesn't render like Page 2 or Page 1

Expected Results:  
All should render pretty equal

I tested only Mozilla 1.2b for win32 but I can't imagine that this is an
OS-specific problem.

Comment 1

[Axel Hecht]

this is due to the fact that 
@mozilla.org/content/syncload-dom-service;1
doesn't set the mDocumentBaseURI of nsDocument.
Not sure if it should use nsDocument::ResetToURI or so.

XSLT is wrong component, but there ain't a right one. Reassigning to sicking, he
did that. Anybody else may cheer and grab that bug, I guess.

Comment 2

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

i'll investigate. This sounds bad

Comment 3

[Axel Hecht]

*** Bug 182172 has been marked as a duplicate of this bug. ***

Comment 4

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

*** Bug 187490 has been marked as a duplicate of this bug. ***

Comment 5

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

The problem is that the syncloader doesn't reset the document so it therefor
doesn't have a base-uri (and probably misses a few other things as well).
However just chaning aReset from PR_FALSE to PR_TRUE in the call to
StartDocumentLoad since that will drop the loadlistener that we have attached.

Unfortunatly it is dangerous to add the loadlistener after calling
StartDocumentLoad since if that fails we will be left with a circular ownership
between sink->document->scriptloader->sink.

I've removed that circle in bug 188729 so adding a dependency on that. Once that
is fixed it is however easy enough to just attach the loadlistener after
StartDocumentLoad is called.

Comment 6

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

*** Bug 174372 has been marked as a duplicate of this bug. ***

Comment 7

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

*** Bug 184249 has been marked as a duplicate of this bug. ***

Comment 8

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

Attachment: patch to fix

this should not go in until bug 188729 is fixed

Comment 9

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 111321 [details] [diff] [review]
patch to fix

mmmm shorter code.... (add a one-line comment about why we start the load
before attaching the listener, ok?)

Comment 10

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

fix checked in

Comment 11

[Axel Hecht]

*** Bug 190299 has been marked as a duplicate of this bug. ***

Comment 12

[Axel Hecht]

*** Bug 196153 has been marked as a duplicate of this bug. ***