1278947 - Remove the context callback

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jan de Mooij [:jandem]]

XPConnect uses the context callback for the NewContext and DestroyContext 'events', but now that there's one context we can just inline the code there and remove the callback mechanism.

Comment 1

[Jan de Mooij [:jandem]]

Attachment: Part 1 - Remove cx callback from XPConnect

Comment 2

[Jan de Mooij [:jandem]]

Attachment: Part 2 - Remove cx callback from JSAPI

This only removes lines.

Comment 3

[Jan de Mooij [:jandem]]

bz would you be able to look at part 1? When bholley is back from PTO he can review other XPConnect patches hopefully...

Comment 4

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8761286 [details] [diff] [review]
Part 1 - Remove cx callback from XPConnect

r=me.  This XPCContext stuff will all die in bug 1276310, but for the moment we're preserving behavior at least.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b974e754526b
part 1 - Stop using JSContext callback in XPConnect. r=bz
https://hg.mozilla.org/integration/mozilla-inbound/rev/f133ef6fc341
part 2 - Remove JSContext callback. r=luke

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b974e754526b
https://hg.mozilla.org/mozilla-central/rev/f133ef6fc341