This is kept alive for the lifetime of the browser, and adds 100 or so nodes to the CC graph.

I was trying to avoid adding code to nsContentUtils, but that would have required exposing the definition of nsIThreadJSContextStack to nsCCUncollectableMarker, which seems to be a somewhat private XPConnect class.

Comment on attachment 610368 [details] [diff] [review] fix if braces Try run looked good https://tbpl.mozilla.org/?tree=Try&rev=c9baf630c73c

Comment on attachment 610368 [details] [diff] [review] fix if braces >+nsContentUtils::TraceSafeJSContext(JSTracer *aTrc) >+{ >+ if (!sThreadJSContextStack) { >+ return; >+ } >+ JSContext* cx = nsnull; >+ sThreadJSContextStack->GetSafeJSContext(&cx); >+ if (!cx) { >+ return; >+ } >+ if (JSObject *global = JS_GetGlobalObject(cx)) { >+ JS_CALL_OBJECT_TRACER(aTrc, global, "safe context"); >+ } Nit, you're not consistent with *. JSContext* but JSObject * I prefer JSContext*

Good point. I'm bad about that. I changed them all to Foo*. https://hg.mozilla.org/integration/mozilla-inbound/rev/2a60f2559489