Closed
Bug 85355
Opened 24 years ago
Closed 24 years ago
XPCWrappedNative::ToString should sometimes yield '[object Foo]'
Categories
(Core :: XPConnect, defect)
Core
XPConnect
Tracking
()
VERIFIED
FIXED
mozilla0.9.2
People
(Reporter: jband_mozilla, Assigned: dbradley)
Details
(Whiteboard: approved)
Attachments
(3 files)
582 bytes,
patch
|
Details | Diff | Splinter Review | |
748 bytes,
patch
|
Details | Diff | Splinter Review | |
731 bytes,
patch
|
Details | Diff | Splinter Review |
I'm thinking that in the case where a wrappedNative object has a scriptable
helper (and thus a custom JSClass name) we should name the default ToString
function yield '[object Foo]' rather than '[xpconnect wrapped Foo]'. I imagine
this might give us better backward compatibility for misc. DOM objects. If
nothing else this might limit confusion.
This is easy to change and I think we should just do it before the next major
release.
Assignee | ||
Updated•24 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•24 years ago
|
||
Assignee | ||
Comment 3•24 years ago
|
||
Assignee | ||
Comment 4•24 years ago
|
||
Third times a charm. Here's the "final" patch. It tests based on the scriptable
info, not the class info.
Assignee | ||
Comment 5•24 years ago
|
||
Comment 6•24 years ago
|
||
sr=jst for the last patch.
Comment 7•24 years ago
|
||
r=beard
a=dbaron for trunk checkin (on behalf of drivers)
Assignee | ||
Updated•24 years ago
|
Whiteboard: aproved
Assignee | ||
Updated•24 years ago
|
Whiteboard: aproved → approved
Comment 9•24 years ago
|
||
Checked in.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•