24346 - src attribute of img can't have leading whitespace

Status: VERIFIED FIXED

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

Description

[Steven de Hoog]

I get the alt contents when trying to view any of these images.
Example: <IMG SRC="   test.gif" alt="booboo">
This should be allowed however (according to rfc-2396, which is referenced in
the html 4.01 specs).

Comment 1

[pnunn]

I'm guessing this is a job for htmlparser people...
Rickg, do you know who handles parser/token issues?
-p

Comment 2

[rickg]

No -- the "parser-people" are agnostic when it comes to how well formed strings
like this are. However, I'll make a change for you in the html document to
correct for this problem. In the future, it'd be great if you could get familiar
with this code.

Comment 3

[harishd]

Stealing the bug.

Comment 4

[rickg]

Stealing this one back.

Comment 5

[Steven de Hoog]

After some further tests, it seems that all URL parameters are affected,
including the HREF param. in anchors (ex.: <A HREF="    thisfails.txt">link</A>)
So this would indeed seem to be a parser bug, rather than ImageLib.

Comment 6

[rickg]

ACK! The parser should not be changing your input in this way; it needs to 
accurately reflect the input stream. The content sink can, and the fix will be 
done there.

Comment 7

[rickg]

Fixed in my tree awaiting chance to land.

Comment 8

[rickg]

Fixed by change to contentsink where we strip leading whitespace.

Comment 9

[Steven de Hoog]

Verified. thx