404419 - XML not displayed

Status: VERIFIED FIXED

(Core :: XML, defect, P2)

Description

[Nick Thomas [:nthomas] (UTC+12)]

When trying to view update information from aus2.m.o (see URL), there is a blank page instead of displayed XML, and this error in the Error Console:

Error: 
Source File: jar:file:///Users/nrthomas/Desktop/20071119_0331.app/Contents/MacOS/chrome/toolkit.jar!/content/global/xml/XMLPrettyPrint.xsl
Line: 48, Column: 99
Source Code:
    <link href="chrome://global/content/xml/XMLPrettyPrint.css" type="text/css" rel="stylesheet"/>

For mac, this occurs in the build from 20071119_0331 but not 20071119_0253. Which points to bug 401613, although bug 403678 also landed in this window.

Comment 1

[Peter Van der Beken [:peterv]]

Attachment: v1

Maybe we're not ready yet for stopping the parser on errors. nsStyleLinkElement::DoUpdateStyleSheet was returning the NS_ERROR_NOT_AVAILABLE from CSSLoaderImpl::LoadInlineStyle/LoadStyleLink. Bailing out early if the CSSLoader is disabled works.

Comment 2

[Peter Van der Beken [:peterv]]

nsScriptElement sanitizes the error codes that it returns (see http://bonsai.mozilla.org/cvsblame.cgi?file=/mozilla/content/base/src/nsScriptElement.cpp&rev=3.9&mark=190-196#188). Maybe we should do the same thing in nsStyleLinkElement::DoUpdateStyleSheet too?

Comment 3

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 289479 [details] [diff] [review]
v1

No need for the else-after-return; other than that looks fine.

But you're right; we should consider sanitizing errors here...  but on the other hand, if there's an OOM in CSSLoader, maybe we do want to abort the XML parser.

Perhaps the CSSLoader should silently return NS_OK when disabled?

Comment 4

[Boris Zbarsky [:bzbarsky]]

Actually, silent NS_OK could be tough given the API.  So I think this patch is the way to go.

Comment 5

[Jeff Walden [:Waldo]]

http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&subdir=mozilla/layout/reftests/bugs&command=DIFF_FRAMESET&file=reftest.list&rev1=1.221&rev2=1.222&root=/cvsroot