16091 - xml:space="preserve" not working

Status: VERIFIED WORKSFORME

(Core :: XML, defect, P3)

Description

[Christine Hoffman]

Using Commercial build 1999100618 Apprunner.

The attached XML file uses the xml:space attribute per the 2.10 section of the
XML 1.0 spec.
Expected result: The whitespace in the poem element should be preserved
Actual result: Parsing error occurs indicating document is not well-formed

Comment 1

[Christine Hoffman]

Attachment: Test case to demonstrate problem

Comment 2

[Christine Hoffman]

Incorrect test - new one submitted. Page displays but does not preserve
whitespace.

Comment 3

[Christine Hoffman]

Attachment: Test to demonstrate problem

Comment 4

[Nisheeth Ranjan]

Setting milestone to M13 for now...

Comment 5

[Hixie (not reading bugmail)]

The spec says (section 2.10):
#  the value "preserve" indicates the intent that applications preserve all
#  the white space.

We do this already. That is, the parser preserves all white-space regardless.
In fact, this is our 'default' behaviour -- see below.

The spec further says:
#  The value "default" signals that applications' default white-space
#  processing modes are acceptable for this element

So we need not do anything special based on the value of xml:space, since our
'default' behaviour and the 'preserve' behaviour are identical.

Note that the xml:space 'attribute' does NOT imply ANYTHING about how the
document is layed out -- that would be the role of the white-space CSS property.
We support that fully already. The xml:space attribute is merely saying that we
should preserve the white-space in the document when parsing and making up the
document tree of the XML file.

To show that the parser is preserving white-space, you can use a stylesheet
containing merely:

   * { white-space: pre; }

...you will then see that any element with xml:space='preserve' does indeed have
its white-space preserved... just like any element with xml:space='default'.
i.e., all of them.

I hope that's clear!

I'm marking this WORKSFORME. If I've made some mistake, please reopen...

Comment 6

[Christine Hoffman]

Adding 'verifyme' keyword

Comment 7

[Keyser Sose]

Verified
Platform: PC
OS: Windows 98
Mozilla Build: 2000101020 M18 Trunk Build