Closed Bug 582751 Opened 14 years ago Closed 14 years ago

list properties in leaked objects with generic toString methods

Categories

(Add-on SDK Graveyard :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: myk, Assigned: myk)

Details

Attachments

(1 file, 2 obsolete files)

Attached patch patch v1: implements change (obsolete) — Splinter Review
Leak warnings about objects with generic toString methods should list the properties in the object to make it easier to identify the objects via duck typing.
Attachment #461018 - Flags: review?(avarma)
Comment on attachment 461018 [details] [diff] [review]
patch v1: implements change

Yeah, awesome!
Attachment #461018 - Flags: review?(avarma) → review+
Not to hijack this bug, but I've used this patch to print "bin" info along with leaked objects, in case you're interested...
I'm interested!  Here's a combined patch.
Assignee: nobody → myk
Attachment #461018 - Attachment is obsolete: true
Attachment #461022 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #461028 - Flags: review?(avarma)
Comment on attachment 461028 [details] [diff] [review]
patch v2: incorporates Drew's patch

Fine tag-teaming.
Attachment #461028 - Flags: review?(avarma) → review+
https://hg.mozilla.org/labs/jetpack-sdk/rev/9e28b016db36
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
The Add-on SDK is no longer a Mozilla Labs experiment and has become a big enough project to warrant its own Bugzilla product, so the "Add-on SDK" product has been created for it, and I am moving its bugs to that product.

To filter bugmail related to this change, filter on the word "looptid".
Component: Jetpack SDK → General
Product: Mozilla Labs → Add-on SDK
QA Contact: jetpack-sdk → general
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: