Closed
Bug 403871
Opened 17 years ago
Closed 17 years ago
Make frame trees with i18n text readable
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
People
(Reporter: smontagu, Assigned: smontagu)
Details
Attachments
(1 file, 1 obsolete file)
|
2.58 KB,
patch
|
roc
:
review+
roc
:
superreview+
damons
:
approval1.9+
|
Details | Diff | Splinter Review |
For some reason frame tree dumps convert non-ASCII text in text frames to octal escapes. If they were Unicode escapes life would be so much easier.
|
Assignee | |
Comment 1•17 years ago
|
||
Attachment #288819 -
Flags: superreview?(roc)
Attachment #288819 -
Flags: review?(roc)
Can't you use nsPrintfCString or PR_sprintf here instead of manually translating?
|
|
Assignee | |
Comment 3•17 years ago
|
||
I suppose so, but I thought there was some benefit in being consistent. Feel free to quote Emerson at me :)
Consistent with what?
|
|
Assignee | |
Comment 5•17 years ago
|
||
Attachment #288819 -
Attachment is obsolete: true
Attachment #288825 -
Flags: superreview?(roc)
Attachment #288825 -
Flags: review?(roc)
Attachment #288819 -
Flags: superreview?(roc)
Attachment #288819 -
Flags: review?(roc)
Attachment #288825 -
Flags: superreview?(roc)
Attachment #288825 -
Flags: superreview+
Attachment #288825 -
Flags: review?(roc)
Attachment #288825 -
Flags: review+
|
|
Assignee | |
Comment 6•17 years ago
|
||
Comment on attachment 288825 [details] [diff] [review] Using nsPrintfCString Debug-only change.
Attachment #288825 -
Flags: approval1.9?
|
||
Comment 7•17 years ago
|
||
Comment on attachment 288825 [details] [diff] [review] Using nsPrintfCString a=release drivers.
Attachment #288825 -
Flags: approval1.9? → approval1.9+
|
|
Assignee | |
Comment 8•17 years ago
|
||
Checked in.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•17 years ago
|
Flags: in-testsuite-
|
||
Updated•6 years ago
|
Product: Core → Core Graveyard
|
||
Updated•6 years ago
|
Component: Layout: Misc Code → Layout
Product: Core Graveyard → Core
You need to log in
before you can comment on or make changes to this bug.
Description
•