85232 - Whitespace (CR/LF, tab) in attribute values breaks them.

Status: RESOLVED DUPLICATE of bug 47078

(Core :: Layout, defect)

Description

[Justin H.]

From Bugzilla Helper:
User-Agent: Mozilla/4.77 [en] (Win98; U)
BuildID:    20010607

Any CR/LF or tab characters in attribute values, and perhaps other whitespace 
chars, cause those values to become broken.  If the whitespace is in filenames, 
the attribute becomes completely broken.  If, in the case of URLs, the 
whitespace comes in the parameter section, the whitespace is %escaped and 
malforms the URL, breaking it (though I can't confirm in offline content).  All 
other browsers I tested simply ignore the whitespace.

Reproducible: Always
Steps to Reproduce:
1. Attached files show some simplified cases including background, href and src 
attributes.  Download and load in mozilla.

Actual Results:  The background doesn't show up, the image in the table doesn't 
show up, and the links are unusable.

Expected Results:  Mozilla should have ignored the CR/LF and tab chars and 
displayed the pictures.  The links should have been followable.

There are several bugs spread about that cover individual attributes, such as 
title and href, but the bug actually affects all attributes.  Doctype doesn't 
affect this bug.

Comment 1

[Justin H.]

Attachment: Simplified test case

Comment 2

[Justin H.]

Attachment: Banner image to go with test case

Comment 3

[Justin H.]

Attachment: Background image for test case.

Comment 4

[Chris Petersen]

Confirming..

Comment 5

[Andreas M. "Clarence" Schneider]

Looks invalid. See bug 42287.

Comment 6

[Justin H.]

No, not invalid.  I think it's just a duplicate of bug 47078.  From the comments
Ian made there, I think we're talking about the same thing.  All the attributes
I tested with were CDATA.

Would anyone care to confirm?  Just for my peace of mind.

Comment 7

[Andreas M. "Clarence" Schneider]

A fix for bug 47078 won't make the testcase work. There'd be still whitespace
inside the links.

Comment 8

[Justin H.]

I know, but at this point, there's not even an indication that the attributes
exist.  If we're going to break something, let's break it right - by the
standards. :-)

Besides, if the whitespace is normalized correctly, there should be broken image
placeholders and other indications that the attributes are broken.  As it
stands, Mozilla just pretends the attributes don't exist.

Also note that Ian said there'd be a difference in how these chars are handled
in quirks mode.  Even though that bug is just talking about CRLF, I still think
it's a dup.

Comment 9

[Justin H.]

*** This bug has been marked as a duplicate of 47078 ***