46788 - nsMimeBaseEmitter.cpp should output correct HTML

Status: VERIFIED FIXED

(MailNews Core :: MIME, defect, P3)

Description

[Henrik Gemal]

Richard, we (you) missing this file when you updated them to output correct 
HTML...:)
This bug is very much like bug 46234...

Comment 1

[Henrik Gemal]

Attachment: Diff to fix html TaG CAse part of bug

Comment 2

[Henrik Gemal]

Another problem in this bug it that the output looks like this:
<table border=0 cellspacing=0 cellpadding=0 width="100%">
<table border=0 cellspacing=0 cellpadding=0 width="100%" name="header-part1">
<tr>
<td><div class="headerdisplayname" style="display:inline;">subject:</div> 
jkjkjkj</td>
</tr>
</table>
</table>

which is not valid since the first table dont have any "tr" or "td".
Why is there a empty table around another table? Shouldn't it just be removed? I 
think so. I cant see any reason for this empty table...

Comment 3

[rhp (gone)]

Well, I worked on what you gave me patches for. I'll apply this one and get it 
in today.

- rhp

Comment 4

[rhp (gone)]

per selmer and mail/news PDT.

- rhp

Comment 5

[rhp (gone)]

Henrik,
Just checked in a fix here. I'm not touching the table issue at this point in 
the game. Sorry.

- rhp

Comment 6

[Henrik Gemal]

Verified on 20000729
Thanx for the checkin. I've filed bug 46975 on the table output bug.