67090 - ASCII values 127 - 168 are not encoded correctly [form sub]

Status: VERIFIED WORKSFORME

(Core :: Layout: Form Controls, defect)

Description

[Vladimir Ermakov]

ASCII values 127 - 168 are being encoded as question marks on form submission. 
This does not happen on IE or Nav4.x. Currency symbol for Euros, British Pounds, 
and Japanese Yens, among other characters like Š(138), š(154), Œ(140), œ(156), 
and Ÿ(159), get corrupted during form submission, which will impact a lot of 
international users. 
The testcase at the url above illustrates the problem.

Comment 1

[rods (gone)]

I think necko does the necoding, is that right? reassiging

Comment 2

[Eric Pollmann]

It looks like the string is converted to question marks by
nsEncoderSupport::Convert, which is called by URLEncode inside of nsFormFrame.
???  Perhaps we should be manually URL encoding the string to prevent this from
happening?

Comment 3

[Eric Pollmann]

Even before we the characters are all converted to question marks, they are
aparently wrong - the string looks like this: .?...........?.??............?..
when we fetch it from the text control frame.  Perhaps the problem is there?

Comment 4

[msanz]

adding keywords; copying Frank and Erik

Comment 5

[J Luh]

This bug may be "invalid." In HTML 4.01, form input is restricted to the
US-ASCII charset unless enctype="multipart/formdata" is specified.

See http://www.w3.org/TR/html4/interact/forms.html#h-17.13.4

Comment 6

[Jaime Rodriguez, Jr.]

Erik is this bug still valid? Need your assessment.

Comment 7

[J Luh]

There's also bug 70838 on this issue.

Comment 8

[Erik van der Poel]

Jaime, please ask a QA person to test this in MSIE, Netscape 4.X and Mozilla.
I tried to take a look at the URL above, but that page doesn't seem to be there
any more. Perhaps vladimire would know about this?

Comment 9

[Vladimir Ermakov]

This is still happening. Sorry the testcase was renamed, I am updating the url.

Comment 10

[Jaime Rodriguez, Jr.]

changing to nsbeta1-.

Comment 11

[Jaime Rodriguez, Jr.]

Per Vishy's request, setting priority P2, so that we can triage.

Comment 12

[Blake Ross]

Missed 0.9.3.

Comment 13

[Kevin McCluskey (gone)]

Moving to Mozilla0.9.6

Comment 14

[Kevin McCluskey (gone)]

Bulk reassigning form bugs to Alex

Comment 15

[Alexandru Savulov]

performed test on URL with mozilla trunk pull from 10-29-2001 and the test passed

marking WORKSFORME

please verify
thx

Comment 16

[Vladimir Ermakov]

COOL! I am so happy this is fixed! Verifying on 2001-11-01-03 trunk
and linux 2001-10-31-08-trunk
:)