903524 - Rename TestJSHolder

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Andrew McCreight [:mccr8]]

"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.

Comment 1

[Andrew McCreight [:mccr8]]

Attachment: rename it

Comment 2

[Andrew McCreight [:mccr8]]

try run: https://tbpl.mozilla.org/?tree=Try&rev=ef9284fca0eb

https://hg.mozilla.org/integration/mozilla-inbound/rev/67f3cc2de725

Comment 3

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/67f3cc2de725