Japanese characters for Web page title display as trash in printer window's document name column

VERIFIED FIXED in mozilla1.0

Status

()

P2
major
VERIFIED FIXED
18 years ago
17 years ago

People

(Reporter: wesleyg, Assigned: rods)

Tracking

({intl})

Trunk
mozilla1.0
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

18 years ago
<summary>
Japanese characters for Web page title display as trash in printer window's 
document name column

<repro environment>
build 03-09-04 + Win2000-JA

not repro on:
IE5.5+ Win2000-JA

<repro step>
1. go to start|settings|printers, open the window for your default printer
2. open browser to http://www.toei.co.jp , or another Japanese web page with a 
title using Japanese characters
3. print the page, view the document name in the printer status window

<result>
Japanese characters for Web page title display as trash in printer window's 
document name column

Comment 1

18 years ago
Reassign to dcone, similar to bug 69610?
Assignee: nhotta → dcone
Keywords: intl

Updated

18 years ago
(Reporter)

Comment 2

18 years ago
Created attachment 27485 [details]
reference image
(Reporter)

Comment 3

18 years ago
The way the characters are trashed and where they appear would suggest this is 
not the same as #69610.

Comment 4

18 years ago
Changing QA Contact to ylong@netscape.com.
QA Contact: andreasb → ylong

Comment 5

18 years ago
per triage: reassigning to rods
Assignee: dcone → rods
Target Milestone: --- → mozilla0.9.1
(Assignee)

Comment 6

18 years ago
Created attachment 30755 [details] [diff] [review]
possible patch
(Assignee)

Comment 7

18 years ago
Erik, could you review this patch? I am not really sure about the UNICODE or 
MBCS parts of the patch. I think the UNICODE is fine but the MBCS is probably 
wrong. 

Casting to (wchar_t*) is probably bad.
Status: NEW → ASSIGNED

Comment 8

18 years ago
Instead of using the _UNICODE ifdef, we normally call the 'W' functions
directly. In this case, that would be ::StartDocW(). You can pass Unicode
strings to this function directly. QA can then test this on several platforms
(English/Japanese Windows 95/98/NT/2000) to see if StartDocW really works
everywhere.
(Assignee)

Updated

18 years ago
Target Milestone: mozilla0.9.1 → mozilla1.0

Comment 9

18 years ago
Nominating for nsbeta1 and M0.9.1. Looks like we have a proposed patch.
Keywords: nsbeta1

Comment 10

18 years ago
Rods - If we have a proposed patch, why are we waiting until 1.0 to fix this?

Comment 11

18 years ago
Rods/Vishy - Can someone pls explain the milestone of M1.0, if a patch already 
exist?
No idea, rod? 
(Assignee)

Comment 13

18 years ago
I am really not the right guy to have this bug. I AM the right guy to help out 
with this bug. It should really be someone else bug. Now that Erik is gone, who 
can help out with this?

I need someone to test the proposed patch.

Comment 14

18 years ago
Adding ftang to cc: list. 

Ftang - Can you help out on this one. Seems like rods needs help, and this may
be out of his area of expertise.

Updated

18 years ago
Keywords: nsbeta1 → nsbeta1-, nsCatFood

Comment 15

18 years ago
Per Vishy's request, setting priority P2 and milestone to M0.9.2, so that we can 
triage.
Severity: normal → major
Priority: -- → P2
I believe this was fixed by Rod checkin to fix localization issues with the
print dialog a few weeks ago. Marking Fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 17

17 years ago
Fixed verified on 08-22 trunk build on Win2k-CN.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.