fix jsapi-tests rooting hazards from static analysis

RESOLVED FIXED in mozilla21

Status

()

defect
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: sfink, Assigned: sfink)

Tracking

unspecified
mozilla21
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

rootingHazards.html identifies 96 rooting hazards in jsapi-tests/.
This patch fixes all of them, and passes |JS_GC_ZEAL=6 jsapi-tests| on my machine. 417 left.
Attachment #703527 - Flags: review?(terrence)
Blocks: 831409
Comment on attachment 703527 [details] [diff] [review]
fix jsapi-tests rooting hazards from static analysis

Review of attachment 703527 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good.
Attachment #703527 - Flags: review?(terrence) → review+
Attachment #703527 - Flags: checkin+
https://hg.mozilla.org/mozilla-central/rev/f5e034e13760
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
You need to log in before you can comment on or make changes to this bug.