829915 - Hide ParisBinding's SomeDictionaryInit(JSContext* aCx, ...) from non-autogenerated code?

Status: NEW

Description

[Olli Pettay [:smaug]]

As far as I see, Init() is a bit error prone. Nothing guarantees correct compartment.
So, might be better to use the current Init only in auto-generated code, and
add another Init for manual use. That new one should enter to the right compartment.

(Of course one shouldn't use Init manually, but not everything is converted to WebIDL yet)

Comment 1

[Boris Zbarsky [:bz] (still digging out from vacation mail)]

Are we talking about WebIDL dictionaries, or some other Init()?

Comment 2

[Olli Pettay [:smaug]]

Oops :) Dictionaries yes.

Comment 3

[Boris Zbarsky [:bz] (still digging out from vacation mail)]

OK.

So one problem is that I have no idea how to hide a member like that, short of making it private and making all the generated functions that use it friends...

Comment 4

[Olli Pettay [:smaug]]

What about just naming it InitInternal or some such to hint that it shouldn't be used normally,
and provide safer Init which does the compartment stuff etc.

Comment 5

[Boris Zbarsky [:bz] (still digging out from vacation mail)]

Ah, we could do that, yes.

That said, the only compartment thing going on is that cx, scopeObj, and val all need to be same-compartment, right?  I guess the concern is C++ consumers that will pass in a wrong-compartment scopeObj?  Or are we worried about cx not matching val too?

Comment 7

[Boris Zbarsky [:bz] (still digging out from vacation mail)]

So I guess what we could do is enter the compartment of scopeObj (assuming it's non-null) and then JS_WrapValue the val....

Want to write a patch?