75301 - Several converter bugs

Status: VERIFIED INVALID

(Core :: DOM: Serializers, defect)

Description

[Ben Bucksch (:BenB)]

During testing of bug 58377, I found several bugs in the html->txt converter,
some of them even with the "composer test page". I'll file them all in once,
because I'm too lazy to file lots of little bugs.

Comment 1

[Ben Bucksch (:BenB)]

Attachment: Testcase

Comment 2

[Ben Bucksch (:BenB)]

Attachment: Output. Trunk, before bugfix for bug 58377

Comment 3

[Ben Bucksch (:BenB)]

Attachment: Output. Trunk plus latest patch for bug 58377

Comment 4

[Ben Bucksch (:BenB)]

Problems:
(I'll note only those that are not fixed by patch for bug 58377.)
1. Quote "Reruns" end with space
2. Preformatted text "Here is some" has a space at the end of the line (already
in the HTML, i.e. entered by the user), which is put out. Must not happen.

I just realized that the output file as in the non=flowed mode. Produced via
Export As Text.... I guess, the same will appear in flowed as well.

More later.

Comment 5

[anthonyd]

moving to 0.9.1

Comment 6

[Kathleen :Brade]

Move this to mozilla0.9.2; anyone should feel free to help Tony out with this bug 
(unless he moves it back to 0.9.1).

Comment 7

[rubydoo123]

added joe and kin

Comment 8

[rubydoo123]

this may all be resolved with joes recent checkin, moving to 1.0 for eval

Comment 9

[Joe Francis]

Ben Bucksch wrote:
>1. Quote "Reruns" end with space
>2. Preformatted text "Here is some" has a space at the end of the line (already
>in the HTML, i.e. entered by the user), which is put out. Must not happen.

Ben, I can't tell what you are talking about.  It looks like you are saying 
preformatted spaces at end of lines should not be output by html->converter.  Why 
do you say that?  

Comment 10

[Daniel Bratell]

The converter is used for mail, and trailing spaces in f=f mails have a special 
meaning. There are code to strip those trailing spaces when the converter is in 
f=f mode, so the spaces Ben sees, might och might not be there in flowed mails. 
It will need more testing to find out.

Comment 11

[Ben Bucksch (:BenB)]

Joe, I know this bug is messy. Please excuse. I intended to make more checks and
update it then, but never did. Feel free to ignore it until (if ever) I clarify it.

Comment 12

[anthonyd]

judging from joe's comments, this seems to be a un-verified whitespace issue.  
I'm gonna reassing it to joe with the understanding that if this ever gets 
verified and is a converter/serializer bug.

anthonyd

Comment 13

[Joe Francis]

closing out for lack of bug description

Comment 14

[Ben Bucksch (:BenB)]

VERIFIED MESSY.