17595 - XML document without stylesheet not displayed

Status: VERIFIED FIXED

Description

[David Baron :dbaron: 🏴󠁵󠁳󠁣󠁡󠁿 ⌚UTC-8 (if account gets disabled due to email bounces, ask a bugzilla admin to reenable it)]

DESCRIPTION: XML documents without a stylesheet no longer show up.  They should,
in particular when they use namespaces (HTML, MathML, XUL) about which Mozilla
has knowledge.

STEPS TO REPRODUCE:
 * load first of attach test cases

ACTUAL RESULTS:
 * some leftover painting

EXPECTED RESULTS:
 * "This is an XML-ish document."

DOES NOT WORK CORRECTLY ON:
 * Linux, apprunner, 1999-10-29-10-M11

WORKS CORRECTLY ON:
 * older builds

Comment 1

[David Baron :dbaron: 🏴󠁵󠁳󠁣󠁡󠁿 ⌚UTC-8 (if account gets disabled due to email bounces, ask a bugzilla admin to reenable it)]

Created attachment 2493 [details]
simple XML document

Comment 2

[David Baron :dbaron: 🏴󠁵󠁳󠁣󠁡󠁿 ⌚UTC-8 (if account gets disabled due to email bounces, ask a bugzilla admin to reenable it)]

Created attachment 2494 [details]
similar document that works (added stylesheet and spread DOCTYPE to multiple lines)

Comment 3

[David Baron :dbaron: 🏴󠁵󠁳󠁣󠁡󠁿 ⌚UTC-8 (if account gets disabled due to email bounces, ask a bugzilla admin to reenable it)]

Created attachment 2495 [details]
similar document that works (added stylesheet and spread DOCTYPE to multiple lines)

Comment 4

[Nisheeth Ranjan]

Accepting bug and setting milestone to M11 for now...

Comment 5

[Nisheeth Ranjan]

Re-assigning to Rick as per our conversation.  The HTML DTD is being selected
over the XML DTD to handle this document because the document contains "html
doctype public" inside it.  BufferContainsHTML() probably needs to be changed to
check for the XML Decl and return FALSE if it is found.

Comment 6

[rickg]

This is now fixed in my tree, awaiting m12 opening.

Comment 7

[David Baron :dbaron: 🏴󠁵󠁳󠁣󠁡󠁿 ⌚UTC-8 (if account gets disabled due to email bounces, ask a bugzilla admin to reenable it)]

Why isn't the MIME type text/xml enough?

Comment 8

[chris hofmann]

if this doesn't arrive then lets move target milestone to m12

Comment 9

[rickg]

Mime types can be misleading. If you tell me you have an XML file, I still need
to determine whether I have a DTD for that XML application (or not). What's
more I've seen HTML files given as text/plain.

Comment 10

[rickg]

Fixed by autodetection code in CNavDTD. The code was too aggressive about taking
ownership of a document if it spotted HTML.

Comment 11

[Christine Hoffman]

Using 11/16 build on Linux, verified fixed. XML page displays.