Closed Bug 164940 Opened 22 years ago Closed 22 years ago

plaintext copy pastes Japanese

Categories

(Core :: DOM: Serializers, defect, P1)

Product:
Core
Component:
DOM: Serializers
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.2alpha

People

(Reporter: akkzilla, Assigned: alecf)

References

Details

(Keywords: smoketest)

Attachments

(2 files)

use ToNewUnicode()
617 bytes, patch
Details | Diff | Splinter Review
Alec's fix
557 bytes, patch
timeless
: review+
alecf
: superreview+
Details | Diff | Splinter Review
Seen in today's linux builds, opt & debug: Doubleclick in the urlbar to copy, then middlemouse in another window to paste. Xterm window: pastes ???????????????????? Composer window: locks up for a while then pastes a bunch of Japanese. Alecf already has a fix.
D&D is also busted (locks up the machine for a while, then the app eventually dies due to lack of resources). Alec suggested changing ToNewCString on line 125 of widget/src/xpwidgets/nsPrimitiveHelpers.cpp to ToNewUnicode, which does indeed fix the problem.
Keywords: smoketest
here's the fix, as tested by akkana
Attached patch Alec's fix — — Splinter Review
I just need a review and sr=
Status: NEW → ASSIGNED
Priority: -- → P1
Whiteboard: fix in hand
Target Milestone: --- → mozilla1.2alpha
Comment on attachment 96877 [details] [diff] [review] Alec's fix r=akkana, does fix the problem for xterm and composer, copy/paste and drag/drop.
Attachment #96877 - Flags: review+
Comment on attachment 96877 [details] [diff] [review] Alec's fix sr=alecf
Attachment #96877 - Flags: review+ → superreview+
*** Bug 164935 has been marked as a duplicate of this bug. ***
ok, fixed.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Comment on attachment 96877 [details] [diff] [review] Alec's fix looks good, r=timeless, we can call it akk's fix :)
Attachment #96877 - Flags: review+
Comment on attachment 96877 [details] [diff] [review] Alec's fix /r=yokoyama looks good
*** Bug 164771 has been marked as a duplicate of this bug. ***
*** Bug 164966 has been marked as a duplicate of this bug. ***
*** Bug 164965 has been marked as a duplicate of this bug. ***
*** Bug 164999 has been marked as a duplicate of this bug. ***
*** Bug 165003 has been marked as a duplicate of this bug. ***
Could someone please consider changing the OS and platform? I didn't find this bug as I tend not to look at PC bugs.
OS: Linux → All
Hardware: PC → All
changed platform and os to all
Whiteboard: fix in hand
*** Bug 165066 has been marked as a duplicate of this bug. ***
*** Bug 165099 has been marked as a duplicate of this bug. ***
*** Bug 165113 has been marked as a duplicate of this bug. ***
*** Bug 165167 has been marked as a duplicate of this bug. ***
*** Bug 165168 has been marked as a duplicate of this bug. ***
*** Bug 139592 has been marked as a duplicate of this bug. ***
*** Bug 164943 has been marked as a duplicate of this bug. ***
*** Bug 165448 has been marked as a duplicate of this bug. ***
*** Bug 165477 has been marked as a duplicate of this bug. ***
You really should add the words "clipboard" to this bug's title - that's why I couldn't find it when I searched for it.
*** Bug 165845 has been marked as a duplicate of this bug. ***
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: