557197 - [HTML5] 
 in element content expands to CR and CRLF in the DOM shows up as a non-collapsing space in layout

Status: RESOLVED FIXED

(Core :: DOM: HTML Parser, defect, P2)

Description

[Ria Klaassen (not reading all bugmail)]

STR:

- Set html5.enable to true
- Go to URL

The first line of every comment is aligned correctly to the left, but not the next new lines.

Works: ac1df371f376
Fails: d91d66514c2f

Comment 1

[Ria Klaassen (not reading all bugmail)]

Query: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=ac1df371f376&tochange=d91d66514c2f

Comment 2

[Henri Sivonen (:hsivonen)]

This is a regression from bug 546641.

Not sure yet what to do. Options:
 * WONTFIX
 * Revert bug 546641
 * Expand 
 differently in element content and in attribute values.

Comment 3

[Henri Sivonen (:hsivonen)]

Oh, and there's the option of changing how layout deals with CR.

Note: If anyone is searching for duplicates before filing a bug due to seeing this problem on another site, please add a comment with the URL of that site here.

Comment 4

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

(In reply to comment #3)
> Oh, and there's the option of changing how layout deals with CR.

Bug 534071?

Comment 5

[Henri Sivonen (:hsivonen)]

(In reply to comment #4)
> (In reply to comment #3)
> > Oh, and there's the option of changing how layout deals with CR.
> 
> Bug 534071?

Looks like it. Good to see it's already on file.

Comment 6

[Henri Sivonen (:hsivonen)]

Attachment: Minimized test case

Curiously, the problem occurs when there's a CRLF in the DOM. A CR alone doesn't render as a space.

Comment 7

[Henri Sivonen (:hsivonen)]

Do the drivers consider this to be a 1.9.3 blocker? (Making this bug a blocker effectively means making the dependency bug 534071 a blocker.)

Comment 8

[Johnny Stenback (:jst)]

We do consider this a blocker (since it's a regression).

Comment 9

[Henri Sivonen (:hsivonen)]

Fixed by the landing of the dependency.