1276309 - Stop relying on the precise value of nsIScriptContext::GetNativeContext

Status: RESOLVED FIXED

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

Description

[Boris Zbarsky [:bzbarsky]]

We obviously need this to move to one JSContext per runtime.

Current places that do this are:

1)  nsGlobalWindow::FinalClose.  I think we can switch this to GetEntryGlobal.

2)  nsScriptLoader::EvaluateScript in the module case.  This should be using aes.cx().

I think that's it.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: part 1. Stop depending on the exact value of nsIScriptContext::GetNativeContext in the scriptloader

Comment 2

[Boris Zbarsky [:bzbarsky]]

Attachment: part 2. Stop depending on the exact value of nsIScriptContext::GetNativeContext in nsGlobalWindow::FinalClose

Comment 3

[Boris Zbarsky [:bzbarsky]]

Attachment: part 2. Stop depending on the exact value of nsIScriptContext::GetNativeContext in nsGlobalWindow::FinalClose

Comment 4

[Ben Kelly [:bkelly, not reviewing]]

Comment on attachment 8757415 [details] [diff] [review]
part 1.  Stop depending on the exact value of nsIScriptContext::GetNativeContext in the scriptloader

Review of attachment 8757415 [details] [diff] [review]:
-----------------------------------------------------------------

Since aes is initialized with the same context, this should be equivalent AFAICT.

Comment 5

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

Comment on attachment 8757421 [details] [diff] [review]
part 2.  Stop depending on the exact value of nsIScriptContext::GetNativeContext in nsGlobalWindow::FinalClose

Doesn't the fix rely on us using AutoJSAPI vs AutoEntryScript correctly, since
AutoJSAPI ends up pushing stuff to stack, but only AutoEntryScript updates the entryglobal? AutoJSAPI vs. AutoEntryScript usage has been a bit ad hoc, but hopefully it is better these days.

So, better to push to try and see how this behaves.

Comment 6

[Boris Zbarsky [:bzbarsky]]

> Doesn't the fix rely on us using AutoJSAPI vs AutoEntryScript correctly

Yes, it does.  I did push this to try as part of https://treeherder.mozilla.org/#/jobs?repo=try&revision=ed3d2c0aa0df and it's looking pretty good.

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/eb35c54faae8
https://hg.mozilla.org/integration/mozilla-inbound/rev/6210926fe4f3

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/eb35c54faae8
https://hg.mozilla.org/mozilla-central/rev/6210926fe4f3