1111076 - Get rid of nsIScriptContext::GC()

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Andrew McCreight [:mccr8]]

This method is just a thin wrapper around PokeGC, with an incorrect comment, so I feel like we should just get rid of it and call PokeGC directly.

Comment 1

[Andrew McCreight [:mccr8]]

Attachment: Get rid of nsIScriptContext::GC().

Only lightly tested so far.

Comment 2

[Andrew McCreight [:mccr8]]

Comment on attachment 8535895 [details] [diff] [review]
Get rid of nsIScriptContext::GC().

try: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=d9a4f7026034

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8535895 [details] [diff] [review]
Get rid of nsIScriptContext::GC().

Update IID of nsIScriptContext

Comment 4

[Andrew McCreight [:mccr8]]

I updated the IID and removed some trivial trailing whitespace I noticed in nsIScriptContext.h.

Comment 5

[Andrew McCreight [:mccr8]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/158a3009efa2

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/158a3009efa2