940317 - Move the varObjFix flag from ContextOptions to RuntimeOptions

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Comment 1

[Eddy Bruel [:ejpbruel]]

Attachment: patch

Try run for this patch:

https://tbpl.mozilla.org/?tree=Try&rev=881f76bea990

Comment 2

[Bobby Holley (:bholley)]

Comment on attachment 8334695 [details] [diff] [review]
patch

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

This is green. r=bholley with comments addressed.

::: js/src/jsapi-tests/tests.h
@@ +300,5 @@
>      virtual JSContext * createContext() {
>          JSContext *cx = JS_NewContext(rt, 8192);
>          if (!cx)
>              return nullptr;
> +        JS::RuntimeOptionsRef(cx).setVarObjFix(true);

This should not be in createContext, but in createRuntime.

Comment 3

[Bobby Holley (:bholley)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/db5ffe95df00

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/db5ffe95df00