Closed Bug 903524 Opened 9 years ago Closed 9 years ago

Rename TestJSHolder

Categories

(Core :: XPCOM, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla26

People

(Reporter: mccr8, Assigned: mccr8)

Details

Attachments

(1 file)

"TestJSHolder" isn't a very useful name.  I was thinking IsJSHolder, to match AddJSHolder and RemoveJSHolder.  It is a little weird that the nsContentUtils equivalents are AreJSObjectsHeld/HoldJSObjects/DropJSObjects instead, so I could align the cyclecollector functions with nsContentUtils if that makes sense.
Attached patch rename itSplinter Review
Attachment #790852 - Flags: review?(bugs)
Attachment #790852 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/67f3cc2de725
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
You need to log in before you can comment on or make changes to this bug.