164201 - Quoted text in GB18030 gets repeated in reply or forwarded email

Status: VERIFIED DUPLICATE of bug 76190

(SeaMonkey :: General, defect, P1)

Description

[Jaime Rodriguez, Jr.]

Steps:

1. Launch the browser.
2. Open Mail Window, and Compose a mail form.
3. Use NeiMa method for the following unicode entry in email body:
"[01F9] [2FF4] [2E88] [345B] [3863] [3A4A] [4BC4] [0637] [A001] [0f00] [1820]
[32A3] [7A59] [6637] [9CD6] [3086] [0041]", 
4. Send the mail to yourself.
5. Open the email in your inbox.
6. Use right mouse click to highlight the whole string and roll the mouse to the
right while holding the mouse key. 
7. Click on Reply or Forward button.

Results: Quoted string appears twice. (part or the whole string concatenate
after the quoted part.)

Comment 1

[Frank Tang]

ok, the reproduce procedure does not apply to mozilla because they are talking
about another mailer.
below is the reproduce procedure for mozilla
"
This appears to be a browser issue - I have attached an html doc with the same
characters - if you open it in mozilla, select the line, copy using ctrl-c, and
then paste it (ctrl-v) in the url field, you can see the characters appear
multiple times.
(One note - you have to drag the mouse _past_ the end of the line of characters
when you do the select to see this problem.) 
"

Comment 2

[Frank Tang]

Attachment: a page created by that and encoded in utf8

I cannot reproduce this in mozilla

Comment 3

[Frank Tang]

not sure this is an valid bug, cannot reproduce this in any form yet

Comment 4

[Frank Tang]

Attachment: reproduceable html

Comment 5

[Frank Tang]

smontagu: could you please take a look at this. I can reproduce this with m1.0
branch 

I think the bidi cause this problem.  [0637]

Comment 6

[Frank Tang]

I don't really think this is an important bug for the origional target embedding
customer. This is a bi-di selection bug
and for that target market, it is not important

Comment 7

[Simon Montagu :smontagu]

cc-ing jkeiser. This may be caused by the problems we were discussing in
Conv_FE_06 or Conv_06_FE, which are called from nsCopySupport::HTMLCopy

Comment 8

[Simon Montagu :smontagu]

Ignore comment 7.

This problem only occurs when Bidi characters are present, and is a bug in the
Bidi visual selection code which causes the same range to be added to the
selection twice.

Attachment 97353 [details] [diff] in bug 76190 (which I will be checking in as soon as I get r=)
will fix this bug.

Comment 9

[Jaime Rodriguez, Jr.]

This is needed by a major embeddor. Let's get it reviewed, and landed on the
trunk asap.

Comment 10

[bobj]

Shouldn't this be marked fixed (on trunk) too?
We will need to land this on the 1.0 branch soon too.

Comment 11

[Simon Montagu :smontagu]

Yes, this is fixed now on trunk.

Comment 12

[Jaime Rodriguez, Jr.]

Changing to QA to [e-mail].

ji: pls verify this as fixed on the trunk. thanks!

simon: once ji has verified this as fixed on the trunk, please email drivers,
and edt requesting 1.0 branch approval. thanks!

Comment 13

[ji]

Tested on 2002-09-09-04-trunk build, now the copied string is only pasted once.
But there is one problem:
After the string is highlighted, keyboard is changed to Hebrew keyboard and it
stays on Hebrew keyboard even after switching to a different browser window. The
keyboard will be changed back to the previous one if only specifically clicking
the mouse to set focus. Reopened the bug.

Steps to reproduce:
1. Have one navigator window open, open a form, like this bug report, and set
cursor to the editable area, like Additional Comments field.
2. Have another composer window containing the string.
3. On Composer window, highlight the string, observe the keyboard, it's changed
to HE keyboard.
4. Switch window focus to the navigator window, the keyboard is still HE one.
Type anything, you can see bi-di is entered into the editing area.

Comment 14

[Simon Montagu :smontagu]

We already have bug 162242 on the behaviour described in comment 13. 

Comment 15

[ji]

Marked this bug as verified then. Thanks.

Comment 16

[Simon Montagu :smontagu]

reopening in order to dupe...

Comment 17

[Simon Montagu :smontagu]

Duping to bug 76190, because I will be checking in to branch from there, and we
already know that fixing that bug fixes this one.

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

Comment 18

[ji]

Marked it as verified.

Comment 19

[bobj]

> 3. On Composer window, highlight the string, observe the keyboard, it's changed
> to HE keyboard.
> 4. Switch window focus to the navigator window, the keyboard is still HE one.
> Type anything, you can see bi-di is entered into the editing area.

What happens if I do not have the HE input locale installed?  I assume that will
be the case for most Chinese users.

Comment 20

[ji]

If the system doesn't have HE input locale installed, then this problem doesn't
happen，The keyboard won't be changed when the string is selected.

Comment 21

[Jaime Rodriguez, Jr.]

Pls tranfers all remarks wrt this bug to the bug report it is filed as a dupe
aginst.